Fix: disallowCommaBeforeLineBreak: fix for function params

Fixes #1746

Author: hzoo

lib/rules/disallow-comma-before-line-break.js

@@ -87,6 +87,12 @@ module.exports.prototype = {
                 return true;
             }
 
+            // exception for function params
+            if (node.params &&
+                node.params[0].loc.start.line === node.params[node.params.length - 1].loc.end.line) {
+                return true;
+            }
+
             // See #1841
             if (!exceptFunction || !node.properties) {
                 return false;

test/specs/rules/disallow-comma-before-line-break.js

@@ -64,6 +64,10 @@ describe('rules/disallow-comma-before-line-break', function() {
         expect(checker.checkString('var a = {a:1, c:3};')).to.have.no.errors();
     });
 
+    it('should not report comma placement in function declaration #1746', function() {
+        expect(checker.checkString('function a(b, c) {\n  console.log(1)\n}')).to.have.no.errors();
+    });
+
     describe('options as object', function() {
         describe('allExcept as option', function() {
             describe('with value `function`', function() {