On Sat, Nov 03, 2007 at 01:12:24AM +0200, Martin Vermeer wrote: > On Sat, Nov 03, 2007 at 12:00:42AM +0100, Abdelrazak Younes wrote: > > ... > > > >> OK, I've done some cleanup an the crashes are gone but the inset > > >> layouts > > >> are broken! > > >> > > >> Martin, Richard, do you have an idea here? I don't know if we shall we > > >> fix Inset::getLayout() or if we shall assume that each inset hard-code > > >> its own layout. > > >> > > >> Help please, > > >> Abdel. > > > Abdel, > > > this is precisely the way _not_ to do it. The calls to setLayout() in the > > > various insets are designed (by Jean-Marc during the Bromarv meeting) to > > > get precisely the right insetlayout based on the inset's name(). > > > > Year I noticed that... calling virtual methods in ctors is dangerous and > > should be avoided. > > OK...
Well, it has a well-defined semantics (the incarnation of the currently constructed part is called). It's not 'dangerous' but usually not too helpful. Andre'