Re: [Gnustep-cvs] gnustep/core/gui ChangeLog Source/NSPopUpButton...

[Enrico Sersale]

On 2005-05-30 20:09:27 +0300 Fred Kiefer <address@hidden> wrote:

> Hi Enrico,
>
> could you please describe in detail how to reproduce this bug? I tried before 
> and after applying the latest patch be Matt and faild to find the correct 
> order of operations to reproduce it.
> And are you sure this patch was the reason for the traceback not the other 
> one? 

Yes. Using the latest CVS and substituting in it *only* NSPopUpButtonCell.m 
fixes the bug.
To reproduce it:
with GWorkspace: close a window that contains a popup (you must put it in "spatial" mode 
"View->Viewer behaviour->Spatial").
with GNUMail: do something that closes a window, for example, try to send a 
message.

> This looks like memory problems and I would rather expect the  copying 
> to trigger them.
>
> Fred
>
> Enrico Sersale wrote:
> > On 2005-05-27 16:41:46 +0300 Fred Kiefer <address@hidden> wrote:
> >
> > > CVSROOT:    /cvsroot/gnustep
> > > Module name:    gnustep
> > > Branch:    Changes by:    Fred Kiefer <address@hidden> 
> > > 05/05/27 13:41:46
> > >
> > > Modified files:
> > >       core/gui       : ChangeLog     core/gui/Source: NSPopUpButtonCell.m
> > > Log message:
> > >       Make sure popup button cells wont get notifications after being 
> > > dealloceated.
> > >
> > > CVSWeb URLs:
> > > http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/ChangeLog.diff?tr1=1.2529&tr2=1.2530&r1=text&r2=text
> > > http://savannah.gnu.org/cgi-bin/viewcvs/gnustep/gnustep/core/gui/Source/NSPopUpButtonCell.m.diff?tr1=1.67&tr2=1.68&r1=text&r2=text
> >
> >
> > This change causes strange segfaults in GNUMail (sending a message, closing 
> > a window)
> >
> > [snip]