Merge pull request #2 from fijshion/jshint-linter

Add jshint linter

2 files changed, 59 insertions, 1 deletions

diff --git a/README.md b/README.md
index 9f09e0ad..f0ca5f66 100644
--- a/README.md
+++ b/README.md
@@ -27,7 +27,7 @@ name. That seems to be the fairest way to arrange this table.
 | -------- | ----- |
 | Bash | [-n flag](https://www.gnu.org/software/bash/manual/bash.html#index-set) |
 | Bourne Shell | [-n flag](http://linux.die.net/man/1/sh) |
-| JavaScript | [eslint](http://eslint.org/), [jscs](http://jscs.info/) |
+| JavaScript | [eslint](http://eslint.org/), [jscs](http://jscs.info/), [jshint](http://jshint.com/) |
 | Python | [flake8](http://flake8.pycqa.org/en/latest/) |
 | Ruby   | [rubocop](https://github.com/bbatsov/rubocop) |
 
diff --git a/ale_linters/javascript/jshint.vim b/ale_linters/javascript/jshint.vim
new file mode 100644
index 00000000..ff440221
--- /dev/null
+++ b/ale_linters/javascript/jshint.vim
@@ -0,0 +1,58 @@
+" Author: Chris Kyrouac - https://github.com/fijshion
+
+if exists('g:loaded_ale_linters_javascript_jshint')
+    finish
+endif
+
+let g:loaded_ale_linters_javascript_jshint = 1
+
+" Set this to the location of the jshint configuration file
+if !exists('g:ale_jshint_config_loc')
+    let g:ale_jshint_config_loc = '.jshintrc'
+endif
+
+function! ale_linters#javascript#jshint#Handle(buffer, lines)
+    " Matches patterns line the following:
+    "
+    " stdin:57:9: Missing name in function declaration.
+    " 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 = []
+
+    for line in a:lines
+        let l:match = matchlist(line, pattern)
+
+        if len(l:match) == 0
+            continue
+        endif
+
+        let text = l:match[3]
+        let marker_parts = l:match[4]
+
+        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': a:buffer,
+        \   'lnum': l:match[1] + 0,
+        \   'vcol': 0,
+        \   'col': l:match[2] + 0,
+        \   'text': text,
+        \   'type': 'E',
+        \   'nr': -1,
+        \})
+    endfor
+
+    return output
+endfunction
+
+call ALEAddLinter('javascript', {
+\   'name': 'jshint',
+\   'executable': 'jshint',
+\   'command': 'jshint --reporter unix --config ' . g:ale_jshint_config_loc . ' -',
+\   'callback': 'ale_linters#javascript#jshint#Handle',
+\})