adding the files

Author: Stephen Mathieson

lib/index.js

@@ -0,0 +1,28 @@
+/*jslint node:true*/
+
+var runner = require('./runner.js');
+
+var reporters = require('./reporters/index.js');
+
+/**
+ * [cpplint description]
+ *
+ * @async
+ * @param  {Object} options
+ * @param  {Function} reporter
+ */
+function cpplint(options, reporter) {
+	'use strict';
+
+	return runner(options, reporter);
+}
+
+module.exports = cpplint;
+
+
+cpplint({
+	'files': [
+		'/Users/stephenmathieson/work/amaze/signing/amaze-cpp-signing-extension-stephen/src/signing.cc',
+		'/Users/stephenmathieson/work/amaze/signing/amaze-cpp-signing-extension-stephen/CPPSigningExtension.cc'
+	]
+}, reporters.spec);

lib/lint-error.js

@@ -0,0 +1,64 @@
+/*jslint node:true*/
+
+require('colors');
+
+var path = require('path');
+
+var extend = require('./extend.js');
+
+// [whitespace/blank_line] [3]
+var category = /\[([a-z]+)\/([a-z_]+)\] \[([1-5])\]/i;
+
+function LintError(line) {
+	'use strict';
+
+	var temp,
+		parts = line.split('  ');
+
+	this.line = line;
+
+	// hack to handle reasons like this:
+	// "Blank line at the end of a code block.  Is this needed?"
+	if (parts.length === 4) {
+		temp = [];
+		temp.push(parts[0]);
+		temp.push([parts[1], parts[2]].join('  '));
+		temp.push(parts[3]);
+		parts = temp;
+	}
+
+	temp = parts[0].split(':');
+	this.file = temp[0];
+	this.linenumber = temp[1];
+
+	this.reason = parts[1];
+
+	temp = parts[2].match(category);
+	this.category = temp[1];
+	this.sub_category = temp[2];
+	this.level = temp[3];
+}
+
+/**
+ * @return {String} this.line
+ */
+LintError.prototype.toString = function () {
+	'use strict';
+
+	return this.line;
+};
+
+/**
+ * Creates a plain Object based on properties of the LintError
+ *
+ * @return {Object}
+ */
+LintError.prototype.toObject = function () {
+	'use strict';
+
+	var obj = extend({}, this);
+
+	return this;
+};
+
+module.exports = LintError;

lib/make-args.js

@@ -0,0 +1,57 @@
+/*jslint node:true*/
+
+var path = require('path');
+
+/*jslint nomen:true*/
+var CPPLINT = path.resolve(__dirname, '../cpplint/cpplint.py');
+/*jslint nomen:false*/
+
+/**
+ * [verbosity description]
+ *
+ * @param  {String} type
+ * @return {String}
+ */
+function verbosity(level) {
+	'use strict';
+
+	return '--verbose=' + level.toString();
+}
+
+/**
+ * [counting description]
+ *
+ * @param  {String} type
+ * @return {String}
+ */
+function counting(type) {
+	'use strict';
+
+	return '--counting=' + type;
+}
+
+/**
+ * [makeArgs description]
+ *
+ * @param {Object} conf
+ * @param {Function} next
+ * @return {Array}
+ */
+function makeArgs(conf, next) {
+	'use strict';
+
+	var args = [ CPPLINT ];
+
+	args.push(verbosity(conf.verbosity));
+	args.push(counting(conf.counting));
+
+	args.push(conf.files.join(' '));
+
+	if (typeof next === 'function') {
+		return next(null, args);
+	}
+
+	return args;
+}
+
+module.exports = makeArgs;

lib/out.js

@@ -0,0 +1,26 @@
+/*jslint node:true*/
+
+/**
+ * Output the given `str` to _stdout_ or the stream specified by `options`.
+ *
+ * Options:
+ *
+ *   - `stream` defaulting to _stdout_
+ *
+ * Examples:
+ *
+ *     mymodule.write('foo')
+ *     mymodule.write('foo', { stream: process.stderr })
+ *
+ */
+function out(str, options) {
+
+	'use strict';
+
+	options = options || {};
+
+	(options.stream || process.stdout).write(str);
+
+}
+
+module.exports = out;

lib/parse-options.js

