Perform less comparisons in compareBufferPositions

atom · May 10, 2017 · b131c3b · b131c3b
1 parent 8b273ef
commit b131c3b
Showing 1 changed file with 2 additions and 13 deletions.
diff --git a/src/screen-line-builder.js b/src/screen-line-builder.js
@@ -406,18 +406,7 @@ class ScreenLineBuilder {
   }
 
   compareBufferPosition (position) {
-    if (this.bufferRow < position.row) {
-      return -1
-    } else if (this.bufferRow === position.row) {
-      if (this.bufferColumn < position.column) {
-        return -1
-      } else if (this.bufferColumn === position.column) {
-        return 0
-      } else {
-        return 1
-      }
-    } else {
-      return 1
-    }
+    const rowComparison = this.bufferRow - position.row
+    return rowComparison === 0 ? (this.bufferColumn - position.column) : rowComparison
   }
 }