Re: SIGSEGV on master regarding math completion

2022-03-08 Thread Scott Kostyshak
On Tue, Mar 08, 2022 at 04:20:34PM +0100, Jean-Marc Lasgouttes wrote: > Le 06/03/2022 à 19:55, Scott Kostyshak a écrit : > > It is not minimal, but I see the message when I open ch01.lyx, after > > lots of messages about modules not available, from the below ticket: > > > >https://www.lyx.org/

Re: SIGSEGV on master regarding math completion

2022-03-08 Thread Jean-Marc Lasgouttes
Le 06/03/2022 à 19:55, Scott Kostyshak a écrit : It is not minimal, but I see the message when I open ch01.lyx, after lots of messages about modules not available, from the below ticket: https://www.lyx.org/trac/ticket/12486 Indeed, thanks. Should be fixed now. JMarc -- lyx-devel mailing l

Re: SIGSEGV on master regarding math completion

2022-03-06 Thread Scott Kostyshak
On Tue, Jan 25, 2022 at 11:49:58AM -0500, Scott Kostyshak wrote: > On Tue, Jan 25, 2022 at 05:28:44PM +0100, Jean-Marc Lasgouttes wrote: > > Le 25/01/2022 à 17:23, Scott Kostyshak a écrit : > > > I have seen that terminal message on master in other contexts (I think > > > it had something to do wit

Re: SIGSEGV on master regarding math completion

2022-01-28 Thread Scott Kostyshak
On Fri, Jan 28, 2022 at 06:09:25PM +0100, Jean-Marc Lasgouttes wrote: > Le 28/01/2022 à 17:49, Kornel Benko a écrit : > > > This seems to work and makes sense. Could someone do a quick check > > > before I commit ? I will run it under valgrind later, when I am in front > > > of an old enough ubuntu

Re: SIGSEGV on master regarding math completion

2022-01-28 Thread Jean-Marc Lasgouttes
Le 28/01/2022 à 17:49, Kornel Benko a écrit : This seems to work and makes sense. Could someone do a quick check before I commit ? I will run it under valgrind later, when I am in front of an old enough ubuntu box. Works well here. Compiled with -fsanitize, no problems. Thanks Kornel. I pus

Re: SIGSEGV on master regarding math completion

2022-01-28 Thread Kornel Benko
Am Fri, 28 Jan 2022 17:17:40 +0100 schrieb Jean-Marc Lasgouttes : > Le 27/01/2022 à 17:26, Jean-Marc Lasgouttes a écrit : > > So finally, the MathRow object points to an inset that has been deleted > > at the time of the completion. There should be code somewhere that > > should regenerate the m

Re: SIGSEGV on master regarding math completion

2022-01-28 Thread Jean-Marc Lasgouttes
Le 27/01/2022 à 17:26, Jean-Marc Lasgouttes a écrit : So finally, the MathRow object points to an inset that has been deleted at the time of the completion. There should be code somewhere that should regenerate the math row. I'll have a look. This seems to work and makes sense. Could someone d

Re: SIGSEGV on master regarding math completion

2022-01-28 Thread Jean-Marc Lasgouttes
Le 27/01/2022 à 23:39, Scott Kostyshak a écrit : So finally, the MathRow object points to an inset that has been deleted at the time of the completion. There should be code somewhere that should regenerate the math row. I'll have a look. Thanks for walking this through! The only part you skippe

Re: SIGSEGV on master regarding math completion

2022-01-27 Thread Scott Kostyshak
On Thu, Jan 27, 2022 at 05:26:33PM +0100, Jean-Marc Lasgouttes wrote: > Le 10/01/2022 à 03:49, Scott Kostyshak a écrit : > > To reproduce: > > > > 1. Start a new document. > > 2. Ctrl + m to start math inset. > > 3. Type \phantomx. Note that \phantomx is not a command so don't > > expect it to

Re: SIGSEGV on master regarding math completion

2022-01-27 Thread Jean-Marc Lasgouttes
Le 10/01/2022 à 03:49, Scott Kostyshak a écrit : To reproduce: 1. Start a new document. 2. Ctrl + m to start math inset. 3. Type \phantomx. Note that \phantomx is not a command so don't expect it to be recognized. 4. Press , , to delete the "x" and the "m". 5. Type "m" to finish "phantom".

Re: SIGSEGV on master regarding math completion

2022-01-25 Thread Scott Kostyshak
On Tue, Jan 25, 2022 at 05:28:44PM +0100, Jean-Marc Lasgouttes wrote: > Le 25/01/2022 à 17:23, Scott Kostyshak a écrit : > > I have seen that terminal message on master in other contexts (I think > > it had something to do with undefined layout insets). I guessed that it > > was not a problem since

Re: SIGSEGV on master regarding math completion

2022-01-25 Thread Jean-Marc Lasgouttes
Le 25/01/2022 à 17:23, Scott Kostyshak a écrit : I have seen that terminal message on master in other contexts (I think it had something to do with undefined layout insets). I guessed that it was not a problem since it did not assert. Should I report an MWE whenever I see that message? Yes plea

Re: SIGSEGV on master regarding math completion

2022-01-25 Thread Scott Kostyshak
On Tue, Jan 25, 2022 at 05:12:06PM +0100, Jean-Marc Lasgouttes wrote: > Le 25/01/2022 à 16:07, Scott Kostyshak a écrit : > > On testing a different commit I received the following terminal message: > > > >Undo.cpp (319): There is no group open (creating one) > > If you can reproduce this, I a

Re: SIGSEGV on master regarding math completion

2022-01-25 Thread Jean-Marc Lasgouttes
Le 25/01/2022 à 16:07, Scott Kostyshak a écrit : On testing a different commit I received the following terminal message: Undo.cpp (319): There is no group open (creating one) If you can reproduce this, I am interested. JMarc, the backtrace suggests that painting code is involved. If a bi

Re: SIGSEGV on master regarding math completion

2022-01-25 Thread Scott Kostyshak
On Mon, Jan 24, 2022 at 10:56:50AM +0100, Pavel Sanda wrote: > On Thu, Jan 20, 2022 at 09:40:21PM -0500, Scott Kostyshak wrote: > > > To reproduce: > > > > > > 1. Start a new document. > > > 2. Ctrl + m to start math inset. > > > 3. Type \phantomx. Note that \phantomx is not a command so don't > >

Re: SIGSEGV on master regarding math completion

2022-01-24 Thread Pavel Sanda
On Thu, Jan 20, 2022 at 09:40:21PM -0500, Scott Kostyshak wrote: > > To reproduce: > > > > 1. Start a new document. > > 2. Ctrl + m to start math inset. > > 3. Type \phantomx. Note that \phantomx is not a command so don't > >expect it to be recognized. > > 4. Press , , to delete the "x" and t

Re: SIGSEGV on master regarding math completion

2022-01-20 Thread Scott Kostyshak
On Sun, Jan 09, 2022 at 09:49:26PM -0500, Scott Kostyshak wrote: > To reproduce: > > 1. Start a new document. > 2. Ctrl + m to start math inset. > 3. Type \phantomx. Note that \phantomx is not a command so don't >expect it to be recognized. > 4. Press , , to delete the "x" and the "m". > 5. T

SIGSEGV on master regarding math completion

2022-01-09 Thread Scott Kostyshak
To reproduce: 1. Start a new document. 2. Ctrl + m to start math inset. 3. Type \phantomx. Note that \phantomx is not a command so don't expect it to be recognized. 4. Press , , to delete the "x" and the "m". 5. Type "m" to finish "phantom". 6. Wait for the completion pop-down to appear (this