Deal with multi-line function calls with nothing on the first line

Previously the regex was checking for the first parameter being on the
first line, but this isn't required for the function call to be wrong.

Added the check for each of the lines if there is no nested function
call.

Author: thk123

regression/cpp-linter/multi-line-function-call1/main.cpp

@@ -75,5 +75,14 @@ static void fun()
   var->fun(
       x,
       y);
+
+  // Incorrect
+  fun(
+    x, y);
+
+  // Incorrect
+  fun(
+    x, y,
+    z);
 }
 

regression/cpp-linter/multi-line-function-call1/test.desc

@@ -9,7 +9,9 @@ main.cpp
 ^main\.cpp:62:  Indent of wrapped parenthesized expression or parameter or argument list should be 2  \[whitespace/indent\] \[4\]
 ^main\.cpp:71:  If parameters or arguments require a line break, each parameter should be put on its own line\.  \[whitespace/indent\] \[4\]
 ^main\.cpp:76:  Indent of wrapped parenthesized expression or parameter or argument list should be 2  \[whitespace/indent\] \[4\]
-^Total errors found: 8$
+^main\.cpp:81:  If parameters or arguments require a line break, the closing bracket should be on the same line as the final parameter  \[whitespace/indent\] \[4\]
+^main\.cpp:85:  If parameters or arguments require a line break, each parameter should be put on its own line.  \[whitespace/indent\] \[4\]
+^Total errors found: 10$
 ^EXIT=1$
 ^SIGNAL=0$
 --

scripts/cpplint.py

@@ -4696,7 +4696,7 @@ def CheckStyle(filename, clean_lines, linenum, file_extension, nesting_state,
   #  - ignore the line if it is a for/if etc since rules are different
 
   # Look for an opening bracket that doesn't have a semi-colon on the same line
-  bracket_search = Search(r'(?P<bracket>\()[^;]*,[^;]*$', elided_line)
+  bracket_search = Search(r'(?P<bracket>\()[^;]*$', elided_line)
 
   # Exclude the check if any of these keywords are present
   # They could trip us up as they have different formatting rules to functions
@@ -4737,6 +4737,10 @@ def CheckStyle(filename, clean_lines, linenum, file_extension, nesting_state,
                     'If parameters or arguments require a line break, each parameter should be put on its own line.')
             # Skip to the end of the bracket
             start_linenum = nested_close_linenum
+          else:
+            if(not Match('^\s*[^,]+,$', arg_line)):
+              error(filename, start_linenum, 'whitespace/indent', 4,
+                    'If parameters or arguments require a line break, each parameter should be put on its own line.')
 
           start_linenum+=1
         # For the final line we also need to check one parameter on it