Codestyle.js 9.9 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280
  1. /* vim: set expandtab ts=4 sw=4: */
  2. /*
  3. * You may redistribute this program and/or modify it under the terms of
  4. * the GNU General Public License as published by the Free Software Foundation,
  5. * either version 3 of the License, or (at your option) any later version.
  6. *
  7. * This program is distributed in the hope that it will be useful,
  8. * but WITHOUT ANY WARRANTY; without even the implied warranty of
  9. * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
  10. * GNU General Public License for more details.
  11. *
  12. * You should have received a copy of the GNU General Public License
  13. * along with this program. If not, see <http://www.gnu.org/licenses/>.
  14. */
  15. var Fs = require('fs');
  16. var nThen = require('nthen');
  17. var Semaphore = require('./Semaphore');
  18. var Child = require('child_process');
  19. var headerLines = [
  20. '/* vim: set expandtab ts=4 sw=4: */',
  21. '/*',
  22. ' * You may redistribute this program and/or modify it under the terms of',
  23. ' * the GNU General Public License as published by the Free Software Foundation,',
  24. ' * either version 3 of the License, or (at your option) any later version.',
  25. ' *',
  26. ' * This program is distributed in the hope that it will be useful,',
  27. ' * but WITHOUT ANY WARRANTY; without even the implied warranty of',
  28. ' * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the',
  29. ' * GNU General Public License for more details.',
  30. ' *',
  31. ' * You should have received a copy of the GNU General Public License',
  32. ' * along with this program. If not, see <http://www.gnu.org/licenses/>.',
  33. ' */'
  34. ];
  35. var parseFile = function (fileName, fileContent) {
  36. var output = '';
  37. var parenthCount = 0;
  38. var functionParenthCount = 0;
  39. var expectBracket = 0;
  40. var name = fileName.replace(/^.*\//, '').replace(/\..*$/,'');
  41. var lines = fileContent.split('\n');
  42. var lineInfo = '';
  43. var ignore = false;
  44. var error = function(msg) {
  45. if (!ignore) {
  46. output += lineInfo + ' ' + msg + '\n';
  47. }
  48. };
  49. for (var lineNum = 0; lineNum < lines.length; lineNum++) {
  50. var line = lines[lineNum];
  51. // switch to 1 indexing for human readability
  52. lineInfo = fileName + ":" + (lineNum+1);
  53. ignore = false;
  54. if (lineNum < headerLines.length) {
  55. var expectedLine = headerLines[lineNum];
  56. if (line !== headerLines[lineNum]) {
  57. error("missing header\n" + expectedLine + "\n" + line);
  58. }
  59. } else if (/\.h$/.test(fileName) && lineNum < headerLines.length + 1) {
  60. if (line !== '#ifndef ' + name + "_H") {
  61. error("expected #ifndef " + name + "_H found " + line);
  62. }
  63. } else if (/\.h$/.test(fileName) && lineNum < headerLines.length + 2) {
  64. if (line !== '#define ' + name + "_H") {
  65. error("expected #define " + name + "_H found " + line);
  66. }
  67. }
  68. ignore = /CHECKFILES_IGNORE/.test(line);
  69. if (expectBracket === 1) {
  70. expectBracket = 0;
  71. if (!(/^[\s]*{/.test(line))) {
  72. error("expecting a { bracket " + line);
  73. }
  74. }
  75. // implementations.. TUNConfigurator_Linux contains TUNConfigurator_doStuff...
  76. var n = name.replace(/_.*/, '');
  77. if ((/^\w+\s.*\(/).test(line)) {
  78. if (!(/^int main\(/.test(line)
  79. || line.indexOf(' '+n) > -1
  80. || /^[ ]?static /.test(line)
  81. || /^typedef /.test(line)))
  82. {
  83. error("all globally visible functions must begin with the name of the file.");
  84. }
  85. }
  86. var matches;
  87. if (functionParenthCount === 0) {
  88. matches = /^\w+\s.*(\(.*)$/.exec(line);
  89. }
  90. if (functionParenthCount > 0 || matches) {
  91. var txt = (functionParenthCount > 0) ? line : matches[1];
  92. functionParenthCount += (txt.match(/\(/g)||[]).length;
  93. functionParenthCount -= (txt.match(/\)/g)||[]).length;
  94. if (functionParenthCount === 0) {
  95. txt = txt.substring(txt.lastIndexOf(')') + 1);
  96. if (/{/.test(txt)) {
  97. error("please put the opening bracket on the next line.");
  98. }
  99. }
  100. }
  101. if (/[\w]*int[\w]*\s+\*+\w/.test(line) || /[\w]*struct\s+[\w]+\s+\*+\w/.test(line)) {
  102. error("int* blah; means int pointer named blah, int *blah; means int names splatblah");
  103. }
  104. if (line.length > 100) {
  105. error("cjd's editor window is only 100 characters wide");
  106. }
  107. if (/\.h$/.test(fileName) && fileName.indexOf('util/platform/libc/') === -1) {
  108. // If the name is CryptoAuth_pvt.h, it's ok to make a structure called CryptoAuth
  109. var nameRe = name.replace(/_pvt$/, '').replace(/_impl$/, '');
  110. if (/^struct /.test(line) && line.indexOf('struct ' + nameRe) !== 0 && !(/\(/.test(line))) {
  111. error("all structures must begin with the name of the file.");
  112. }
  113. if (/#define /.test(line) && line.indexOf('#define ' + nameRe) === -1) {
  114. error("all defines must begin with the name of the file.");
  115. }
  116. }
  117. if (/\t/.test(line)) {
  118. error("tabs are not allowed, use 4 spaces.");
  119. }
  120. if (/\s$/.test(line)) {
  121. error("trailing whitespace.");
  122. }
  123. if (/[^A-Z](TODO|FIXME|XXX)[^\(A-Z]/.test(line)) {
  124. error("Please take responsibility for your TODO: eg: // TODO(cjd): make this work");
  125. }
  126. if (/(if|for|while)\(/.test(line)) {
  127. error("If/for/while statements must be followed by whitespace.");
  128. }
  129. matches = null;
  130. if (parenthCount === 0) {
  131. matches = /[^\w#](if|for|while) (\(.*$)/.exec(line);
  132. }
  133. if (parenthCount > 0 || matches) {
  134. var txt1 = (parenthCount > 0) ? line : matches[2];
  135. parenthCount += (txt1.match(/\(/g)||[]).length;
  136. parenthCount -= (txt1.match(/\)/g)||[]).length;
  137. if (parenthCount === 0) {
  138. txt1 = txt1.substring(txt1.lastIndexOf(')') + 1);
  139. // for (x; y; z) ; <-- ok
  140. // for (x; y; z) { <-- ok
  141. // for (x; y; z) { \ <-- ok (in preprocessor macro)
  142. // for (x; y; z) <-- ok but you better put a bracket on the next line
  143. // for (x; y; z) { j++; } <-- ok
  144. // for (x; y; z) j++; <-- BZZZZZZZZZZT
  145. if (!(/^[\s]*[;{].*$/.test(txt1)) && !(/^[\s]+{[\s]*\\$/).test(txt1)) {
  146. if (/[\s]*$/.test(txt1)) {
  147. expectBracket = 1;
  148. } else {
  149. error(parenthCount + ' ' + line);
  150. }
  151. }
  152. }
  153. }
  154. }
  155. return output;
  156. };
  157. var checkFile = module.exports.checkFile = function (file, callback) {
  158. Fs.readFile(file, function (err, ret) {
  159. if (err) { throw err; }
  160. callback(parseFile(file, ret.toString()));
  161. });
  162. };
  163. var lint = module.exports.lint = function (fileName, fileContent, callback) {
  164. var out = parseFile(fileName, fileContent);
  165. callback(out, !!out);
  166. };
  167. var checkFiles = module.exports.checkFiles = function (files, callback) {
  168. var sema = Semaphore.create(64);
  169. var errors = '';
  170. nThen(function (waitFor) {
  171. files.forEach(function (file) {
  172. sema.take(waitFor(function (returnAfter) {
  173. checkFile(file, waitFor(returnAfter(function (err) {
  174. if (err) {
  175. errors += file + '\n' + err + '\n';
  176. }
  177. })));
  178. }));
  179. });
  180. }).nThen(function (waitFor) {
  181. callback(errors);
  182. });
  183. };
  184. var checkDir = module.exports.checkDir = function (dir, runInFork, callback) {
  185. var gitIgnoreLines;
  186. if (runInFork) {
  187. var err = '';
  188. var out = '';
  189. var proc = Child.spawn(process.execPath, [__filename]);
  190. proc.stdout.on('data', function (data) { err += data.toString('utf8'); });
  191. proc.stderr.on('data', function (data) { err += data.toString('utf8'); });
  192. proc.on('close', function (ret) {
  193. out += err;
  194. var error;
  195. if (ret !== 0) { error = new Error(out); }
  196. callback(error, out);
  197. });
  198. return;
  199. }
  200. var output = '';
  201. nThen(function (waitFor) {
  202. Fs.readFile('.gitignore', waitFor(function (err, ret) {
  203. if (err) { throw err; }
  204. gitIgnoreLines = ret.toString('utf8').split('\n');
  205. }));
  206. }).nThen(function (waitFor) {
  207. var addDir = function (dir) {
  208. Fs.readdir(dir, waitFor(function (err, files) {
  209. if (err) { throw err; }
  210. files.forEach(function (file) {
  211. Fs.stat(dir + '/' + file, waitFor(function (err, stat) {
  212. if (err) { throw err; }
  213. if (file === '.git') {
  214. } else if (file === 'contrib') {
  215. } else if (file === 'dependencies') {
  216. } else if (gitIgnoreLines.indexOf(file) !== -1) {
  217. } else {
  218. if (stat.isDirectory()) {
  219. addDir(dir + '/' + file);
  220. } else if (/.*\.[ch]$/.test(file)) {
  221. checkFile(dir + '/' + file, waitFor(function (ret) {
  222. output += ret;
  223. }));
  224. }
  225. }
  226. }));
  227. });
  228. }));
  229. };
  230. addDir(dir);
  231. }).nThen(function (waitFor) {
  232. callback(output);
  233. });
  234. };
  235. if (module.parent === null) {
  236. checkDir('.', false, function(output) {
  237. if (output !== '') {
  238. console.log(output);
  239. process.exit(1);
  240. }
  241. });
  242. }