diff options
author | w0rp <devw0rp@gmail.com> | 2016-09-09 22:25:26 +0100 |
---|---|---|
committer | w0rp <devw0rp@gmail.com> | 2016-09-09 22:25:26 +0100 |
commit | 37065bb25248508bcfcae2b2ef65cf5efb46ccd7 (patch) | |
tree | cb120caf38876a546e51cf0892571b7929ef9697 /ale_linters/javascript/eslint.vim | |
parent | 11c11e578f22cda52048fdec1354f9675b413495 (diff) | |
download | ale-37065bb25248508bcfcae2b2ef65cf5efb46ccd7.zip |
Fix a bug with eslint parsing which mistakenly cleared markers
Diffstat (limited to 'ale_linters/javascript/eslint.vim')
-rw-r--r-- | ale_linters/javascript/eslint.vim | 22 |
1 files changed, 15 insertions, 7 deletions
diff --git a/ale_linters/javascript/eslint.vim b/ale_linters/javascript/eslint.vim index d6763eef..83460033 100644 --- a/ale_linters/javascript/eslint.vim +++ b/ale_linters/javascript/eslint.vim @@ -9,24 +9,32 @@ function! ale_linters#javascript#eslint#Handle(lines) " " <text>:47:14: Missing trailing comma. [Warning/comma-dangle] " <text>:56:41: Missing semicolon. [Error/semi] - let pattern = '^<text>:\(\d\+\):\(\d\+\): \(.\+\) \[\(.\+\)/\(.\+\)\]' + let pattern = '^<text>:\(\d\+\):\(\d\+\): \(.\+\) \[\(.\+\)\]' let output = [] for line in a:lines - let match = matchlist(line, pattern) + let l:match = matchlist(line, pattern) - if len(match) == 0 + if len(l:match) == 0 break endif + let text = l:match[3] + let marker_parts = l:match[4] + let type = marker_parts[0] + + if len(marker_parts) == 2 + let text = text . ' (' . marker_parts[1] . ')' + endif + " vcol is Needed to indicate that the column is a character. call add(output, { \ 'bufnr': bufnr('%'), - \ 'lnum': match[1] + 0, + \ 'lnum': l:match[1] + 0, \ 'vcol': 0, - \ 'col': match[2] + 0, - \ 'text': match[3] . '(' . match[5] . ')', - \ 'type': match[4] ==# 'Warning' ? 'W' : 'E', + \ 'col': l:match[2] + 0, + \ 'text': text, + \ 'type': type ==# 'Warning' ? 'W' : 'E', \ 'nr': -1, \}) endfor |