Re: r26369 - /lyx-devel/trunk/src/Text2.cpp

2008-09-12 Thread Abdelrazak Younes
Jean-Marc Lasgouttes wrote: Abdelrazak Younes<[EMAIL PROTECTED]> writes: Agreed. Could the paste code be the culprit? Found it: breakParagraphConservative(): don't forget to set the inset owner. This is the real fix for bughttp://bugzilla.lyx.org/show_bug.cgi?id=5099

Re: r26369 - /lyx-devel/trunk/src/Text2.cpp

2008-09-12 Thread Jean-Marc Lasgouttes
Abdelrazak Younes <[EMAIL PROTECTED]> writes: >> Agreed. Could the paste code be the culprit? >> > > Found it: > breakParagraphConservative(): don't forget to set the inset owner. This is > the real fix for bughttp://bugzilla.lyx.org/show_bug.cgi?id=5099 Great. It might even be my fault...

Re: r26369 - /lyx-devel/trunk/src/Text2.cpp

2008-09-12 Thread Abdelrazak Younes
Jean-Marc Lasgouttes wrote: Abdelrazak Younes<[EMAIL PROTECTED]> writes: I think there's something fishy in there. I don't like at all this inset owner concept but I guess we should never be in a case where this is not initialized, should we? Agreed. Could the paste code be the culp

Re: [Cvslog] r26369 - /lyx-devel/trunk/src/Text2.cpp

2008-09-12 Thread Jean-Marc . Lasgouttes
Abdelrazak Younes <[EMAIL PROTECTED]> writes: > He he, I guess we hit the reply button at the same time :-) Yes, and I was in the process of pointing to labelFont in bugzilla %-| JMarc

Re: r26369 - /lyx-devel/trunk/src/Text2.cpp

2008-09-12 Thread Abdelrazak Younes
Jean-Marc Lasgouttes wrote: Abdelrazak Younes<[EMAIL PROTECTED]> writes: I think there's something fishy in there. I don't like at all this inset owner concept but I guess we should never be in a case where this is not initialized, should we? Agreed. Could the paste code be the culp

Re: [Cvslog] r26369 - /lyx-devel/trunk/src/Text2.cpp

2008-09-12 Thread Abdelrazak Younes
Jean-Marc Lasgouttes wrote: [EMAIL PROTECTED] writes: Author: younes Date: Fri Sep 12 14:55:35 2008 New Revision: 26369 URL: http://www.lyx.org/trac/changeset/26369 Log: Test Paragraph::inInset() before using it. Fixes bug http://bugzilla.lyx.org/show_bug.cgi?id=5099 But how come

Re: r26369 - /lyx-devel/trunk/src/Text2.cpp

2008-09-12 Thread Jean-Marc Lasgouttes
Abdelrazak Younes <[EMAIL PROTECTED]> writes: > I think there's something fishy in there. I don't like at all this > inset owner concept but I guess we should never be in a case where > this is not initialized, should we? Agreed. Could the paste code be the culprit? JMarc

Re: r26369 - /lyx-devel/trunk/src/Text2.cpp

2008-09-12 Thread Abdelrazak Younes
[EMAIL PROTECTED] wrote: Author: younes Date: Fri Sep 12 14:55:35 2008 New Revision: 26369 URL: http://www.lyx.org/trac/changeset/26369 Log: Test Paragraph::inInset() before using it. Fixes bug http://bugzilla.lyx.org/show_bug.cgi?id=5099 I think there's something fishy in there. I don't like

Re: [Cvslog] r26369 - /lyx-devel/trunk/src/Text2.cpp

2008-09-12 Thread Jean-Marc Lasgouttes
[EMAIL PROTECTED] writes: > Author: younes > Date: Fri Sep 12 14:55:35 2008 > New Revision: 26369 > > URL: http://www.lyx.org/trac/changeset/26369 > Log: > Test Paragraph::inInset() before using it. Fixes bug > http://bugzilla.lyx.org/show_bug.cgi?id=5099 But how come inInset can be null? JMarc