diff --git a/README.md b/README.md index d9f133c..bf62f31 100644 --- a/README.md +++ b/README.md @@ -65,6 +65,7 @@ name. That seems to be the fairest way to arrange this table. | HTML | [HTMLHint](http://htmlhint.com/), [tidy](http://www.html-tidy.org/) | | JavaScript | [eslint](http://eslint.org/), [jscs](http://jscs.info/), [jshint](http://jshint.com/), [FlowType](https://flowtype.org/) | | JSON | [jsonlint](http://zaa.ch/jsonlint/) | +| LaTeX | [chktex](http://www.nongnu.org/chktex/), [lacheck](https://www.ctan.org/pkg/lacheck) | | Lua | [luacheck](https://github.com/mpeterv/luacheck) | | Markdown | [mdl](https://github.com/mivok/markdownlint), [proselint](http://proselint.com/)| | MATLAB | [mlint](https://www.mathworks.com/help/matlab/ref/mlint.html) | diff --git a/ale_linters/tex/chktex.vim b/ale_linters/tex/chktex.vim new file mode 100644 index 0000000..2d53c66 --- /dev/null +++ b/ale_linters/tex/chktex.vim @@ -0,0 +1,63 @@ +" Author: Andrew Balmos - +" Description: chktex for LaTeX files + +let g:ale_tex_chktex_executable = +\ get(g:, 'ale_tex_chktex_executable', 'chktex') + +let g:ale_tex_chktex_options = +\ get(g:, 'ale_tex_chktex_options', '-I') + +function! ale_linters#tex#chktex#GetCommand(buffer) abort + " Check for optional .chktexrc + let l:chktex_config = ale#util#FindNearestFile( + \ a:buffer, + \ '.chktexrc') + + let l:command = g:ale_tex_chktex_executable + " Avoid bug when used without -p (last warning has gibberish for a filename) + let l:command .= ' -v0 -p stdin -q' + + if !empty(l:chktex_config) + let l:command .= ' -l ' . fnameescape(l:chktex_config) + endif + + let l:command .= ' ' . g:ale_tex_chktex_options + + return l:command +endfunction + +function! ale_linters#tex#chktex#Handle(buffer, lines) abort + " Mattes lines like: + " + " stdin:499:2:24:Delete this space to maintain correct pagereferences. + " stdin:507:81:3:You should enclose the previous parenthesis with `{}'. + let l:pattern = '^stdin:\(\d\+\):\(\d\+\):\(\d\+\):\(.\+\)$' + let l:output = [] + + for l:line in a:lines + let l:match = matchlist(l:line, l:pattern) + + if len(l:match) == 0 + continue + endif + + call add(l:output, { + \ 'bufnr': a:buffer, + \ 'lnum': l:match[1] + 0, + \ 'vcol': 0, + \ 'col': l:match[2] + 0, + \ 'text': l:match[4] . ' (' . (l:match[3]+0) . ')', + \ 'type': 'W', + \ 'nr': -1 + \}) + endfor + + return l:output +endfunction + +call ale#linter#Define('tex', { +\ 'name': 'chktex', +\ 'executable': 'chktex', +\ 'command_callback': 'ale_linters#tex#chktex#GetCommand', +\ 'callback': 'ale_linters#tex#chktex#Handle' +\}) diff --git a/ale_linters/tex/lacheck.vim b/ale_linters/tex/lacheck.vim new file mode 100644 index 0000000..ffa1daa --- /dev/null +++ b/ale_linters/tex/lacheck.vim @@ -0,0 +1,50 @@ +" Author: Andrew Balmos - +" Description: lacheck for LaTeX files + +let g:ale_tex_lacheck_executable = +\ get(g:, 'ale_tex_lacheck_executable', 'lacheck') + +function! ale_linters#tex#lacheck#Handle(buffer, lines) abort + " Mattes lines like: + " + " "book.tex", line 37: possible unwanted space at "{" + " "book.tex", line 38: missing `\ ' after "etc." + + let l:pattern = '^".\+", line \(\d\+\): \(.\+\)$' + let l:output = [] + + for l:line in a:lines + let l:match = matchlist(l:line, l:pattern) + + if len(l:match) == 0 + continue + endif + + " 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 + " want warnings from those items anyway + if !empty(matchstr(l:match[2], '^Could not open ".\+"$')) + continue + endif + + call add(l:output, { + \ 'bufnr': a:buffer, + \ 'lnum': l:match[1] + 0, + \ 'vcol': 0, + \ 'col': 0, + \ 'text': l:match[2], + \ 'type': 'W', + \ 'nr': -1 + \}) + endfor + + return l:output +endfunction + +call ale#linter#Define('tex', { +\ 'name': 'lacheck', +\ 'executable': 'lacheck', +\ 'command': g:ale#util#stdin_wrapper . ' .tex ' +\ . g:ale_tex_lacheck_executable, +\ 'callback': 'ale_linters#tex#lacheck#Handle' +\}) diff --git a/autoload/ale/linter.vim b/autoload/ale/linter.vim index 9f4eb79..56cb765 100644 --- a/autoload/ale/linter.vim +++ b/autoload/ale/linter.vim @@ -9,6 +9,7 @@ let s:linters = {} let s:default_ale_linter_aliases = { \ 'zsh': 'sh', \ 'csh': 'sh', +\ 'plaintex': 'tex' \} " Default linters to run for particular filetypes. diff --git a/doc/ale.txt b/doc/ale.txt index 01b2332..eae5c28 100644 --- a/doc/ale.txt +++ b/doc/ale.txt @@ -27,6 +27,9 @@ CONTENTS *ale-contents* 4.15. htmlhint..............................|ale-linter-options-htmlhint| 4.16. c-clang...............................|ale-linter-options-c-clang| 4.17. python-flake8.........................|ale-linter-options-python-flake8| + 4.18. ruby-rubocop..........................|ale-linter-options-ruby-rubocop| + 4.19. chktex................................|ale-linter-options-chktex| + 4.20. lacheck...............................|ale-linter-options-lacheck| 5. Linter Integration Notes...................|ale-linter-integration| 5.1. merlin................................|ale-linter-integration-ocaml-merlin| 6. Commands/Keybinds..........................|ale-commands| @@ -74,6 +77,7 @@ The following languages and tools are supported. * HTML: 'HTMLHint', 'tidy' * JavaScript: 'eslint', 'jscs', 'jshint', 'flow' * JSON: 'jsonlint' +* LaTeX: 'chktex', 'lacheck' * Lua: 'luacheck' * Markdown: 'mdl' * MATLAB: 'mlint' @@ -641,6 +645,33 @@ g:ale_ruby_rubocop_options *g:ale_ruby_rubocop_options* This variable can be change to modify flags given to rubocop. +------------------------------------------------------------------------------- +4.19. chktex *ale-linter-options-chktex* + +g:ale_tex_chktex_executable *g:ale_tex_chktex_executable* + + Type: |String| + Default: `'chktex'` + + This variable can be changed to change the path to chktex. + +g:ale_tex_chktex_options *g:ale_tex_chktex_options* + + Type: |String| + Default: `'-I'` + + This variable can be changed to modify flags given to chktex. + +------------------------------------------------------------------------------ +4.20. lacheck *ale-linter-options-lacheck* + +g:ale_lacheck_executable *g:ale_lacheck_executable* + + Type: |String| + Default: '`lacheck`' + + This variable can be changed to change the path to lacheck. + =============================================================================== 5. Linter Integration Notes *ale-linter-integration*