From 32e8463479fc80187e84cf56a70c5bfa061fe219 Mon Sep 17 00:00:00 2001 From: kangax Date: Mon, 22 Sep 2014 20:40:28 +0200 Subject: [PATCH] =?UTF-8?q?Add=20requirement=20for=20space=20before=20func?= =?UTF-8?q?tion=E2=80=99s=20{?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .jscs.json | 4 +++- src/color.class.js | 2 +- src/filters/blend_filter.class.js | 4 ++-- src/mixins/canvas_events.mixin.js | 2 +- src/mixins/canvas_grouping.mixin.js | 2 +- src/mixins/object_interactivity.mixin.js | 2 +- src/mixins/observable.mixin.js | 2 +- src/node.js | 2 +- src/shapes/ellipse.class.js | 2 +- src/shapes/group.class.js | 2 +- src/static_canvas.class.js | 2 +- src/util/dom_request.js | 2 +- src/util/lang_class.js | 2 +- src/util/lang_object.js | 2 +- 14 files changed, 17 insertions(+), 15 deletions(-) diff --git a/.jscs.json b/.jscs.json index ec02d5b5..f55c4a1a 100644 --- a/.jscs.json +++ b/.jscs.json @@ -13,7 +13,9 @@ "requireCapitalizedConstructors": true, "requireDotNotation": true, "requireMultipleVarDecl": true, - + "requireSpacesInFunction": { + "beforeOpeningCurlyBrace": true + }, "disallowEmptyBlocks": true, "disallowQuotedKeysInObjects": "allButReserved", diff --git a/src/color.class.js b/src/color.class.js index da6af426..e7329a7d 100644 --- a/src/color.class.js +++ b/src/color.class.js @@ -313,7 +313,7 @@ * @param {Number} t * @return {Number} */ - function hue2rgb(p, q, t){ + function hue2rgb(p, q, t) { if (t < 0) { t += 1; } diff --git a/src/filters/blend_filter.class.js b/src/filters/blend_filter.class.js index 39bbc196..93c632bf 100644 --- a/src/filters/blend_filter.class.js +++ b/src/filters/blend_filter.class.js @@ -1,4 +1,4 @@ -(function(global){ +(function(global) { 'use strict'; var fabric = global.fabric; @@ -26,7 +26,7 @@ fabric.Image.filters.Blend = fabric.util.createClass({ type: 'Blend', - initialize: function(options){ + initialize: function(options) { options = options || {}; this.color = options.color || '#000'; this.image = options.image || false; diff --git a/src/mixins/canvas_events.mixin.js b/src/mixins/canvas_events.mixin.js index 8167dbad..466ea6e1 100644 --- a/src/mixins/canvas_events.mixin.js +++ b/src/mixins/canvas_events.mixin.js @@ -1,4 +1,4 @@ -(function(){ +(function() { var cursorOffset = { mt: 0, // n diff --git a/src/mixins/canvas_grouping.mixin.js b/src/mixins/canvas_grouping.mixin.js index e0821a43..739b49f8 100644 --- a/src/mixins/canvas_grouping.mixin.js +++ b/src/mixins/canvas_grouping.mixin.js @@ -1,4 +1,4 @@ -(function(){ +(function() { var min = Math.min, max = Math.max; diff --git a/src/mixins/object_interactivity.mixin.js b/src/mixins/object_interactivity.mixin.js index 59ec95b7..074716d7 100644 --- a/src/mixins/object_interactivity.mixin.js +++ b/src/mixins/object_interactivity.mixin.js @@ -1,4 +1,4 @@ -(function(){ +(function() { var degreesToRadians = fabric.util.degreesToRadians, //jscs:disable requireCamelCaseOrUpperCaseIdentifiers diff --git a/src/mixins/observable.mixin.js b/src/mixins/observable.mixin.js index e467da02..b5411abb 100644 --- a/src/mixins/observable.mixin.js +++ b/src/mixins/observable.mixin.js @@ -1,4 +1,4 @@ -(function(){ +(function() { /** * @private diff --git a/src/node.js b/src/node.js index d2d5f96d..9ea80b4b 100644 --- a/src/node.js +++ b/src/node.js @@ -56,7 +56,7 @@ } /** @private */ - function requestFs(path, callback){ + function requestFs(path, callback) { var fs = require('fs'); fs.readFile(path, function (err, data) { if (err) { diff --git a/src/shapes/ellipse.class.js b/src/shapes/ellipse.class.js index a6d63e9d..55d37f69 100644 --- a/src/shapes/ellipse.class.js +++ b/src/shapes/ellipse.class.js @@ -1,4 +1,4 @@ -(function(global){ +(function(global) { 'use strict'; diff --git a/src/shapes/group.class.js b/src/shapes/group.class.js index 62d21a16..80edc5ab 100644 --- a/src/shapes/group.class.js +++ b/src/shapes/group.class.js @@ -1,4 +1,4 @@ -(function(global){ +(function(global) { 'use strict'; diff --git a/src/static_canvas.class.js b/src/static_canvas.class.js index 8b934cc3..6b4ea2c5 100644 --- a/src/static_canvas.class.js +++ b/src/static_canvas.class.js @@ -321,7 +321,7 @@ * @private * @see {@link http://www.whatwg.org/specs/web-apps/current-work/multipage/the-canvas-element.html#dom-context-2d-imagesmoothingenabled|WhatWG Canvas Standard} */ - _setImageSmoothing: function(){ + _setImageSmoothing: function() { var ctx = this.getContext(); ctx.imageSmoothingEnabled = this.imageSmoothingEnabled; diff --git a/src/util/dom_request.js b/src/util/dom_request.js index 33079409..1f53c714 100644 --- a/src/util/dom_request.js +++ b/src/util/dom_request.js @@ -1,4 +1,4 @@ -(function(){ +(function() { function addParamToUrl(url, param) { return url + (/\?/.test(url) ? '&' : '?') + param; diff --git a/src/util/lang_class.js b/src/util/lang_class.js index 69220ea3..7a565302 100644 --- a/src/util/lang_class.js +++ b/src/util/lang_class.js @@ -2,7 +2,7 @@ var slice = Array.prototype.slice, emptyFunction = function() { }, - IS_DONTENUM_BUGGY = (function(){ + IS_DONTENUM_BUGGY = (function() { for (var p in { toString: 1 }) { if (p === 'toString') { return false; diff --git a/src/util/lang_object.js b/src/util/lang_object.js index f6f4659d..a21f874a 100644 --- a/src/util/lang_object.js +++ b/src/util/lang_object.js @@ -1,4 +1,4 @@ -(function(){ +(function() { /** * Copies all enumerable properties of one object to another