Andre Poenitz wrote:
> This always returns 'true', so we can have 'void' instead and remove
> some never executed code...
Hold on! These lfuns can be called from outside can't they? So it's
possible for a user to try and insert an inset in a layout that
doesn't allow it. Isn't this the reason for the
if (!insertInset(inset))
delete inset;
???
Maybe I've got this wrong, but I woudl have thought that the right
thing to do was make insertInset more intelligent (interrogate
whether such an inset were allowed in such a layout) rather than go
your way.
--
Angus
