cleanup

ChangeLog.txt

@@ -1,3 +1,8 @@
+  
+* API Changes
+  - `editor.commands.commandKeyBinding` now contains direct map from keys to commands instead of grouping them by hashid
+  - 
+  
 2014.09.21 Version 1.1.7
 
 * Bugfixes

lib/ace/anchor.js

@@ -201,7 +201,7 @@ var Anchor = exports.Anchor = function(doc, row, column) {
     };
 
     /**
-     * When called, the `'change'` event listener is removed.
+     * When called, the `"change"` event listener is removed.
      *
      **/
     this.detach = function() {

lib/ace/commands/command_manager.js

@@ -12,7 +12,7 @@ var EventEmitter = require("../lib/event_emitter").EventEmitter;
 
 /**
  * new CommandManager(platform, commands)
- * @param {String} platform Identifier for the platform; must be either `'mac'` or `'win'`
+ * @param {String} platform Identifier for the platform; must be either `"mac"` or `"win"`
  * @param {Array} commands A list of commands
  *
  **/
@@ -39,7 +39,7 @@ oop.inherits(CommandManager, MultiHashHandler);
             return false;
         }
         
-        if (typeof command === 'string')
+        if (typeof command === "string")
             command = this.commands[command];
 
         if (!command)

lib/ace/edit_session.js

@@ -1601,7 +1601,7 @@ var EditSession = function(text, mode) {
     * @private
     **/
     this.adjustWrapLimit = function(desiredLimit, $printMargin) {
-        var limits = this.$wrapLimitRange
+        var limits = this.$wrapLimitRange;
         if (limits.max < 0)
             limits = {min: $printMargin, max: $printMargin};
         var wrapLimit = this.$constrainWrapLimit(desiredLimit, limits.min, limits.max);
@@ -1725,7 +1725,7 @@ var EditSession = function(text, mode) {
                 var foldLine = this.getFoldLine(firstRow);
                 var idx = 0;
                 if (foldLine) {
-                    var cmp = foldLine.range.compareInside(start.row, start.column)
+                    var cmp = foldLine.range.compareInside(start.row, start.column);
                     // Inside of the foldLine range. Need to split stuff up.
                     if (cmp == 0) {
                         foldLine = foldLine.split(start.row, start.column);
@@ -2201,7 +2201,7 @@ var EditSession = function(text, mode) {
             return {
                 row: maxRow,
                 column: this.getLine(maxRow).length
-            }
+            };
         } else {
             line = this.getLine(docRow);
             foldLine = null;

lib/ace/lib/lang.js

@@ -150,7 +150,6 @@ exports.getMatchOffsets = function(string, regExp) {
 
 /* deprecated */
 exports.deferredCall = function(fcn) {
-
     var timer = null;
     var callback = function() {
         timer = null;

lib/ace/scrollbar.js

@@ -156,9 +156,9 @@ oop.inherits(VScrollBar, ScrollBar);
      * Sets the scroll top of the scroll bar.
      * @param {Number} scrollTop The new scroll top
      **/
-    // on chrome 17+ for small zoom levels after calling this function
-    // this.element.scrollTop != scrollTop which makes page to scroll up.
     this.setScrollTop = function(scrollTop) {
+        // on chrome 17+ for small zoom levels after calling this function
+        // this.element.scrollTop != scrollTop which makes page to scroll up.
         if (this.scrollTop != scrollTop) {
             this.skipEvent = true;
             this.scrollTop = this.element.scrollTop = scrollTop;
@@ -249,9 +249,9 @@ oop.inherits(HScrollBar, ScrollBar);
      * Sets the scroll left of the scroll bar.
      * @param {Number} scrollTop The new scroll left
      **/
-    // on chrome 17+ for small zoom levels after calling this function
-    // this.element.scrollTop != scrollTop which makes page to scroll up.
     this.setScrollLeft = function(scrollLeft) {
+        // on chrome 17+ for small zoom levels after calling this function
+        // this.element.scrollTop != scrollTop which makes page to scroll up.
         if (this.scrollLeft != scrollLeft) {
             this.skipEvent = true;
             this.scrollLeft = this.element.scrollLeft = scrollLeft;