Re: Crash from feature of reloading file changed on disk

2017-10-08 Thread Richard Heck
On 10/08/2017 02:41 PM, Scott Kostyshak wrote: > On Sun, Oct 08, 2017 at 02:56:32PM +, Richard Heck wrote: >> On 10/07/2017 09:30 PM, Scott Kostyshak wrote: >>> On Sat, Oct 07, 2017 at 10:39:37PM +, Richard Heck wrote: On 10/07/2017 06:15 PM, Scott Kostyshak wrote: > On Sat, Oct

Re: Crash from feature of reloading file changed on disk

2017-10-08 Thread Scott Kostyshak
On Sun, Oct 08, 2017 at 02:56:32PM +, Richard Heck wrote: > On 10/07/2017 09:30 PM, Scott Kostyshak wrote: > > On Sat, Oct 07, 2017 at 10:39:37PM +, Richard Heck wrote: > >> On 10/07/2017 06:15 PM, Scott Kostyshak wrote: > >>> On Sat, Oct 07, 2017 at 02:16:39PM +, Richard Heck wrote: >

Re: Crash from feature of reloading file changed on disk

2017-10-08 Thread Richard Heck
On 10/07/2017 09:30 PM, Scott Kostyshak wrote: > On Sat, Oct 07, 2017 at 10:39:37PM +, Richard Heck wrote: >> On 10/07/2017 06:15 PM, Scott Kostyshak wrote: >>> On Sat, Oct 07, 2017 at 02:16:39PM +, Richard Heck wrote: >>> Probably more for master, as there is a simpler solution for

Re: Crash from feature of reloading file changed on disk

2017-10-07 Thread Scott Kostyshak
On Sat, Oct 07, 2017 at 10:39:37PM +, Richard Heck wrote: > On 10/07/2017 06:15 PM, Scott Kostyshak wrote: > > On Sat, Oct 07, 2017 at 02:16:39PM +, Richard Heck wrote: > > > >> Probably more for master, as there is a simpler solution for 2.3.x. But > >> it would be good > >> to test it. >

Re: Crash from feature of reloading file changed on disk

2017-10-07 Thread Richard Heck
On 10/07/2017 06:15 PM, Scott Kostyshak wrote: > On Sat, Oct 07, 2017 at 02:16:39PM +, Richard Heck wrote: > >> Probably more for master, as there is a simpler solution for 2.3.x. But >> it would be good >> to test it. > Do you have a patch for the simpler solution that I can test for 2.3.x?

Re: Crash from feature of reloading file changed on disk

2017-10-07 Thread Scott Kostyshak
On Sat, Oct 07, 2017 at 02:16:39PM +, Richard Heck wrote: > Probably more for master, as there is a simpler solution for 2.3.x. But > it would be good > to test it. Do you have a patch for the simpler solution that I can test for 2.3.x? Scott signature.asc Description: PGP signature

Re: Crash from feature of reloading file changed on disk

2017-10-07 Thread Richard Heck
On 10/07/2017 12:51 AM, Scott Kostyshak wrote: > On Tue, Oct 03, 2017 at 10:08:28PM +, Richard Heck wrote: >> On 10/03/2017 05:28 PM, Richard Heck wrote: >>> On 10/03/2017 03:54 PM, Jean-Marc Lasgouttes wrote: Le 03/10/17 à 21:43, Richard Heck a écrit : > The crucial thing here

Re: Crash from feature of reloading file changed on disk

2017-10-06 Thread Scott Kostyshak
On Tue, Oct 03, 2017 at 10:08:28PM +, Richard Heck wrote: > On 10/03/2017 05:28 PM, Richard Heck wrote: > > On 10/03/2017 03:54 PM, Jean-Marc Lasgouttes wrote: > >> Le 03/10/17 à 21:43, Richard Heck a écrit : > >> > >>> The crucial thing here seems to be that it's a "menu" inset. That > >>>

Re: Crash from feature of reloading file changed on disk

2017-10-03 Thread Richard Heck
On 10/03/2017 05:28 PM, Richard Heck wrote: > On 10/03/2017 03:54 PM, Jean-Marc Lasgouttes wrote: >> Le 03/10/17 à 21:43, Richard Heck a écrit : >> >>> The crucial thing here seems to be that it's a "menu" inset. That >>> triggers a menu search, which calls getStatus, which ends up accessing >>>

Re: Crash from feature of reloading file changed on disk

2017-10-03 Thread Richard Heck
On 10/03/2017 03:54 PM, Jean-Marc Lasgouttes wrote: > Le 03/10/17 à 21:43, Richard Heck a écrit : > >> The crucial thing here seems to be that it's a "menu" inset. That >> triggers a menu search, which calls getStatus, which ends up accessing >> the Cursor, which is now out of date. So we end up

Re: Crash from feature of reloading file changed on disk

2017-10-03 Thread Jean-Marc Lasgouttes
Le 03/10/17 à 21:43, Richard Heck a écrit : The crucial thing here seems to be that it's a "menu" inset. That triggers a menu search, which calls getStatus, which ends up accessing the Cursor, which is now out of date. So we end up here: A different solution would be to avoid calling

Re: Crash from feature of reloading file changed on disk

2017-10-03 Thread Richard Heck
On 10/03/2017 03:10 PM, Scott Kostyshak wrote: > Can anyone else reproduce? > > 1. Open Additional.lyx in LyX > 2. Go to a URL inset (e.g. search for "www.literateprogramming.com") > 3. Highlight the inside of the inset (e.g. ctrl-a). > 4. Change that URL externally (e.g. add a letter somewhere) >

Re: Crash from feature of reloading file changed on disk

2017-10-03 Thread Stephan Witt
Am 03.10.2017 um 21:10 schrieb Scott Kostyshak : > > Can anyone else reproduce? > > 1. Open Additional.lyx in LyX > 2. Go to a URL inset (e.g. search for "www.literateprogramming.com") > 3. Highlight the inside of the inset (e.g. ctrl-a). > 4. Change that URL externally (e.g.

Crash from feature of reloading file changed on disk

2017-10-03 Thread Scott Kostyshak
Can anyone else reproduce? 1. Open Additional.lyx in LyX 2. Go to a URL inset (e.g. search for "www.literateprogramming.com") 3. Highlight the inside of the inset (e.g. ctrl-a). 4. Change that URL externally (e.g. add a letter somewhere) 5. Click "Reload" in LyX's blue banner notification. 6.