mirror of
https://github.com/dense-analysis/ale
synced 2024-12-25 07:32:24 +00:00
Fix 368 - Lacheck reports errors from input{} files.
This PR adds additional check to lacheck linter to exclude any warnings related to sourced files via latex \input{} command. Closes: #368
This commit is contained in:
parent
9226e13b31
commit
a6a8c90126
@ -8,20 +8,26 @@ function! ale_linters#tex#lacheck#Handle(buffer, lines) abort
|
|||||||
"
|
"
|
||||||
" "book.tex", line 37: possible unwanted space at "{"
|
" "book.tex", line 37: possible unwanted space at "{"
|
||||||
" "book.tex", line 38: missing `\ ' after "etc."
|
" "book.tex", line 38: missing `\ ' after "etc."
|
||||||
let l:pattern = '^".\+", line \(\d\+\): \(.\+\)$'
|
let l:pattern = '^"\(.\+\)", line \(\d\+\): \(.\+\)$'
|
||||||
let l:output = []
|
let l:output = []
|
||||||
|
|
||||||
for l:match in ale#util#GetMatches(a:lines, l:pattern)
|
for l:match in ale#util#GetMatches(a:lines, l:pattern)
|
||||||
" lacheck follows `\input{}` commands. If the cwd is not the same as the
|
" lacheck follows `\input{}` commands. If the cwd is not the same as the
|
||||||
" file in the buffer then it will fail to find the inputed items. We do not
|
" file in the buffer then it will fail to find the inputed items. We do not
|
||||||
" want warnings from those items anyway
|
" want warnings from those items anyway
|
||||||
if !empty(matchstr(l:match[2], '^Could not open ".\+"$'))
|
if !empty(matchstr(l:match[3], '^Could not open ".\+"$'))
|
||||||
|
continue
|
||||||
|
endif
|
||||||
|
|
||||||
|
" lacheck follows `\input{}` commands. We are only interested in
|
||||||
|
" reporting errors for the current buffer only.
|
||||||
|
if empty(matchstr(fnamemodify(l:match[1], ':t'), bufname(a:buffer)))
|
||||||
continue
|
continue
|
||||||
endif
|
endif
|
||||||
|
|
||||||
call add(l:output, {
|
call add(l:output, {
|
||||||
\ 'lnum': l:match[1] + 0,
|
\ 'lnum': l:match[2] + 0,
|
||||||
\ 'text': l:match[2],
|
\ 'text': l:match[3],
|
||||||
\ 'type': 'W',
|
\ 'type': 'W',
|
||||||
\})
|
\})
|
||||||
endfor
|
endfor
|
||||||
|
Loading…
Reference in New Issue
Block a user