[sublime keymap] Fix coding style in 8eb523db08b80b0ff70728d9b71475cadb2bcf93

Also, wrap the command in an operation and use the head rather than
the anchor of the selections.

Author: marijnh

keymap/sublime.js

@@ -394,17 +394,24 @@
   };
 
   cmds[map["Shift-Alt-Up"] = "selectLinesUpward"] = function(cm) {
-    var curs = cm.listSelections();
-    for (var i = 0; i < curs.length; i++) {
-      if(curs[i].anchor.line>0)
-        cm.addSelection({line:curs[i].anchor.line-1,ch:curs[i].anchor.ch});
-    };
+    cm.operation(function() {
+      var ranges = cm.listSelections();
+      for (var i = 0; i < ranges.length; i++) {
+        var range = ranges[i];
+        if (range.head.line > cm.firstLine())
+          cm.addSelection(Pos(range.head.line - 1, range.head.ch));
+      }
+    });
   };
   cmds[map["Shift-Alt-Down"] = "selectLinesDownward"] = function(cm) {
-    var curs = cm.listSelections();
-    for (var i = 0; i < curs.length; i++) {
-      cm.addSelection({line:curs[i].anchor.line+1,ch:curs[i].anchor.ch});
-    };
+    cm.operation(function() {
+      var ranges = cm.listSelections();
+      for (var i = 0; i < ranges.length; i++) {
+        var range = ranges[i];
+        if (range.head.line < cm.lastLine())
+          cm.addSelection(Pos(range.head.line + 1, range.head.ch));
+      }
+    });
   };
 
   map["Shift-" + ctrl + "["] = "fold";

test/sublime_test.js

@@ -184,6 +184,31 @@
          val("abc\nabc\ndef\ndef\nghi\nghi"), hasSel(1, 1, 1, 1,
                                                      3, 1, 3, 1,
                                                      5, 1, 5, 1));
+  stTest("selectLinesUpward", "123\n345\n789\n012",
+         setSel(0, 1, 0, 1,
+                1, 1, 1, 3,
+                2, 0, 2, 0,
+                3, 0, 3, 0),
+         "selectLinesUpward",
+         hasSel(0, 1, 0, 1,
+                0, 3, 0, 3,
+                1, 0, 1, 0,
+                1, 1, 1, 3,
+                2, 0, 2, 0,
+                3, 0, 3, 0));
+
+  stTest("selectLinesDownward", "123\n345\n789\n012",
+         setSel(0, 1, 0, 1,
+                1, 1, 1, 3,
+                2, 0, 2, 0,
+                3, 0, 3, 0),
+         "selectLinesDownward",
+         hasSel(0, 1, 0, 1,
+                1, 1, 1, 3,
+                2, 0, 2, 0,
+                2, 3, 2, 3,
+                3, 0, 3, 0));
+
   stTest("sortLines", "c\nb\na\nC\nB\nA",
          "sortLines", val("A\nB\nC\na\nb\nc"),
          "undo",