Re: Coverity Loop Issue

2017-01-06 Thread Richard Heck
On 01/06/2017 05:06 PM, Jean-Marc Lasgouttes wrote: > Le 18/12/2016 à 04:21, Richard Heck a écrit : >> JMarc, I think the attached patches take care of this. I broke it up so >> it was more obvious what was happening. The first patch takes care of >> the for-loop issue, and the others re-organize a

Testing of the 2.2.x Branch (esp Slowness)

2017-01-06 Thread Richard Heck
We (specifically, Jean-Marc Lasgouttes) have recently made some changes to the text rendering algorithms in LyX. We would appreciate it if those who are able could compile and use the 2.2.x branch of the git repository, so that these changes can receive sufficient testing before we schedule the rel

Re: Fwd: X-Apps

2017-01-06 Thread Paul A. Rubin
On 01/06/2017 12:52 PM, Pavel Sanda wrote: Paul A. Rubin wrote: I've opened a ticket (10534) and attached the diff file for an attempt by me to add them to the script. Would the patch work if you put all the 'x..' entries at the end of the list? I don't see reason why we should suddenly change

Re: Coverity Loop Issue

2017-01-06 Thread Jean-Marc Lasgouttes
Le 06/01/2017 à 23:06, Jean-Marc Lasgouttes a écrit : Le 18/12/2016 à 04:21, Richard Heck a écrit : JMarc, I think the attached patches take care of this. I broke it up so it was more obvious what was happening. The first patch takes care of the for-loop issue, and the others re-organize a bit t

Re: Coverity Loop Issue

2017-01-06 Thread Jean-Marc Lasgouttes
Le 18/12/2016 à 04:21, Richard Heck a écrit : JMarc, I think the attached patches take care of this. I broke it up so it was more obvious what was happening. The first patch takes care of the for-loop issue, and the others re-organize a bit to make the logic clearer. My question is: did you wor

Re: LyX 2.2 slowness

2017-01-06 Thread Jean-Marc Lasgouttes
Le 06/01/2017 à 20:16, Richard Heck a écrit : OK, go ahead and commit then. I will send a note to lyx-devel and lyx-users announcing that this has happened and encouraging people, as they are able, to use 2.2.x. It is in. JMarc

Re: LyX 2.2 slowness

2017-01-06 Thread Richard Heck
On 01/06/2017 10:17 AM, Jean-Marc Lasgouttes wrote: > Le 06/01/2017 à 16:13, Richard Heck a écrit : >> I guess I'd be inclined to go >> ahead and commit it, then. Maybe we can just encourage as many people as >> possible to use the 2.2.x branch (if they're not already using master), >> so it gets t

Re: Fwd: X-Apps

2017-01-06 Thread Pavel Sanda
Paul A. Rubin wrote: > I've opened a ticket (10534) and attached the diff file for an attempt by > me to add them to the script. Would the patch work if you put all the 'x..' entries at the end of the list? I don't see reason why we should suddenly change the order for other people. If you post

Re: Fwd: X-Apps

2017-01-06 Thread Paul A. Rubin
On 12/31/2016 04:26 AM, Guenter Milde wrote: On 2016-12-27, Paul A. Rubin wrote: With version 18 (Sarah), Linux Mint switched to a new set of tools called "X-Apps ", at least for Cinnamon users. In particular, the new default PDF viewer

Re: LyX 2.2 slowness

2017-01-06 Thread Richard Heck
On 01/06/2017 04:56 AM, Jean-Marc Lasgouttes wrote: > Le 31/12/2016 à 15:26, Jean-Marc Lasgouttes a écrit : >> Le 31/12/2016 à 15:24, Jean-Marc Lasgouttes a écrit : >>> It is this particular patch. I amended it since the initial July 5 >>> creattion, but apparently git does not update date by defau

Re: LyX 2.2 slowness

2017-01-06 Thread Jean-Marc Lasgouttes
Le 06/01/2017 à 16:13, Richard Heck a écrit : I guess I'd be inclined to go ahead and commit it, then. Maybe we can just encourage as many people as possible to use the 2.2.x branch (if they're not already using master), so it gets tested. I would agree with that. One question: Do you want to

Re: LyX 2.2 slowness

2017-01-06 Thread Jean-Marc Lasgouttes
Le 31/12/2016 à 15:26, Jean-Marc Lasgouttes a écrit : Le 31/12/2016 à 15:24, Jean-Marc Lasgouttes a écrit : It is this particular patch. I amended it since the initial July 5 creattion, but apparently git does not update date by default. To be more precise, this is the commit below, pushed on