Merge pull request #2068 from asturur/fix-jscs

attemp to fix jscs crash

src/brushes/spray_brush.class.js

@@ -123,10 +123,10 @@ fabric.SprayBrush = fabric.util.createClass( fabric.BaseBrush, /** @lends fabric
     this.canvas.renderAll();
   },
 
-    /**
+  /**
-     * @private
+   * @private
-     * @param {Array} rects
+   * @param {Array} rects
-     */
+   */
   _getOptimizedRects: function(rects) {
 
     // avoid creating duplicate rects at the same coordinates

src/canvas.class.js

@@ -808,7 +808,7 @@
       var target,
           pointer = this.getPointer(e, true),
           i = this._objects.length;
-     // Do not check for currently grouped objects, since we check the parent group itself.
+      // Do not check for currently grouped objects, since we check the parent group itself.
       while (i--) {
         if (!this._objects[i].group && this._checkTarget(e, this._objects[i], pointer)){
           this.relatedTarget = this._objects[i];

src/filters/blend_filter.class.js

@@ -40,6 +40,7 @@
           data = imageData.data,
           tr, tg, tb,
           r, g, b,
+          _r, _g, _b,
           source,
           isImage = false;
 
@@ -100,9 +101,9 @@
             data[i + 2] = Math.abs(b - tb);
             break;
           case 'subtract':
-            var _r = r - tr,
+            _r = r - tr;
-                _g = g - tg,
+            _g = g - tg;
-                _b = b - tb;
+            _b = b - tb;
 
             data[i] = (_r < 0) ? 0 : _r;
             data[i + 1] = (_g < 0) ? 0 : _g;

src/mixins/canvas_events.mixin.js

@@ -638,7 +638,7 @@
       if (transform.action === 'scale' || transform.action === 'scaleX' || transform.action === 'scaleY') {
         var centerTransform = this._shouldCenterTransform(e, transform.target);
 
-           // Switch from a normal resize to center-based
+        // Switch from a normal resize to center-based
         if ((centerTransform && (transform.originX !== 'center' || transform.originY !== 'center')) ||
            // Switch from center-based resize to normal one
            (!centerTransform && transform.originX === 'center' && transform.originY === 'center')

src/mixins/itext_behavior.mixin.js

@@ -558,8 +558,8 @@
         this.insertStyleObjects(_chars, isEndOfLine, useCopiedStyle);
       }
       // else if (this.selectionEnd - this.selectionStart > 1) {
-        // TODO: replace styles properly
+      // TODO: replace styles properly
-        // console.log('replacing MORE than 1 char');
+      // console.log('replacing MORE than 1 char');
       // }
       this.setSelectionStart(this.selectionStart + _chars.length);
       this.setSelectionEnd(this.selectionStart);

src/mixins/object_interactivity.mixin.js

@@ -3,7 +3,7 @@
   var degreesToRadians = fabric.util.degreesToRadians,
       //jscs:disable requireCamelCaseOrUpperCaseIdentifiers
       isVML = function() { return typeof G_vmlCanvasManager !== 'undefined'; };
-      //jscs:enable requireCamelCaseOrUpperCaseIdentifiers
+  //jscs:enable requireCamelCaseOrUpperCaseIdentifiers
 
   fabric.util.object.extend(fabric.Object.prototype, /** @lends fabric.Object.prototype */ {
 

src/parser.js

@@ -555,10 +555,10 @@
     };
   })();
 
-   /**
+  /**
-    * Used for caching SVG documents (loaded via `fabric.Canvas#loadSVGFromURL`)
+   * Used for caching SVG documents (loaded via `fabric.Canvas#loadSVGFromURL`)
-    * @namespace
+   * @namespace
-    */
+   */
   var svgCache = {
 
     /**
@@ -821,7 +821,7 @@
 
       // odd number of points is an error
       // if (parsedPoints.length % 2 !== 0) {
-        // return null;
+      //   return null;
       // }
 
       return parsedPoints;

src/shapes/itext.class.js

@@ -2,52 +2,52 @@
 
   var clone = fabric.util.object.clone;
 
-   /**
+  /**
-    * IText class (introduced in <b>v1.4</b>) Events are also fired with "text:"
+   * IText class (introduced in <b>v1.4</b>) Events are also fired with "text:"
-    * prefix when observing canvas.
+   * prefix when observing canvas.
-    * @class fabric.IText
+   * @class fabric.IText
-    * @extends fabric.Text
+   * @extends fabric.Text
-    * @mixes fabric.Observable
+   * @mixes fabric.Observable
-    *
+   *
-    * @fires changed
+   * @fires changed
-    * @fires selection:changed
+   * @fires selection:changed
-    * @fires editing:entered
+   * @fires editing:entered
-    * @fires editing:exited
+   * @fires editing:exited
-    *
+   *
-    * @return {fabric.IText} thisArg
+   * @return {fabric.IText} thisArg
-    * @see {@link fabric.IText#initialize} for constructor definition
+   * @see {@link fabric.IText#initialize} for constructor definition
-    *
+   *
-    * <p>Supported key combinations:</p>
+   * <p>Supported key combinations:</p>
-    * <pre>
+   * <pre>
-    *   Move cursor:                    left, right, up, down
+   *   Move cursor:                    left, right, up, down
-    *   Select character:               shift + left, shift + right
+   *   Select character:               shift + left, shift + right
-    *   Select text vertically:         shift + up, shift + down
+   *   Select text vertically:         shift + up, shift + down
-    *   Move cursor by word:            alt + left, alt + right
+   *   Move cursor by word:            alt + left, alt + right
-    *   Select words:                   shift + alt + left, shift + alt + right
+   *   Select words:                   shift + alt + left, shift + alt + right
-    *   Move cursor to line start/end:  cmd + left, cmd + right or home, end
+   *   Move cursor to line start/end:  cmd + left, cmd + right or home, end
-    *   Select till start/end of line:  cmd + shift + left, cmd + shift + right or shift + home, shift + end
+   *   Select till start/end of line:  cmd + shift + left, cmd + shift + right or shift + home, shift + end
-    *   Jump to start/end of text:      cmd + up, cmd + down
+   *   Jump to start/end of text:      cmd + up, cmd + down
-    *   Select till start/end of text:  cmd + shift + up, cmd + shift + down or shift + pgUp, shift + pgDown
+   *   Select till start/end of text:  cmd + shift + up, cmd + shift + down or shift + pgUp, shift + pgDown
-    *   Delete character:               backspace
+   *   Delete character:               backspace
-    *   Delete word:                    alt + backspace
+   *   Delete word:                    alt + backspace
-    *   Delete line:                    cmd + backspace
+   *   Delete line:                    cmd + backspace
-    *   Forward delete:                 delete
+   *   Forward delete:                 delete
-    *   Copy text:                      ctrl/cmd + c
+   *   Copy text:                      ctrl/cmd + c
-    *   Paste text:                     ctrl/cmd + v
+   *   Paste text:                     ctrl/cmd + v
-    *   Cut text:                       ctrl/cmd + x
+   *   Cut text:                       ctrl/cmd + x
-    *   Select entire text:             ctrl/cmd + a
+   *   Select entire text:             ctrl/cmd + a
-    *   Quit editing                    tab or esc
+   *   Quit editing                    tab or esc
-    * </pre>
+   * </pre>
-    *
+   *
-    * <p>Supported mouse/touch combination</p>
+   * <p>Supported mouse/touch combination</p>
-    * <pre>
+   * <pre>
-    *   Position cursor:                click/touch
+   *   Position cursor:                click/touch
-    *   Create selection:               click/touch & drag
+   *   Create selection:               click/touch & drag
-    *   Create selection:               click & shift + click
+   *   Create selection:               click & shift + click
-    *   Select word:                    double click
+   *   Select word:                    double click
-    *   Select line:                    triple click
+   *   Select line:                    triple click
-    * </pre>
+   * </pre>
-    */
+   */
   fabric.IText = fabric.util.createClass(fabric.Text, fabric.Observable, /** @lends fabric.IText.prototype */ {
 
     /**
@@ -405,8 +405,8 @@
      */
     _getCursorBoundaries: function(chars, typeOfBoundaries) {
 
-          // left/top are left/top of entire text box
+      // left/top are left/top of entire text box
-          // leftOffset/topOffset are offset from that left/top point of a text box
+      // leftOffset/topOffset are offset from that left/top point of a text box
 
       var left = Math.round(this._getLeftOffset()),
           top = this._getTopOffset(),

src/util/animate.js

@@ -1,17 +1,17 @@
 (function() {
 
-   /**
+  /**
-    * Changes value from one to another within certain period of time, invoking callbacks as value is being changed.
+   * Changes value from one to another within certain period of time, invoking callbacks as value is being changed.
-    * @memberOf fabric.util
+   * @memberOf fabric.util
-    * @param {Object} [options] Animation options
+   * @param {Object} [options] Animation options
-    * @param {Function} [options.onChange] Callback; invoked on every value change
+   * @param {Function} [options.onChange] Callback; invoked on every value change
-    * @param {Function} [options.onComplete] Callback; invoked when value change is completed
+   * @param {Function} [options.onComplete] Callback; invoked when value change is completed
-    * @param {Number} [options.startValue=0] Starting value
+   * @param {Number} [options.startValue=0] Starting value
-    * @param {Number} [options.endValue=100] Ending value
+   * @param {Number} [options.endValue=100] Ending value
-    * @param {Number} [options.byValue=100] Value to modify the property by
+   * @param {Number} [options.byValue=100] Value to modify the property by
-    * @param {Function} [options.easing] Easing function
+   * @param {Function} [options.easing] Easing function
-    * @param {Number} [options.duration=500] Duration of change (in ms)
+   * @param {Number} [options.duration=500] Duration of change (in ms)
-    */
+   */
   function animate(options) {
 
     requestAnimFrame(function(timestamp) {