summaryrefslogtreecommitdiff
path: root/ale_linters/javascript/jshint.vim
diff options
context:
space:
mode:
authorBjorn Neergaard <bjorn@neersighted.com>2016-10-10 18:43:45 -0500
committerBjorn Neergaard <bjorn@neersighted.com>2016-10-11 06:14:26 -0500
commitfb4b797dd22b2ec225542c97e5c0c4195edf037e (patch)
treebe6bc337e1403aaded57cfaaf0ce506e73cfb225 /ale_linters/javascript/jshint.vim
parentca4badfb3a0ae73d4fcac3512c8cfae2a6a94f03 (diff)
downloadale-fb4b797dd22b2ec225542c97e5c0c4195edf037e.zip
Use explicit scope in all ale_linters
vint -s is now clean
Diffstat (limited to 'ale_linters/javascript/jshint.vim')
-rw-r--r--ale_linters/javascript/jshint.vim36
1 files changed, 18 insertions, 18 deletions
diff --git a/ale_linters/javascript/jshint.vim b/ale_linters/javascript/jshint.vim
index 11c60409..6dbd4c2a 100644
--- a/ale_linters/javascript/jshint.vim
+++ b/ale_linters/javascript/jshint.vim
@@ -14,21 +14,21 @@ function! ale_linters#javascript#jshint#GetCommand(buffer)
" Set this to the location of the jshint configuration file to
" use a fixed location for .jshintrc
if exists('g:ale_jshint_config_loc')
- let jshint_config = g:ale_jshint_config_loc
+ let l:jshint_config = g:ale_jshint_config_loc
else
" Look for the JSHint config in parent directories.
- let jshint_config = ale#util#FindNearestFile(a:buffer, '.jshintrc')
+ let l:jshint_config = ale#util#FindNearestFile(a:buffer, '.jshintrc')
endif
- let command = g:ale_javascript_jshint_executable . ' --reporter unix'
+ let l:command = g:ale_javascript_jshint_executable . ' --reporter unix'
- if !empty(jshint_config)
- let command .= ' --config ' . fnameescape(jshint_config)
+ if !empty(l:jshint_config)
+ let l:command .= ' --config ' . fnameescape(l:jshint_config)
endif
- let command .= ' -'
+ let l:command .= ' -'
- return command
+ return l:command
endfunction
function! ale_linters#javascript#jshint#Handle(buffer, lines)
@@ -38,36 +38,36 @@ function! ale_linters#javascript#jshint#Handle(buffer, lines)
" stdin:60:5: Attempting to override 'test2' which is a constant.
" stdin:57:10: 'test' is defined but never used.
" stdin:57:1: 'function' is defined but never used.
- 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_parts = l:match[4]
+ let l:text = l:match[3]
+ let l:marker_parts = l:match[4]
- 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,
+ \ 'text': l:text,
\ 'type': 'E',
\ 'nr': -1,
\})
endfor
- return output
+ return l:output
endfunction
call ale#linter#Define('javascript', {