Remove cch parameter for CountNewlines in Scan.cpp

As commented by sethbrenith, all callers do not pass cch and use default -1, so
there's no need to keep cch.

Author: xiaoyinl

lib/Parser/Scan.cpp

@@ -10,19 +10,18 @@
 *  a given character can be part of an identifier, and so on.
 */
 
-int CountNewlines(LPCOLESTR psz, int cch)
+int CountNewlines(LPCOLESTR psz)
 {
     int cln = 0;
 
-    while (0 != *psz && 0 != cch--)
+    while (0 != *psz)
     {
         switch (*psz++)
         {
         case _u('\xD'):
-            if (cch != 0 && *psz == _u('\xA'))
+            if (*psz == _u('\xA'))
             {
                 ++psz;
-                --cch;
             }
             // fall-through
         case _u('\xA'):

lib/Parser/Scan.h

@@ -12,7 +12,7 @@ namespace Js
 #include "Windows.Globalization.h"
 #endif
 
-int CountNewlines(LPCOLESTR psz, int cch = -1);
+int CountNewlines(LPCOLESTR psz);
 
 class Parser;
 struct ParseContext;