cleanup

lib/ace/editor.js

@@ -1276,43 +1276,35 @@ var Editor = function(renderer, session) {
         this.session.outdentRows(selection.getRange());
     };
 
-    /**
+    // TODO: move out of core when we have good mechanism for managing extensions
-     * Editor.sortLines() 
-     * 
-     * Given the currently selected range, this function either comments all lines or uncomments all lines (depending on whether it's commented or not).
-     **/
     this.sortLines = function() {
-        var state = this.session.getState(this.getCursorPosition().row);
         var rows = this.$getSelectedRows();
-        var doc = this.session;
+        var session = this.session;
 
         var lines = [];
-        for(i = rows.first; i <= rows.last; i++) {
+        for (i = rows.first; i <= rows.last; i++)
-            lines.push( doc.getLine(i) );
+            lines.push(session.getLine(i));
-        }
+
-        lines.sort(
+        lines.sort(function(a, b) {
-            function(a, b) {
             if (a.toLowerCase() < b.toLowerCase()) return -1;
             if (a.toLowerCase() > b.toLowerCase()) return 1;
             return 0;
-                }
+        });
-            );
 
         var deleteRange = new Range(0, 0, 0, 0);
-        for (var i=rows.first; i<= rows.last; i++)
+        for (var i = rows.first; i <= rows.last; i++) {
-        {
+            var line = session.getLine(i);
-            var line = doc.getLine(i);           
             deleteRange.start.row = i;
             deleteRange.end.row = i;
             deleteRange.end.column = line.length;
-            doc.replace(deleteRange, lines[i-rows.first]);
+            session.replace(deleteRange, lines[i-rows.first]);
         }
     };
 
     /**
      * Editor.toggleCommentLines() 
      * 
-     * Given the currently selected range, this function sorts the lines alphabetically ascending.
+     * Given the currently selected range, this function either comments all lines or uncomments all lines (depending on whether it's commented or not).
      **/
     this.toggleCommentLines = function() {
         var state = this.session.getState(this.getCursorPosition().row);