marc/ale
Archived
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix a bug with eslint parsing which mistakenly cleared markers

Browse source
This commit is contained in:
w0rp 2016-09-09 22:25:26 +01:00
parent 11c11e578f
commit 37065bb252
1 changed files with 15 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
ale_linters/javascript/eslint.vim
View file

@ -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