Regression?

[Cody Tracy]

On Wednesday 06 February 2008 16:06:23 Chani wrote:
> On February 7, 2008 06:56:20 Sebastian Sauer wrote:
> > On Wednesday 06 February 2008, you wrote:
> > > > which does in the Layout::Layout(LayoutItem *parent) ctor call
> > > > parent->setLayout(this) results somehow in an infinite loop calling
> > > > LayoutItem::setLayout(Layout* layout)
> > >
> > > does this patch fix it?
> >
> > unfortunately not :-/ Anyway, I'll try to investigate it tomorrow a bit
> > more in detail.
>
> the layout's destructor calls LayoutItem::setLayout(0) on the parent which
> then tries to delete the layout that's in the middle of being deleted...
> maybe we need a deleteLater() ? maybe the destructor shouldn't be calling
> that?

Does the parent delete its layout?  If anything I think it should be calling 
deleteLater().