-
Notifications
You must be signed in to change notification settings - Fork 390
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Missing character" error message of LuaTeX is not detected #2943
Comments
I think it's fixed now; please update and test. Notice, I was not able to parse the line number of the error. |
Yes that works 😄 but there's a little quirk: if it's only set to be a warning instead of an error, the parser will gobble the next line. Quickfix window:
Source: %! TEX program = lualatex
\documentclass{article}
\begin{document}
\symbol{1}
\end{document} Log file partial:
|
Huh, so So, I propose we "demote" the "Missing character" to a warning, and then I'll also fix the gobble-problem. Does that sound fine by you? |
I didn't know vimtex handles a warning and an error differently. I don't see any problem either way as long as it's visible to the user. Note that the default mode of vimtex is in |
Ah, well, that's not precisely what I meant. What we do, more precisely is to parse the latex log file line for line with the errorformat spec, see e.g.: vimtex/autoload/vimtex/qf/latexlog.vim Lines 39 to 66 in 8b668fb
This is definitely an esoteric DSL that takes some getting used to! So, in this sense, we handle warnings and errors the same. But we do match them slightly differently. The problem with your last example is that this line in the log: vimtex/test/example-quickfix/main.log Line 55 in 8b668fb
was matched as a multi-line continuation in line 40 of
Yes, but I don't think that is relevant here. |
Results in "Compilation failed!" without any error message.
Must compile with LuaLaTeX. For PDFLaTeX the character at position 1 does exist (you can replace 1 with 255, but then for PDFLaTeX then the error is detected)
The text was updated successfully, but these errors were encountered: