Re: Interesting discovery from automatic tools (python code)

2022-08-11 Thread José Matos
On Thu, 2022-08-11 at 20:29 +0200, Pavel Sanda wrote: > > I did not use the exceptions to understand the failure reasons. > > The issue with the except is that it is a catch all option and so > > the > > exception could be caused by faulty assumptions from us in the > > initial > > code. > > My un

Re: Interesting discovery from automatic tools (python code)

2022-08-11 Thread Pavel Sanda
On Thu, Aug 11, 2022 at 05:55:47PM +0100, José Matos wrote: > On Thu, 2022-08-11 at 17:07 +0200, Pavel Sanda wrote: > > It seems that subprocess.check_call is raising exception instead of > > returning non-zero, > > so I gave it except block. > > What type of exception did you get? convert-im6.q1

Re: Interesting discovery from automatic tools (python code)

2022-08-11 Thread José Matos
On Thu, 2022-08-11 at 17:07 +0200, Pavel Sanda wrote: > It seems that subprocess.check_call is raising exception instead of > returning non-zero, > so I gave it except block. What type of exception did you get? I did not use the exceptions to understand the failure reasons. The issue with the exc

Re: Interesting discovery from automatic tools (python code)

2022-08-11 Thread Pavel Sanda
On Sun, Jul 31, 2022 at 11:41:02AM +0100, José Matos wrote: > On Sun, 2022-07-31 at 11:16 +0200, Pavel Sanda wrote: > > Sure, whatever you need :) > > The patch is in. > > > > Pavel > > First attempt. :-) Hi Jose, attached is my take on it. It seems that subprocess.check_call is raising except

Re: [RFC][PATCH] Improved synctex support

2022-08-11 Thread Pavel Sanda
On Thu, Aug 11, 2022 at 12:37:50PM +0200, Stephan Witt wrote: > >> 1) > >> AFAICS the synctex activation is possible for more than pfdlatex output > >> only. I???ve tried dvi, luatex and xetex and all of them work for me. So > >> I???ve changed the check in BufferParams::writeLaTeX to use > >> O

Re: [RFC][PATCH] Improved synctex support

2022-08-11 Thread Stephan Witt
Am 11.08.2022 um 09:37 schrieb Pavel Sanda : > Thanks for having a look at it. > On Mon, Aug 08, 2022 at 04:46:54PM +0200, Stephan Witt wrote: >> 1) >> AFAICS the synctex activation is possible for more than pfdlatex output >> only. I???ve tried dvi, luatex and xetex and all of them work for me.

Re: [RFC][PATCH] Improved synctex support

2022-08-11 Thread Pavel Sanda
On Mon, Aug 08, 2022 at 04:46:54PM +0200, Stephan Witt wrote: > 1) > AFAICS the synctex activation is possible for more than pfdlatex output only. > I???ve tried dvi, luatex and xetex and all of them work for me. So I???ve > changed the check in BufferParams::writeLaTeX to use OutputParams::isLaT