diff options
author | Bjorn Neergaard <bjorn@neersighted.com> | 2016-10-10 18:43:45 -0500 |
---|---|---|
committer | Bjorn Neergaard <bjorn@neersighted.com> | 2016-10-11 06:14:26 -0500 |
commit | fb4b797dd22b2ec225542c97e5c0c4195edf037e (patch) | |
tree | be6bc337e1403aaded57cfaaf0ce506e73cfb225 /ale_linters/javascript/eslint.vim | |
parent | ca4badfb3a0ae73d4fcac3512c8cfae2a6a94f03 (diff) | |
download | ale-fb4b797dd22b2ec225542c97e5c0c4195edf037e.zip |
Use explicit scope in all ale_linters
vint -s is now clean
Diffstat (limited to 'ale_linters/javascript/eslint.vim')
-rw-r--r-- | ale_linters/javascript/eslint.vim | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/ale_linters/javascript/eslint.vim b/ale_linters/javascript/eslint.vim index d140a957..909c1128 100644 --- a/ale_linters/javascript/eslint.vim +++ b/ale_linters/javascript/eslint.vim @@ -15,38 +15,38 @@ function! ale_linters#javascript#eslint#Handle(buffer, lines) " " /path/to/some-filename.js:47:14: Missing trailing comma. [Warning/comma-dangle] " /path/to/some-filename.js:56:41: Missing semicolon. [Error/semi] - let pattern = '^.*:\(\d\+\):\(\d\+\): \(.\+\) \[\(.\+\)\]$' - let output = [] + let l:pattern = '^.*:\(\d\+\):\(\d\+\): \(.\+\) \[\(.\+\)\]$' + let l:output = [] - for line in a:lines - let l:match = matchlist(line, pattern) + for l:line in a:lines + let l:match = matchlist(l:line, l:pattern) if len(l:match) == 0 continue endif - let text = l:match[3] - let marker = l:match[4] - let marker_parts = split(marker, '/') - let type = marker_parts[0] + let l:text = l:match[3] + let l:marker = l:match[4] + let l:marker_parts = split(l:marker, '/') + let l:type = l:marker_parts[0] - if len(marker_parts) == 2 - let text = text . ' (' . marker_parts[1] . ')' + if len(l:marker_parts) == 2 + let l:text = l:text . ' (' . l:marker_parts[1] . ')' endif " vcol is Needed to indicate that the column is a character. - call add(output, { + call add(l:output, { \ 'bufnr': a:buffer, \ 'lnum': l:match[1] + 0, \ 'vcol': 0, \ 'col': l:match[2] + 0, - \ 'text': text, - \ 'type': type ==# 'Warning' ? 'W' : 'E', + \ 'text': l:text, + \ 'type': l:type ==# 'Warning' ? 'W' : 'E', \ 'nr': -1, \}) endfor - return output + return l:output endfunction call ale#linter#Define('javascript', { |