Fix: no-unused-disable false positive

Author: mysticatea

lib/disabled-area.js

@@ -25,18 +25,6 @@ function lte(a, b) {
     return a.line < b.line || (a.line === b.line && a.column <= b.column)
 }
 
-/**
- * Checks `a` is less than `b`.
- *
- * @param {{line: number, column: number}} a - A location to compare.
- * @param {{line: number, column: number}} b - Another location to compare.
- * @returns {boolean} `true` if `a` is less than `b`.
- * @private
- */
-function lt(a, b) {
-    return a.line < b.line || (a.line === b.line && a.column < b.column)
-}
-
 //------------------------------------------------------------------------------
 // Exports
 //------------------------------------------------------------------------------
@@ -182,9 +170,6 @@ module.exports = class DisabledArea {
         for (let i = this.areas.length - 1; i >= 0; --i) {
             const area = this.areas[i]
 
-            if (lt(location, area.start)) {
-                break
-            }
             if ((area.ruleId === null || area.ruleId === ruleId) &&
                 lte(area.start, location) &&
                 (area.end === null || lte(location, area.end))

tests/lib/rules/no-unused-disable.js

@@ -56,6 +56,17 @@ var a = b
 /*eslint-disable no-undef,no-unused-vars*/
 var a = b
 /*eslint-enable no-undef*/`,
+
+        `
+/*eslint no-shadow:error */
+var foo = 1
+function bar() {
+    var foo = 2  //eslint-disable-line no-shadow
+}
+function baz() {
+    var foo = 3  //eslint-disable-line no-shadow
+}
+`,
     ],
     invalid: [
         {