Merge pull request #234 from AtomLinter/check-max-line

Workaround jshint#2894
AtomLinter · Mar 11, 2016 · 5951296 · 5951296
2 parents f813820 + 5145023
commit 5951296
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/lib/main.js b/lib/main.js
@@ -137,12 +137,19 @@ module.exports = {
             range = Helpers.rangeFromLineNumber(textEditor, errorLine)
           } else {
             let character = error.character > 0 ? error.character - 1 : 0
+            let line = errorLine
+            const buffer = textEditor.getBuffer()
+            const maxLine = buffer.getLineCount()
+            // TODO: Remove workaround of jshint/jshint#2894
+            if (errorLine >= maxLine) {
+              line = maxLine
+            }
+            const maxCharacter = buffer.lineLengthForRow(line)
             // TODO: Remove workaround of jquery/esprima#1457
-            const maxCharacter = textEditor.getBuffer().lineLengthForRow(errorLine)
             if (character > maxCharacter) {
               character = maxCharacter
             }
-            range = Helpers.rangeFromLineNumber(textEditor, errorLine, character)
+            range = Helpers.rangeFromLineNumber(textEditor, line, character)
           }
 
           results.push({