Merge pull request #71 from anton-rudeshko/welcome-fb

Small changes, typos and issues.

Author: mishanga

.gitignore

@@ -1 +1,4 @@
 node_modules
+
+.idea
+*.iml

lib/cli.js

@@ -1,5 +1,5 @@
 /**
- * Command line implementation for CSScomb
+ * Command line implementation for CSSComb
  *
  * Usage example:
  * ./node_modules/.bin/csscomb file1 [dir1 [fileN [dirN]]]
@@ -15,19 +15,22 @@ program
     .option('-c, --config [path]', 'configuration file path')
     .parse(process.argv);
 
-var comb = new Comb();
+if (!program.args.length) {
+    console.log('No input paths specified');
+    program.help();
+}
+
 var configPath = program.config || (process.cwd() + '/.csscomb.json');
 
 if (fs.existsSync(configPath)) {
+    var comb = new Comb();
     comb.configure(require(configPath));
-    if (program.args.length > 0) {
-        vow.all(program.args.map(function(path) {
-            return comb.processPath(path);
-        })).fail(function(e) {
-            console.log('stack: ', e.stack);
-            process.exit(1);
-        });
-    }
+    vow.all(program.args.map(function(path) {
+        return comb.processPath(path);
+    })).fail(function(e) {
+        console.log('stack: ', e.stack);
+        process.exit(1);
+    });
 } else {
     console.log('Configuration file ' + configPath + ' was not found.');
     process.exit(1);

lib/csscomb.js

@@ -6,10 +6,10 @@ var vfs = require('vow-fs');
 /**
  * Starts Code Style processing process.
  *
+ * @constructor
  * @name Comb
  */
 var Comb = function() {
-    this._handlers;
     this._options = [
         'always-semicolon',
         'color-case',
@@ -24,7 +24,6 @@ var Comb = function() {
         'unitless-zero',
         'sort-order'
     ];
-    this._config = {};
     this._exclude = null;
 };
 
@@ -37,17 +36,18 @@ Comb.prototype = {
      * @param {Object} config
      */
     configure: function(config) {
-        var _this = this;
         this._handlers = [];
         this._options.forEach(function(option) {
-            if (config[option] !== undefined) {
-                try {
-                    var handler = require('./options/' + option).setValue(config[option]);
-                    handler && _this._handlers.push(handler);
-                } catch (e) {
-                }
+            if (typeof config[option] === 'undefined') return;
+
+            try {
+                var handler = require('./options/' + option).setValue(config[option]);
+                handler && this._handlers.push(handler);
+            } catch (e) {
+                console.warn('Error loading "%s" handler: %s', option, e.message);
             }
-        });
+        }, this);
+
         this._exclude = (config.exclude || []).map(function(pattern) {
             return new minimatch.Minimatch(pattern);
         });
@@ -151,7 +151,7 @@ Comb.prototype = {
     },
 
     /**
-     * Processs directory or file.
+     * Process directory or file.
      *
      * @param {String} path
      */
@@ -177,7 +177,7 @@ Comb.prototype = {
     },
 
     /**
-     * Returns true if specified path is not in exluded list.
+     * Returns true if specified path is not in excluded list.
      *
      * @returns {Boolean}
      */