JavaScript - wrong regex for Function followed by one space vs. Function followed by no space

[tkrotoff]

See https://github.com/outsideris/popularconvention/blob/master/src/parser/js-parser.coffee#L136

onespace = /function(\s+.)*\s+\(/
nospace = /function(\s+.)*\(/

These regex check for function() vs function () without a function name, not for function foo () vs function foo() like advertised.

Either the regex are wrong or the documentation is.

Ideal would be to check for function() vs function () and also for function foo () vs function foo()

Tests:

var onespace = /function(\s+.)*\s+\(/;
'function foo () {'.match(onespace); // => null
'function foo() {'.match(onespace);  // => null
'function () {'.match(onespace);     // => ["function (", undefined]
'function() {'.match(onespace);      // => null

var nospace = /function(\s+.)*\(/;
'function foo () {'.match(nospace); // => null
'function foo() {'.match(nospace);  // => null
'function () {'.match(nospace);     // => null
'function() {'.match(nospace);      // => ["function(", undefined]