Camelize

Author: asamuzaK

lib/parsers.js

@@ -426,49 +426,46 @@ var dashedToCamelCase = function (dashed) {
 };
 exports.dashedToCamelCase = dashedToCamelCase;
 
-var is_space = /\s/;
-var opening_deliminators = ['"', "'", '('];
-var closing_deliminators = ['"', "'", ')'];
+var isSpace = /\s/;
+var openingDeliminators = ['"', "'", '('];
+var closingDeliminators = ['"', "'", ')'];
 // this splits on whitespace, but keeps quoted and parened parts together
 var getParts = function (str) {
-  var deliminator_stack = [];
+  var deliminatorStack = [];
   var length = str.length;
   var i;
   var parts = [];
-  var current_part = '';
-  var opening_index;
-  var closing_index;
+  var currentPart = '';
+  var openingIndex;
+  var closingIndex;
   for (i = 0; i < length; i++) {
-    opening_index = opening_deliminators.indexOf(str[i]);
-    closing_index = closing_deliminators.indexOf(str[i]);
-    if (is_space.test(str[i])) {
-      if (deliminator_stack.length === 0) {
-        if (current_part !== '') {
-          parts.push(current_part);
+    openingIndex = openingDeliminators.indexOf(str[i]);
+    closingIndex = closingDeliminators.indexOf(str[i]);
+    if (isSpace.test(str[i])) {
+      if (deliminatorStack.length === 0) {
+        if (currentPart !== '') {
+          parts.push(currentPart);
         }
-        current_part = '';
+        currentPart = '';
       } else {
-        current_part += str[i];
+        currentPart += str[i];
       }
     } else {
       if (str[i] === '\\') {
         i++;
-        current_part += str[i];
+        currentPart += str[i];
       } else {
-        current_part += str[i];
-        if (
-          closing_index !== -1 &&
-          closing_index === deliminator_stack[deliminator_stack.length - 1]
-        ) {
-          deliminator_stack.pop();
-        } else if (opening_index !== -1) {
-          deliminator_stack.push(opening_index);
+        currentPart += str[i];
+        if (closingIndex !== -1 && closingIndex === deliminatorStack[deliminatorStack.length - 1]) {
+          deliminatorStack.pop();
+        } else if (openingIndex !== -1) {
+          deliminatorStack.push(openingIndex);
         }
       }
     }
   }
-  if (current_part !== '') {
-    parts.push(current_part);
+  if (currentPart !== '') {
+    parts.push(currentPart);
   }
   return parts;
 };
@@ -479,11 +476,11 @@ var getParts = function (str) {
  * hand properties and the values are the values to set
  * on them
  */
-exports.shorthandParser = function parse(v, shorthand_for) {
+exports.shorthandParser = function parse(v, shorthandFor) {
   var obj = {};
   var type = exports.valueType(v);
   if (type === exports.TYPES.NULL_OR_EMPTY_STR) {
-    Object.keys(shorthand_for).forEach(function (property) {
+    Object.keys(shorthandFor).forEach(function (property) {
       obj[property] = '';
     });
     return obj;
@@ -503,24 +500,24 @@ exports.shorthandParser = function parse(v, shorthand_for) {
   var parts = getParts(v);
   var valid = true;
   parts.forEach(function (part, i) {
-    var part_valid = false;
-    Object.keys(shorthand_for).forEach(function (property) {
-      if (shorthand_for[property].isValid(part, i)) {
-        part_valid = true;
+    var partValid = false;
+    Object.keys(shorthandFor).forEach(function (property) {
+      if (shorthandFor[property].isValid(part, i)) {
+        partValid = true;
         obj[property] = part;
       }
     });
-    valid = valid && part_valid;
+    valid = valid && partValid;
   });
   if (!valid) {
     return undefined;
   }
   return obj;
 };
 
-exports.shorthandSetter = function (property, shorthand_for) {
+exports.shorthandSetter = function (property, shorthandFor) {
   return function (v) {
-    var obj = exports.shorthandParser(v, shorthand_for);
+    var obj = exports.shorthandParser(v, shorthandFor);
     if (obj === undefined) {
       return;
     }
@@ -538,7 +535,7 @@ exports.shorthandSetter = function (property, shorthand_for) {
         this._values[subprop] = obj[subprop];
       }
     }, this);
-    Object.keys(shorthand_for).forEach(function (subprop) {
+    Object.keys(shorthandFor).forEach(function (subprop) {
       if (!obj.hasOwnProperty(subprop)) {
         this.removeProperty(subprop);
         delete this._values[subprop];
@@ -549,19 +546,19 @@ exports.shorthandSetter = function (property, shorthand_for) {
     // if it already exists, then call the shorthandGetter, if it's an empty
     // string, don't set the property
     this.removeProperty(property);
-    var calculated = exports.shorthandGetter(property, shorthand_for).call(this);
+    var calculated = exports.shorthandGetter(property, shorthandFor).call(this);
     if (calculated !== '') {
       this._setProperty(property, calculated);
     }
   };
 };
 
-exports.shorthandGetter = function (property, shorthand_for) {
+exports.shorthandGetter = function (property, shorthandFor) {
   return function () {
     if (this._values[property] !== undefined) {
       return this.getPropertyValue(property);
     }
-    return Object.keys(shorthand_for)
+    return Object.keys(shorthandFor)
       .map(function (subprop) {
         return this.getPropertyValue(subprop);
       }, this)
@@ -577,12 +574,12 @@ exports.shorthandGetter = function (property, shorthand_for) {
 // if two, the first applies to the top and bottom, and the second to left and right
 // if three, the first applies to the top, the second to left and right, the third bottom
 // if four, top, right, bottom, left
-exports.implicitSetter = function (property_before, property_after, isValid, parser) {
-  property_after = property_after || '';
-  if (property_after !== '') {
-    property_after = '-' + property_after;
+exports.implicitSetter = function (propertyBefore, propertyAfter, isValid, parser) {
+  propertyAfter = propertyAfter || '';
+  if (propertyAfter !== '') {
+    propertyAfter = '-' + propertyAfter;
   }
-  var part_names = ['top', 'right', 'bottom', 'left'];
+  var partNames = ['top', 'right', 'bottom', 'left'];
 
   return function (v) {
     if (typeof v === 'number') {
@@ -608,7 +605,7 @@ exports.implicitSetter = function (property_before, property_after, isValid, par
     parts = parts.map(function (part) {
       return parser(part);
     });
-    this._setProperty(property_before + property_after, parts.join(' '));
+    this._setProperty(propertyBefore + propertyAfter, parts.join(' '));
     if (parts.length === 1) {
       parts[1] = parts[0];
     }
@@ -620,7 +617,7 @@ exports.implicitSetter = function (property_before, property_after, isValid, par
     }
 
     for (var i = 0; i < 4; i++) {
-      var property = property_before + '-' + part_names[i] + property_after;
+      var property = propertyBefore + '-' + partNames[i] + propertyAfter;
       this.removeProperty(property);
       if (parts[i] !== '') {
         this._values[property] = parts[i];
@@ -682,14 +679,14 @@ exports.subImplicitSetter = function (prefix, part, isValid, parser) {
   };
 };
 
-var camel_to_dashed = /[A-Z]/g;
-var first_segment = /^\([^-]\)-/;
-var vendor_prefixes = ['o', 'moz', 'ms', 'webkit'];
-exports.camelToDashed = function (camel_case) {
+var camelToDashed = /[A-Z]/g;
+var firstSegment = /^\([^-]\)-/;
+var vendorPrefixes = ['o', 'moz', 'ms', 'webkit'];
+exports.camelToDashed = function (camelCase) {
   var match;
-  var dashed = camel_case.replace(camel_to_dashed, '-$&').toLowerCase();
-  match = dashed.match(first_segment);
-  if (match && vendor_prefixes.indexOf(match[1]) !== -1) {
+  var dashed = camelCase.replace(camelToDashed, '-$&').toLowerCase();
+  match = dashed.match(firstSegment);
+  if (match && vendorPrefixes.indexOf(match[1]) !== -1) {
     dashed = '-' + dashed;
   }
   return dashed;