Skip to content

R: bug in windows GUI/script editor (PR#8288)

3 messages · roberto.ugoccioni@sanpaoloimi.com, Duncan Murdoch

#
SGkgRHVuY2FuLA0KSSBhbSBydW5uaW5nIFIgaW4gTURJIG1vZGUgKHdpdGggbG9jYWxpc2F0
aW9uIG5vdCBpbnN0YWxsZWQgYmVjYXVzZSBJIG5lZWQgdG8gdXNlIFJXaW5FZHQpLCBvbiBh
biBJdGFsaWFuIHZlcnNpb24gb2YgV2luZG93cyAyMDAwIFByby4NCkkganVzdCBsb2dnZWQg
aW4gdG9kYXkgYW5kIHRyaWVkIHRvIHJlcHJvZHVjZSB0aGUgYnVnOiB0aGUgZmlyc3QgdGhy
ZWUgdGltZXMgYWxsIHdhcyBmaW5lLCB0aGUgZm91cnRoIG9uZSBJIGdvdCB0aGUgYnVnLCBh
bmQgZmluYWxseSBub3RpY2VkIG9uZSBkZXRhaWw6IHRoZSBidWcgYXBwZWFycyB3aGVuIEkg
b3BlbiBhbiBSIHNjcmlwdCAoSSB0cmllZCBzZXZlcmFsKSB3aGljaCBjYXVzZXMgdGhlICJw
YXJ0IG9mIHRoZSBzY3JvbGxiYXIgd2hpY2ggb25lIGNhbiBkcmFnIiBpbiB0aGUgaW50ZXJu
YWwgZWRpdG9yIHdpbmRvdyB0byBiZSByZXNpemVkIC0gaS5lLjoNCi0gdGhlIGVkaXRvciB3
aW5kb3cgb3BlbnMgdXAgKG5vdCBtYXhpbWlzZWQpIHdpdGggdGhlIHZlcnRpY2FsIHNjcm9s
bGJhciBhYm91dCAxLzMgb2YgdGhlIHdpbmRvdyBoZWlnaHQNCi0gdGhlIHNjcm9sbGJhciBn
ZXRzIHJlc2l6ZWQgdG8gYWJvdXQgaGFsZiBvZiB3aGF0IGl0IHdhcyAod2l0aG91dCBteSBp
bnRlcnZlbnRpb24pDQotIG5vdyBJIGp1c3QgY2xvc2UgdGhlIHdpbmRvdywgY2xpY2sgb24g
bWVudSBGaWxlIGFuZCB0aGVyZSBJIGdldCBiaXR0ZW4NCg0KTWF5IGl0IGJlIHRoZSBmaWxl
IHNpemU/ICd3YycgZ2l2ZXMgZm9yIGEgY291cGxlIG9mIGNhc2VzIGluIHdoaWNoIEkgaGF2
ZSBwcm9ibGVtczoNCiAgIDMzMyAgIDEwOTcgIDE0NjgxIEFuYWxpc2ktc3RkLTIuUg0KICAg
MzM3ICAgMTM5MSAgMTA1MzIgUi1yaW9wLWRpc3RyaWJ1dGlvbnMuUg0KDQpJIGFsc28gdHJp
ZWQgd2l0aCBwbGFpbiBSIChpLmUuLCBteSAuUnByb2ZpbGUgaXMgbm90IGxvYWRlZCk6IHNh
bWUgYmVoYXZpb3VyLg0KDQpIb3BlIHRoaXMgaGVscHM7IGxldCBtZSBrbm93IGlmIGFuZCB3
aGF0IG90aGVyIGRldGFpbHMgSSBjYW4gcHJvdmlkZS4NClJlZ2FyZHMsDQogIFJvYmVydG8N
Cg0KDQo+IC0tLS0tTWVzc2FnZ2lvIG9yaWdpbmFsZS0tLS0tDQo+IERhOiBEdW5jYW4gTXVy
ZG9jaCBbbWFpbHRvOm11cmRvY2hAc3RhdHMudXdvLmNhXQ0KPiBJbnZpYXRvOiBtYXJ0ZWTs
IDggbm92ZW1icmUgMjAwNSAxOTowMA0KPiBBOiBVZ29jY2lvbmkgUm9iZXJ0bw0KPiBDYzog
Ui1idWdzQGJpb3N0YXQua3UuZGsNCj4gT2dnZXR0bzogUmU6IFtSZF0gYnVnIGluIHdpbmRv
d3MgR1VJL3NjcmlwdCBlZGl0b3IgKFBSIzgyODgpDQo+IA0KPiANCj4gT24gMTEvOC8yMDA1
IDExOjUxIEFNLCByb2JlcnRvLnVnb2NjaW9uaUBzYW5wYW9sb2ltaS5jb20gd3JvdGU6DQo+
ID4gRnVsbF9OYW1lOiBSb2JlcnRvIFVnb2NjaW9uaQ0KPiA+IFZlcnNpb246IDIuMi4wDQo+
ID4gT1M6IFdpbmRvd3MgMjAwMA0KPiA+IFN1Ym1pc3Npb24gZnJvbTogKE5VTEwpICgxOTMu
MjAzLjIzMi41KQ0KPiA+IA0KPiA+IA0KPiA+IFJ1bm5pbmcgV2luZG93cyAyMDAwIFByb2Zl
c3Npb25hbCwgYWxsIHBhdGNoZXMgdXAgdG8gbm92IDgsIDIwMDUuDQo+ID4gDQo+ID4gDQo+
ID4+dmVyc2lvbg0KPiA+IA0KPiA+ICAgICAgICAgIF8gICAgICAgICAgICAgIA0KPiA+IHBs
YXRmb3JtIGkzODYtcGMtbWluZ3czMg0KPiA+IGFyY2ggICAgIGkzODYgICAgICAgICAgIA0K
PiA+IG9zICAgICAgIG1pbmd3MzIgICAgICAgIA0KPiA+IHN5c3RlbSAgIGkzODYsIG1pbmd3
MzIgIA0KPiA+IHN0YXR1cyAgICAgICAgICAgICAgICAgIA0KPiA+IG1ham9yICAgIDIgICAg
ICAgICAgICAgIA0KPiA+IG1pbm9yICAgIDIuMCAgICAgICAgICAgIA0KPiA+IHllYXIgICAg
IDIwMDUgICAgICAgICAgIA0KPiA+IG1vbnRoICAgIDEwICAgICAgICAgICAgIA0KPiA+IGRh
eSAgICAgIDA2ICAgICAgICAgICAgIA0KPiA+IHN2biByZXYgIDM1NzQ5ICAgICAgICAgIA0K
PiA+IGxhbmd1YWdlIFIgIA0KPiA+IA0KPiA+IEhvdyB0byByZXByb2R1Y2UgdGhlIGJ1ZzoN
Cj4gPiANCj4gPiAxLiBsYXVuY2ggUmd1aS5leGUNCj4gPiAyLiBtZW51IEZpbGUtPm9wZW4g
c2NyaXB0DQo+ID4gMy4gY2xvc2UgZWRpdG9yIGNsaWNraW5nIG9uIFgNCj4gPiA0LiBjbGlj
a2luZyBtZW51IEZpbGUgbm93IGdlbmVyYXRlcyB0aGUgZmF0YWwgZXJyb3IgKG9taXR0aW5n
IG1lbW9yeQ0KPiA+IGFkZHJlc3Nlcyk6DQo+ID4gDQo+ID4gIEFuIGluc3RydWN0aW9uIHJl
ZmVycmVkIHRvIGEgbWVtb3J5IGxvY2F0aW9uIHdoaWNoIGNvdWxkIA0KPiBub3QgYmUgInJl
YWQiDQo+ID4gDQo+ID4gQ2xpY2tpbmcgb24gT0sgaW4gdGhlIGVycm9yIG1lc3NhZ2Ugd2lu
ZG93IGNhdXNlcyBSIHRvIA0KPiBjb25zdW1lIDEwMCUgQ1BVIGFuZCBub3QNCj4gPiB0byBy
ZXNwb25kIC0gbXVzdCBiZSB0ZXJtaW5hdGVkIGZyb20gdGFzayBtYW5hZ2VyLg0KPiANCj4g
SSBkb24ndCBzZWUgdGhpcywgYnV0IGl0IHNvdW5kcyBsaWtlIHNvbWV0aGluZyB0aGF0IHdh
cyBmaXhlZCANCj4gYmVmb3JlIHRoZSANCj4gcmVsZWFzZSBvZiAyLjIuMCwgc28gbWF5YmUg
dGhlcmUncyBhbm90aGVyIHdheSB0byBnZW5lcmF0ZSANCj4gdGhlIHNhbWUgcHJvYmxlbS4N
Cj4gDQo+IENvdWxkIHlvdSBnaXZlIG1vcmUgZGV0YWlsOg0KPiANCj4gICAtIGFyZSB5b3Ug
cnVubmluZyBpbiB0aGUgZGVmYXVsdCBNREkgbW9kZSAob25lIGJpZyB3aW5kb3cgDQo+IGNv
bnRhaW5pbmcgDQo+IHRoZSBjb25zb2xlLCBlZGl0b3IsIGV0Yy4pIG9yIFNESSBtb2RlIChz
ZXBhcmF0ZSB3aW5kb3dzKT8NCj4gDQo+ICAgLSBkb2VzIGl0IG1hdHRlciB3aGF0IHdhcyBp
biB0aGUgZmlsZSB5b3Ugb3BlbmVkPw0KPiANCj4gRHVuY2FuIE11cmRvY2gNCj4gDQoNCg0K
SWwgIGNvbnRlbnV0byAgZSAgZ2xpIGFsbGVnYXRpICBkaSBxdWVzdG8gIG1lc3NhZ2dpbyAg
c29ubyAgc3RyZXR0YW1lbnRlDQpjb25maWRlbnppYWxpLCAgZSBuZSBzb25vIHZpZXRhdGkg
bGEgZGlmZnVzaW9uZSBlIGwndXNvIG5vbiBhdXRvcml6emF0by4NCg0KTGUgIG9waW5pb25p
ICBpdmkgIGV2ZW50dWFsbWVudGUgIGVzcHJlc3NlIHNvbm8gIHF1ZWxsZSAgZGVsbCdhdXRv
cmU6IGRpDQpjb25zZWd1ZW56YSAgaWwgIG1lc3NhZ2dpbyAgbm9uICBjb3N0aXR1aXNjZSAg
aW1wZWdubyAgY29udHJhdHR1YWxlICB0cmENCmlsIEdydXBwbyBTYW5wYW9sbyAgZWQgIGls
ICBkZXN0aW5hdGFyaW8sICAgZSAgbGEgIGJhbmNhICBub24gIGFzc3VtZSAgYWxjdW5hDQpy
ZXNwb25zYWJpbGl0YScgcmlndWFyZG8gYWkgY29udGVudXRpIGRlbCB0ZXN0byBlIGRlaSBy
ZWxhdGl2aSBhbGxlZ2F0aSwNCm5lJyBwZXIgZXZlbnR1YWxpIGludGVyY2V0dGF6aW9uaSwg
bW9kaWZpY2hlIG8gZGFubmVnZ2lhbWVudGkuIA0KIA0KUXVhbG9yYSBpbCBwcmVzZW50ZSBt
ZXNzYWdnaW8gTGUgZm9zc2UgcGVydmVudXRvIHBlciBlcnJvcmUsICBMZSBzYXJlbW1vDQpn
cmF0aSAgc2UgbG8gIGRpc3RydWdnZXNzZSBlLCAgdmlhIGUtbWFpbCwgIGNlIG5lIGNvbXVu
aWNhc3NlICBsJyBlcnJhdGENCnJpY2V6aW9uZSBhbGwnaW5kaXJpenpvIHBvc3RtYXN0ZXJA
c2FucGFvbG9pbWkuY29tLg0KDQoNClRoaXMgZS1tYWlsIChhbmQgYW55IGF0dGFjaG1lbnQo
cykpIGlzIHN0cmljdGx5IGNvbmZpZGVudGlhbCBhbmQgZm9yIHVzZQ0Kb25seSBieSBpbnRl
bmRlZCByZWNpcGllbnQocykuICBBbnkgb3BpbmlvbnMgdGhlcmVpbiBleHByZXNzZWQgYXJl
IHRob3NlDQpvZiB0aGUgYXV0aG9yLiAgVGhlcmVmb3JlICBpdHMgY29udGVudCAgZG9lc24n
dCByZXByZXNlbnQgYW55IGNvbW1pdG1lbnQNCmJldHdlZW4gICBTYW5wYW9sbyBHcm91cCAg
YW5kICB0aGUgIHJlY2lwaWVudChzKSAgIGFuZCAgIG5vICBsaWFiaWxpdHkgIG9yDQpyZXNw
b25zaWJpbGl0eSAgaXMgIGFjY2VwdGVkICBieSAgU2FucGFvbG8gR3JvdXAgIGZvciAgdGhl
IGFib3ZlIG1lbnRpb25lZA0KY29udGVudC4NCg0KU2FucGFvbG8gSU1JIFMucC5BLiBpcyBh
IEJhbmsgYXV0aG9yaXNlZCBieSBCYW5jYSBkJ0l0YWxpYTsgU2FucGFvbG8gSU1JIA0KUy5w
LkEuLCBMb25kb24gQnJhbmNoLCBpcyByZWd1bGF0ZWQgYnkgdGhlIEZpbmFuY2lhbCBTZXJ2
aWNlcyAgQXV0aG9yaXR5DQpmb3IgdGhlIGNvbmR1Y3Qgb2YgaW52ZXN0bWVudCBidXNpbmVz
cyBpbiB0aGUgVUsuDQoNCklmICAgIHlvdSAgIGFyZSAgIG5vdCAgIGFuICAgaW50ZW5kZWQg
ICByZWNpcGllbnQocyksICAgIHBsZWFzZSAgIG5vdGlmeQ0KcG9zdG1hc3RlckBzYW5wYW9s
b2ltaS5jb20gcHJvbXB0bHkgYW5kIGRlc3Ryb3kgdGhpcyBtZXNzYWdlLg0K
#
On 11/10/2005 3:02 AM, roberto.ugoccioni at sanpaoloimi.com wrote:
Roberto:

Something went wrong with your posting -- it came out completely in 
binary.

I think I have found what was wrong with the editor windows.  There were 
two problems:

  - a variable was sometimes uninitialized.  I've fixed this in R-devel 
(though it won't be downloadable for a while).  I had hoped this was 
enough, but it wasn't.

  - the script editor window duplicates some menu items from the main 
window, but unfortunately in doing so it overwrites some pointers to 
those items.  When the script editor is closed, the pointers become 
invalid, and you may crash.

The second fix will require more extensive work, but I should get it 
done today.

Duncan Murdoch
#
On 11/10/2005 8:58 AM, Duncan Murdoch wrote:
Now committed to both R-patched and R-devel.  Will show up in the 
binaries on CRAN in due time.

Duncan Murdoch