style(*): add numerous JSCS rules to unify code-styles in the tree

Changes:

  - add rule requireSpaceBeforeBlockStatements (require space before brace when opening block statement)
  - add operators to rule disallowSpaceAfterPrefixUnaryOperators (no space after prefix inc/dec ops)
  - add rule disallowSpaceBeforePostfixUnaryOperators (no space before postfix inc/dec ops)
  - add rule disallowSpacesInsideArrayBrackets (array literals no longer padded with spaces)
  - add rule requireCommaBeforeLineBreak (line can't start with comma token)
  - add rule validateLineBreaks (require LF linebreaks)

Closes #9792

Author: hzoo

.jscs.json

@@ -4,8 +4,9 @@
   "disallowMultipleLineStrings": true,
   "disallowNewlineBeforeBlockStatements": true,
   "disallowSpaceAfterObjectKeys": true,
-  "disallowSpaceAfterPrefixUnaryOperators": ["!"],
+  "disallowSpaceAfterPrefixUnaryOperators": ["++", "--", "+", "-", "~", "!"],
   "disallowSpaceBeforeBinaryOperators": [","],
+  "disallowSpaceBeforePostfixUnaryOperators": ["++", "--"],
   "disallowSpacesInAnonymousFunctionExpression": {
     "beforeOpeningRoundBrace": true
   },
@@ -15,12 +16,17 @@
   "disallowSpacesInNamedFunctionExpression": {
     "beforeOpeningRoundBrace": true
   },
+  "disallowSpacesInsideArrayBrackets": true,
   "disallowSpacesInsideParentheses": true,
   "disallowTrailingComma": true,
   "disallowTrailingWhitespace": true,
+  "requireCommaBeforeLineBreak": true,
+  "requireLineFeedAtFileEnd": true,
   "requireSpaceAfterKeywords": ["if", "else", "for", "while", "do", "switch", "return", "try", "catch"],
+  "requireSpaceBeforeBlockStatements": true,
   "requireSpacesInFunction": {
     "beforeOpeningCurlyBrace": true
   },
+  "validateLineBreaks": "LF",
   "validateParameterSeparator": ", "
 }

.jscs.json.todo

@@ -10,7 +10,6 @@
   "disallowImplicitTypeConversion": ["string"],
   "disallowMultipleLineBreaks": true,
   "disallowKeywordsOnNewLine": ["else"],
-  "requireLineFeedAtFileEnd": true,
   "validateJSDoc": {
     "checkParamNames": true,
     "requireParamTypes": true

i18n/src/closureI18nExtractor.js

@@ -224,7 +224,7 @@ function outputLocale(localeInfo, localeID) {
           getDecimals + getVF + getWT +
           '$provide.value("$locale", ';
 
-  var suffix = ');\n}]);';
+  var suffix = ');\n}]);\n';
 
   return prefix + content + suffix;
 }

src/jqLite.js

@@ -481,7 +481,7 @@ var JQLitePrototype = JQLite.prototype = {
     }
 
     // check if document is already loaded
-    if (document.readyState === 'complete'){
+    if (document.readyState === 'complete') {
       setTimeout(trigger);
     } else {
       this.on('DOMContentLoaded', trigger); // works for modern browsers and IE9
@@ -802,7 +802,7 @@ forEach({
             var target = this, related = event.relatedTarget;
             // For mousenter/leave call the handler if related is outside the target.
             // NB: No relatedTarget if the mouse left/entered the browser window
-            if (!related || (related !== target && !target.contains(related))){
+            if (!related || (related !== target && !target.contains(related))) {
               handle(event, type);
             }
           });

src/ng/http.js

@@ -1092,7 +1092,7 @@ function $HttpProvider() {
 
         forEach(value, function(v) {
           if (isObject(v)) {
-            if (isDate(v)){
+            if (isDate(v)) {
               v = v.toISOString();
             } else {
               v = toJson(v);

src/ngCookies/cookies.js

@@ -103,7 +103,7 @@ angular.module('ngCookies', ['ng']).
         }
 
         //verify what was actually stored
-        if (updated){
+        if (updated) {
           updated = false;
           browserCookies = $browser.cookies();
 

src/ngLocale/angular-locale_aa-dj.js

@@ -112,4 +112,4 @@ $provide.value("$locale", {
   "id": "aa-dj",
   "pluralCat": function(n, opt_precision) {  var i = n | 0;  var vf = getVF(n, opt_precision);  if (i == 1 && vf.v == 0) {    return PLURAL_CATEGORY.ONE;  }  return PLURAL_CATEGORY.OTHER;}
 });
-}]);
+}]);

src/ngLocale/angular-locale_aa-er.js

@@ -112,4 +112,4 @@ $provide.value("$locale", {
   "id": "aa-er",
   "pluralCat": function(n, opt_precision) {  var i = n | 0;  var vf = getVF(n, opt_precision);  if (i == 1 && vf.v == 0) {    return PLURAL_CATEGORY.ONE;  }  return PLURAL_CATEGORY.OTHER;}
 });
-}]);
+}]);

src/ngLocale/angular-locale_aa-et.js

@@ -112,4 +112,4 @@ $provide.value("$locale", {
   "id": "aa-et",
   "pluralCat": function(n, opt_precision) {  var i = n | 0;  var vf = getVF(n, opt_precision);  if (i == 1 && vf.v == 0) {    return PLURAL_CATEGORY.ONE;  }  return PLURAL_CATEGORY.OTHER;}
 });
-}]);
+}]);

src/ngLocale/angular-locale_aa.js

@@ -112,4 +112,4 @@ $provide.value("$locale", {
   "id": "aa",
   "pluralCat": function(n, opt_precision) {  var i = n | 0;  var vf = getVF(n, opt_precision);  if (i == 1 && vf.v == 0) {    return PLURAL_CATEGORY.ONE;  }  return PLURAL_CATEGORY.OTHER;}
 });
-}]);
+}]);