@@ -0,0 +1,56 @@
+/*jslint node:true*/
+
+var extend = require('./extend.js');
+
+/**
+ * Supported counting (`--counting=`) types
+ * @type {Array}
+ */
+var COUNTING_TYPES = [
+	'total',
+	'toplevel',
+	'detailed'
+];
+
+// verbosity requirements
+var MAX_VERBOSITY = 5;
+var MIN_VERBOSITY = 0;
+
+/**
+ * The default _cpplint_ options
+ * @type {Object}
+ */
+var defaults = {};
+defaults.verbosity = 1;
+defaults.counting = 'total';
+
+
+/**
+ * [parseOptions description]
+ *
+ * @async
+ * @param  {Object} options
+ * @param  {Function} next
+ */
+function parseOptions(options, next) {
+	'use strict';
+
+	var conf = extend(options, defaults);
+
+	if (!conf.files || !conf.files.length) {
+		next(new Error('must provide some files'));
+	}
+
+	if (conf.verbosity > MAX_VERBOSITY || conf.verbosity < MIN_VERBOSITY) {
+		next(new Error('invalid verbosity level (0-5)'));
+	}
+
+	if (COUNTING_TYPES.indexOf(conf.counting) === -1) {
+		next(new Error('invalid counting type (' + COUNTING_TYPES.join(', ') + ')'));
+	}
+
+	next(null, conf);
+}
+
+module.exports.defaults = defaults;
+module.exports.parseOptions = parseOptions;

lib/parse-output.js

@@ -0,0 +1,71 @@
+/*jslint node:true*/
+
+var LintError = require('./lint-error.js');
+
+/*jslint regexp:true*/
+var expressions = {
+	// Done processing /Users/stephenmathieson/work/amaze/signing/amaze-cpp-signing-extension-stephen/CPPSigningExtension.cc
+	'done': {
+		'is': /Done processing [\/a-z\-\_\.\d]+/i,
+		'file': / ([\/a-z\-\_\.\d]+)$/i
+	},
+	// /Users/stephenmathieson/work/amaze/signing/amaze-cpp-signing-extension-stephen/CPPSigningExtension.cc:302:  Blank line at the end of a code block.  Is this needed?  [whitespace/blank_line] [3]
+	'error': {
+		'is': /[\/a-z-_\.]+\:[\d]+\:[ ]{2}.*\[[1-5]\]/i
+	}
+};
+
+/**
+ * [parseOutput description]
+ *
+ * @param  {String} output
+ * @param  {Function} next
+ * @return {[type]}
+ */
+function parseOutput(output, next) {
+	'use strict';
+
+	var errors = [], error,
+		report = {},
+		lines = output.split('\n').filter(function (line) {
+			if (line) {
+				return true;
+			}
+
+			return false;
+		});
+
+	lines.forEach(function (line) {
+
+		var filename;
+
+		if (line.match(expressions.done.is)) {
+
+			filename = line.match(expressions.done.file)[1];
+
+			if (!report.hasOwnProperty(filename)) {
+				report[filename] = [];
+			}
+
+		}
+
+		if (line.match(expressions.error.is)) {
+			error = new LintError(line);
+			filename = error.file;
+
+			if (!report.hasOwnProperty(filename)) {
+				report[filename] = [];
+			}
+
+			report[filename].push(error);
+
+		}
+
+	});
+
+	return next(null, report);
+	//return next(null, errors);
+
+}
+
+module.exports = parseOutput;

lib/runner.js

@@ -0,0 +1,71 @@
+/*jslint node:true*/
+
+require('colors');
+
+var spawn = require('child_process').spawn;
+
+var exec = require('child_process').exec;
+
+var parseOptions = require('./parse-options.js').parseOptions;
+
+var makeArgs = require('./make-args.js');
+
+var parseOutput = require('./parse-output.js');
+
+function runner(options, next) {
+	'use strict';
+
+	var cpplint;
+
+	parseOptions(options, function (err, config) {
+
+		if (err) {
+			return next(err);
+		}
+
+		var args = makeArgs(config);
+
+		cpplint = exec(args.join(' ').trim(), function (err, stdout, stderr) {
+
+			return parseOutput(stderr, next);
+
+		});
+
+	});
+
+}
+
+module.exports = runner;
+
+/*
+
+runner({
+	'files': [
+		'/Users/stephenmathieson/work/amaze/signing/amaze-cpp-signing-extension-stephen/src/signing.cc',
+		'/Users/stephenmathieson/work/amaze/signing/amaze-cpp-signing-extension-stephen/CPPSigningExtension.cc'
+	]
+}, function (err, report) {
+	'use strict';
+
+	var files = Object.keys(report);
+
+	files.forEach(function (filename) {
+		var errors = report[filename];
+
+		if (!errors.length) {
+			console.log('PASS\t'.green + filename);
+
+		} else {
+			console.log('FAIL\t'.red + filename + ' (' + errors.length.toString() + ')');
+
+			errors.forEach(function (error) {
+				console.log(error.toString());
+			});
+
+		}
+
+	});
+
+});
+
+*/