2014-10-21 05:19:14 UTC
successfully and shows the PDF. But if you look at the terminal output
you will see that pdflatex exited with error. The reason LyX doesn't
pick this up is because (1) it does not check the exit code (which is
a separate bug in my opinion) and (2) it does not detect an error when
scanning the log file. This patch addresses (2). I attach the log file
in case you are interested but do not want to install the LaTeX files
needed to test for yourself.
The problem was that we only register an error if there is 10 or fewer
lines in-between the "!" line and the line where the line number where
the error occurred (the "l." line). This patch increases that number
to 15. Why 15? Because we used 10 for 14 years and there weren't many
problems, at least not that anyone noticed. 15 is just enough to
detect the error in this particular case.
The number 10 was introduced at a2c6689c to address the following
+ * src/LaTeX.C (scanLogFile): errors where the line number was not
+ given just after the '!'-line were ignored (from Dekel Tsur).
Am I correct that the only drawback is in theory performance?
I would really like to fix (1) above, so I'm OK if there are
objections to this patch, as long as we agree that (1) should be