Regression?

Chani chanika at gmail.com
Thu Feb 7 02:51:04 CET 2008


On February 7, 2008 09:38:07 Sebastian Sauer wrote:
> On Thursday 07 February 2008, Chani wrote:
> > > +    //lay = 0;
> >
> > *that* would have caused a crash if anyone was using the mediacenter
> > formfactor yet. :) silly layouts.
>
> y, double free's are wired cause they may crash but may also just work
> well. Well, maybe there is a switch for gcc to do it like msvc in
> debug-mode and just always crash/assert, no idea there.

why? because you didn't set lay to 0, but you did call setLayout(0) which 
deletes it, and then lower down in the code there's a block that does stuff 
with lay.

anyways, I just realised I forgot to search for instances of layout() - which 
turns up a few more evil layout deletions in places like setFailedToLaunch...

-- 
This message brought to you by evyl bananas, and the number 3.
www.chani3.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080207/2d39693b/attachment.pgp 


More information about the Panel-devel mailing list