Re: Bug in CS 26372

2008-09-13 Thread Andre Poenitz
On Sat, Sep 13, 2008 at 05:21:27PM +0200, Abdelrazak Younes wrote: > Vincent van Ravesteijn - TNW wrote: >> >> >>> Vincent van Ravesteijn - TNW wrote: >>> In the following piece of code, the tmpInset was deleted if insetAllowed was false. After the change, tmpInset is deleted wh

Re: Bug in CS 26372

2008-09-13 Thread Abdelrazak Younes
Vincent van Ravesteijn - TNW wrote: Vincent van Ravesteijn - TNW wrote: In the following piece of code, the tmpInset was deleted if insetAllowed was false. After the change, tmpInset is deleted when insetAllowed is true. This leads to disappearing Insets and crashes. I'd suggest add

RE: Bug in CS 26372

2008-09-13 Thread Vincent van Ravesteijn - TNW
>Vincent van Ravesteijn - TNW wrote: >> In the following piece of code, the tmpInset was deleted if >> insetAllowed was false. After the change, tmpInset is deleted when >> insetAllowed is true. >> >> This leads to disappearing Insets and crashes. I'd suggest adding a !. >> > >Oups, sorry.

Re: Bug in CS 26372

2008-09-13 Thread Abdelrazak Younes
Vincent van Ravesteijn - TNW wrote: In the following piece of code, the tmpInset was deleted if insetAllowed was false. After the change, tmpInset is deleted when insetAllowed is true. - if (!toPar.insetAllowed(tmpInset->lyxCode())) { + Inset * inset = toPar.inInset(); + if (inset&& inset->