search for: newx11devicedriv

Dear R Development Team, I compile R-1.9.1 on AIX 5.2 under 2.9-aix51-020209, and xlf 7.1. In order to let R compile successfully under gcc 2.9, I have to change one C statement of file(RHOME//src/modules/X11/devX11.c) line 1768 from "retrun FALSE" to "retrun NULL", following is C code snapshot: newX11Desc * Rf_allocNewX11DeviceDesc(double ps) { newX11Desc *xd;

...-----Original Message----- > From: Paul Murrell [mailto:p.murrell at auckland.ac.nz] > Sent: Sunday, August 01, 2004 6:35 PM > To: Liao, Kexiao > Cc: r-devel at stat.math.ethz.ch > Subject: Re: [Rd] plot(x,y) core dump > > Hi > > Rf_allocNewX11DeviceDesc is called by newX11DeviceDriver and the latter > needs to check the value returned. I think Brian Ripley has made a fix > for this problem in the development version of R. The first few lines > of newX11DeviceDriver now look like ... > > newX11Desc *xd; > char *fn; > > xd = Rf_alloc...

...t; > >This was the story, R-core November 29, 2001. Part of thread "X11 still segfaults". > >------------>> >..... >Gah. I've been too tired today. Why did that take me so long? > >The culprit seems to be > >R_ProcessEvents((void*) NULL) > >in newX11DeviceDriver > >This gets called *before* this stuff at the end of Rf_addX11Device > > dd = GEcreateDevDesc(dev); > addDevice((DevDesc*) dd); > initDisplayList((DevDesc*) dd); > >and it is that "dd" that gets called by Rf_playDisplayList. Removing >the offending line stops...

...ll segfaults". >>> >>> ------------>> >>> ..... >>> Gah. I've been too tired today. Why did that take me so long? >>> >>> The culprit seems to be >>> >>> R_ProcessEvents((void*) NULL) >>> >>> in newX11DeviceDriver >>> >>> This gets called *before* this stuff at the end of Rf_addX11Device >>> >>> dd = GEcreateDevDesc(dev); >>> addDevice((DevDesc*) dd); >>> initDisplayList((DevDesc*) dd); >>> >>> and it is that "dd" that get...

I don't recall exactly what I did 18 years ago eiher and I likely don't have the time to dig into the archives and reconstruct. I can imagine that the issue had to do with the protocol around creating and mapping windows. Presumably the segfault comes from looking for events on a window that hasn't been created yet, or has already been destroyed, leading to a NULL reference somewhere.

OK, so I did the archaeology anyway.... This was the story, R-core November 29, 2001. Part of thread "X11 still segfaults". ------------>> ..... Gah. I've been too tired today. Why did that take me so long? The culprit seems to be R_ProcessEvents((void*) NULL) in newX11DeviceDriver This gets called *before* this stuff at the end of Rf_addX11Device dd = GEcreateDevDesc(dev); addDevice((DevDesc*) dd); initDisplayList((DevDesc*) dd); and it is that "dd" that gets called by Rf_playDisplayList. Removing the offending line stops the segfaulting, seemingly with no...

...t; >>>>> ..... >>>>> Gah. I've been too tired today. Why did that take me so long? >>>>> >>>>> The culprit seems to be >>>>> >>>>> R_ProcessEvents((void*) NULL) >>>>> >>>>> in newX11DeviceDriver >>>>> >>>>> This gets called *before* this stuff at the end of Rf_addX11Device >>>>> >>>>> dd = GEcreateDevDesc(dev); >>>>> addDevice((DevDesc*) dd); >>>>> initDisplayList((DevDesc*) dd); >>>>...

..., 2001. Part of thread "X11 still segfaults". >> >> ------------>> >> ..... >> Gah. I've been too tired today. Why did that take me so long? >> >> The culprit seems to be >> >> R_ProcessEvents((void*) NULL) >> >> in newX11DeviceDriver >> >> This gets called *before* this stuff at the end of Rf_addX11Device >> >> dd = GEcreateDevDesc(dev); >> addDevice((DevDesc*) dd); >> initDisplayList((DevDesc*) dd); >> >> and it is that "dd" that gets called by Rf_playDisplayLis...

...t;>> ------------>> >>>> ..... >>>> Gah. I've been too tired today. Why did that take me so long? >>>> >>>> The culprit seems to be >>>> >>>> R_ProcessEvents((void*) NULL) >>>> >>>> in newX11DeviceDriver >>>> >>>> This gets called *before* this stuff at the end of Rf_addX11Device >>>> >>>> dd = GEcreateDevDesc(dev); >>>> addDevice((DevDesc*) dd); >>>> initDisplayList((DevDesc*) dd); >>>> >>>> and...