Added extensions option for non-cli uses

Author: Nathan-Smith

lib/make-args.js

@@ -29,10 +29,14 @@ function counting(type) {
 /**
  * Get the cpplint `extensions` argument
  *
- * @param  {String} type
+ * @param  {String|Array} exts
  * @return {String}
  */
 function extensions(exts) {
+  var extsType = {}.toString.call(exts);
+  if (extsType === '[object Array]') {
+    exts = exts.join(',');
+  }
   return '--extensions=' + exts;
 }
 

readme.md

@@ -21,6 +21,7 @@ a count is provided for each category like `build/class`.
 - **verbose** The verbosity level; defaults to *1*.  A number *0-5* to restrict
 errors to certain verbosity levels.
 - **filters** Enable/disable filtering for specific errors.
+- **extensions** List of file extensions to lint.
 
 
 A list of files is also expected.
@@ -43,9 +44,14 @@ bin/cpplint --verbose 3 --counting detailed file2 file3
 Using the `plain-text` reporter, ignoring *build/deprecated* errors and linting *file1*.
 
 ```bash
-bin/cpplint --filter build-deprecated --reporter plain-text
+bin/cpplint --filter build-deprecated --reporter plain-text file1
 ```
 
+Using the `cc` and `hpp` extensions and linting *source1.cc* and *source1.hpp*.
+
+```bash
+bin/cpplint --extensions cc,hpp source1.cc source1.hpp
+```
 
 ### JavaScript usage
 
@@ -76,7 +82,12 @@ var options = {
       'braces': false,
       'include_alpha': true
     }
-  }
+  },
+  // This could be an array of strings or a comma separated string
+  extensions: [
+    'cc',
+    'hpp'
+  ]
 };
 
 cpplint(options, function (err, report) {
@@ -103,7 +114,12 @@ grunt.initConfig({
       'braces': false,
       'include_alpha': true
     }
-  }
+  },
+  // This could be an array of strings or a comma separated string
+  extensions: [
+    'cc',
+    'hpp'
+  ]
 });
 ```
 

tasks/cpplint.js

@@ -35,6 +35,8 @@ module.exports = function (grunt) {
 
     options.filters = extend(filters.defaults, gruntFilters, true);
 
+    options.extensions = conf('extensions');
+
     options.files = grunt.file.expand(conf('files'));
     options.verbosity = conf('verbosity') || 1;
     options.counting = conf('counting') || 'toplevel';

test/make-args.js

@@ -88,7 +88,12 @@ suite.addBatch({
           'subcat5': false,
           'subcat6': true
         }
-      }
+      },
+      'extensions': [
+        'cpp',
+        'cc',
+        'xx'
+      ]
     }),
     'should set correct verbosity level': function (err, args) {
       assert.includes(args, '--verbose=1');
@@ -101,6 +106,9 @@ suite.addBatch({
     },
     'should pass the correct filters': function (err, args) {
       assert.includes(args, '--filter=+category1/subcat1,-category1/subcat2,+category1/subcat3,+category2/subcat1,-category2/subcat2,+category2/subcat3,+category2/subcat4,-category2/subcat5,+category2/subcat6');
+    },
+    'should pass the correct extensions': function (err, args) {
+      assert.includes(args, '--extensions=cpp,cc,xx');
     }
 
   },