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: 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: 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