Merge pull request #363 from PHPCSStandards/feature/various-sniff-update-skip-to-end-code

jrfnl · web-flow · commit 0eee69132079 · 2024-03-01T10:41:01.000+01:00
Various sniffs: simplify skipping the rest of the file + update docs about skipping rest of file
diff --git a/src/Sniffs/AbstractScopeSniff.php b/src/Sniffs/AbstractScopeSniff.php
@@ -123,7 +123,7 @@ final public function register()
      *
      * @return void|int Optionally returns a stack pointer. The sniff will not be
      *                  called again on the current file until the returned stack
-     *                  pointer is reached. Return ($phpcsFile->numTokens + 1) to skip
+     *                  pointer is reached. Return `$phpcsFile->numTokens` to skip
      *                  the rest of the file.
      * @see    processTokenWithinScope()
      */
@@ -164,7 +164,7 @@ final public function process(File $phpcsFile, $stackPtr)
      *
      * @return void|int Optionally returns a stack pointer. The sniff will not be
      *                  called again on the current file until the returned stack
-     *                  pointer is reached. Return ($phpcsFile->numTokens + 1) to skip
+     *                  pointer is reached. Return `$phpcsFile->numTokens` to skip
      *                  the rest of the file.
      */
     abstract protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScope);
@@ -180,7 +180,7 @@ abstract protected function processTokenWithinScope(File $phpcsFile, $stackPtr,
      *
      * @return void|int Optionally returns a stack pointer. The sniff will not be
      *                  called again on the current file until the returned stack
-     *                  pointer is reached. Return (count($tokens) + 1) to skip
+     *                  pointer is reached. Return `$phpcsFile->numTokens` to skip
      *                  the rest of the file.
      */
     abstract protected function processTokenOutsideScope(File $phpcsFile, $stackPtr);
diff --git a/src/Sniffs/AbstractVariableSniff.php b/src/Sniffs/AbstractVariableSniff.php
@@ -72,7 +72,7 @@ public function __construct()
      *
      * @return void|int Optionally returns a stack pointer. The sniff will not be
      *                  called again on the current file until the returned stack
-     *                  pointer is reached. Return ($phpcsFile->numTokens + 1) to skip
+     *                  pointer is reached. Return `$phpcsFile->numTokens` to skip
      *                  the rest of the file.
      */
     final protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScope)
@@ -156,7 +156,7 @@ final protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $cu
      *
      * @return void|int Optionally returns a stack pointer. The sniff will not be
      *                  called again on the current file until the returned stack
-     *                  pointer is reached. Return ($phpcsFile->numTokens + 1) to skip
+     *                  pointer is reached. Return `$phpcsFile->numTokens` to skip
      *                  the rest of the file.
      */
     final protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
@@ -187,7 +187,7 @@ final protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
      *
      * @return void|int Optionally returns a stack pointer. The sniff will not be
      *                  called again on the current file until the returned stack
-     *                  pointer is reached. Return ($phpcsFile->numTokens + 1) to skip
+     *                  pointer is reached. Return `$phpcsFile->numTokens` to skip
      *                  the rest of the file.
      */
     abstract protected function processMemberVar(File $phpcsFile, $stackPtr);
@@ -202,7 +202,7 @@ abstract protected function processMemberVar(File $phpcsFile, $stackPtr);
      *
      * @return void|int Optionally returns a stack pointer. The sniff will not be
      *                  called again on the current file until the returned stack
-     *                  pointer is reached. Return ($phpcsFile->numTokens + 1) to skip
+     *                  pointer is reached. Return `$phpcsFile->numTokens` to skip
      *                  the rest of the file.
      */
     abstract protected function processVariable(File $phpcsFile, $stackPtr);
@@ -221,7 +221,7 @@ abstract protected function processVariable(File $phpcsFile, $stackPtr);
      *
      * @return void|int Optionally returns a stack pointer. The sniff will not be
      *                  called again on the current file until the returned stack
-     *                  pointer is reached. Return ($phpcsFile->numTokens + 1) to skip
+     *                  pointer is reached. Return `$phpcsFile->numTokens` to skip
      *                  the rest of the file.
      */
     abstract protected function processVariableInString(File $phpcsFile, $stackPtr);
diff --git a/src/Sniffs/Sniff.php b/src/Sniffs/Sniff.php
@@ -71,7 +71,7 @@ public function register();
      *
      * @return void|int Optionally returns a stack pointer. The sniff will not be
      *                  called again on the current file until the returned stack
-     *                  pointer is reached. Return (count($tokens) + 1) to skip
+     *                  pointer is reached. Return `$phpcsFile->numTokens` to skip
      *                  the rest of the file.
      */
     public function process(File $phpcsFile, $stackPtr);
diff --git a/src/Standards/Generic/Sniffs/ControlStructures/InlineControlStructureSniff.php b/src/Standards/Generic/Sniffs/ControlStructures/InlineControlStructureSniff.php
@@ -150,7 +150,7 @@ public function process(File $phpcsFile, $stackPtr)
             // tag in short open tags and scan run with short_open_tag=Off.
             // Bow out completely as any further detection will be unreliable
             // and create incorrect fixes or cause fixer conflicts.
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         unset($nextNonEmpty, $start);
diff --git a/src/Standards/Generic/Sniffs/Debug/CSSLintSniff.php b/src/Standards/Generic/Sniffs/Debug/CSSLintSniff.php
@@ -52,7 +52,7 @@ public function process(File $phpcsFile, $stackPtr)
     {
         $csslintPath = Config::getExecutablePath('csslint');
         if ($csslintPath === null) {
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $fileName = $phpcsFile->getFilename();
@@ -61,7 +61,7 @@ public function process(File $phpcsFile, $stackPtr)
         exec($cmd, $output, $retval);
 
         if (is_array($output) === false) {
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $count = count($output);
@@ -90,7 +90,7 @@ public function process(File $phpcsFile, $stackPtr)
         }//end for
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/Debug/ClosureLinterSniff.php b/src/Standards/Generic/Sniffs/Debug/ClosureLinterSniff.php
@@ -69,7 +69,7 @@ public function process(File $phpcsFile, $stackPtr)
     {
         $lintPath = Config::getExecutablePath('gjslint');
         if ($lintPath === null) {
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $fileName = $phpcsFile->getFilename();
@@ -79,7 +79,7 @@ public function process(File $phpcsFile, $stackPtr)
         exec($cmd, $output, $retval);
 
         if (is_array($output) === false) {
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         foreach ($output as $finding) {
@@ -111,7 +111,7 @@ public function process(File $phpcsFile, $stackPtr)
         }//end foreach
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/Debug/ESLintSniff.php b/src/Standards/Generic/Sniffs/Debug/ESLintSniff.php
@@ -60,7 +60,7 @@ public function process(File $phpcsFile, $stackPtr)
     {
         $eslintPath = Config::getExecutablePath('eslint');
         if ($eslintPath === null) {
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $filename = $phpcsFile->getFilename();
@@ -86,13 +86,13 @@ public function process(File $phpcsFile, $stackPtr)
 
         if ($code <= 0) {
             // No errors, continue.
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $data = json_decode(implode("\n", $stdout));
         if (json_last_error() !== JSON_ERROR_NONE) {
             // Ignore any errors.
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         // Data is a list of files, but we only pass a single one.
@@ -107,7 +107,7 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/Debug/JSHintSniff.php b/src/Standards/Generic/Sniffs/Debug/JSHintSniff.php
@@ -55,7 +55,7 @@ public function process(File $phpcsFile, $stackPtr)
         $rhinoPath  = Config::getExecutablePath('rhino');
         $jshintPath = Config::getExecutablePath('jshint');
         if ($jshintPath === null) {
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $fileName   = $phpcsFile->getFilename();
@@ -89,7 +89,7 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/Files/EndFileNewlineSniff.php b/src/Standards/Generic/Sniffs/Files/EndFileNewlineSniff.php
@@ -76,7 +76,7 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/Files/EndFileNoNewlineSniff.php b/src/Standards/Generic/Sniffs/Files/EndFileNoNewlineSniff.php
@@ -83,7 +83,7 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/Files/ExecutableFileSniff.php b/src/Standards/Generic/Sniffs/Files/ExecutableFileSniff.php
@@ -54,7 +54,7 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/Files/LineEndingsSniff.php b/src/Standards/Generic/Sniffs/Files/LineEndingsSniff.php
@@ -68,7 +68,7 @@ public function process(File $phpcsFile, $stackPtr)
 
         if ($found === $this->eolChar) {
             // Ignore the rest of the file.
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         // Check for single line files without an EOL. This is a very special
@@ -79,7 +79,7 @@ public function process(File $phpcsFile, $stackPtr)
             if ($tokens[$lastToken]['line'] === 1
                 && $tokens[$lastToken]['content'] !== "\n"
             ) {
-                return ($phpcsFile->numTokens + 1);
+                return $phpcsFile->numTokens;
             }
         }
 
@@ -140,7 +140,7 @@ public function process(File $phpcsFile, $stackPtr)
         }//end if
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/Files/LineLengthSniff.php b/src/Standards/Generic/Sniffs/Files/LineLengthSniff.php
@@ -80,7 +80,7 @@ public function process(File $phpcsFile, $stackPtr)
         $this->checkLineLength($phpcsFile, $tokens, $i);
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/Files/LowercasedFilenameSniff.php b/src/Standards/Generic/Sniffs/Files/LowercasedFilenameSniff.php
@@ -44,7 +44,7 @@ public function process(File $phpcsFile, $stackPtr)
     {
         $filename = $phpcsFile->getFilename();
         if ($filename === 'STDIN') {
-            return  ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $filename          = basename($filename);
@@ -62,7 +62,7 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/PHP/SyntaxSniff.php b/src/Standards/Generic/Sniffs/PHP/SyntaxSniff.php
@@ -67,7 +67,7 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/VersionControl/GitMergeConflictSniff.php b/src/Standards/Generic/Sniffs/VersionControl/GitMergeConflictSniff.php
@@ -220,7 +220,7 @@ public function process(File $phpcsFile, $stackPtr)
         }//end for
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/VersionControl/SubversionPropertiesSniff.php b/src/Standards/Generic/Sniffs/VersionControl/SubversionPropertiesSniff.php
@@ -58,7 +58,7 @@ public function process(File $phpcsFile, $stackPtr)
         $properties = $this->getProperties($path);
         if ($properties === null) {
             // Not under version control.
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $allProperties = ($properties + $this->properties);
@@ -101,7 +101,7 @@ public function process(File $phpcsFile, $stackPtr)
         }//end foreach
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/WhiteSpace/DisallowSpaceIndentSniff.php b/src/Standards/Generic/Sniffs/WhiteSpace/DisallowSpaceIndentSniff.php
@@ -212,7 +212,7 @@ public function process(File $phpcsFile, $stackPtr)
         }//end for
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/Generic/Sniffs/WhiteSpace/DisallowTabIndentSniff.php b/src/Standards/Generic/Sniffs/WhiteSpace/DisallowTabIndentSniff.php
@@ -184,7 +184,7 @@ public function process(File $phpcsFile, $stackPtr)
         }//end for
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/PEAR/Sniffs/Commenting/FileCommentSniff.php b/src/Standards/PEAR/Sniffs/Commenting/FileCommentSniff.php
@@ -122,18 +122,18 @@ public function process(File $phpcsFile, $stackPtr)
 
         if ($tokens[$commentStart]['code'] === T_CLOSE_TAG) {
             // We are only interested if this is the first open tag.
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         } else if ($tokens[$commentStart]['code'] === T_COMMENT) {
             $error = 'You must use "/**" style comments for a file comment';
             $phpcsFile->addError($error, $errorToken, 'WrongStyle');
             $phpcsFile->recordMetric($stackPtr, 'File has doc comment', 'yes');
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         } else if ($commentStart === false
             || $tokens[$commentStart]['code'] !== T_DOC_COMMENT_OPEN_TAG
         ) {
             $phpcsFile->addError('Missing file doc comment', $errorToken, 'Missing');
             $phpcsFile->recordMetric($stackPtr, 'File has doc comment', 'no');
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $commentEnd = $tokens[$commentStart]['comment_closer'];
@@ -178,7 +178,7 @@ public function process(File $phpcsFile, $stackPtr)
         if (in_array($tokens[$nextToken]['code'], $ignore, true) === true) {
             $phpcsFile->addError('Missing file doc comment', $stackPtr, 'Missing');
             $phpcsFile->recordMetric($stackPtr, 'File has doc comment', 'no');
-            return ($phpcsFile->numTokens + 1);
+            return $phpcsFile->numTokens;
         }
 
         $phpcsFile->recordMetric($stackPtr, 'File has doc comment', 'yes');
@@ -205,7 +205,7 @@ public function process(File $phpcsFile, $stackPtr)
         $this->processTags($phpcsFile, $stackPtr, $commentStart);
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/PSR1/Sniffs/Files/SideEffectsSniff.php b/src/Standards/PSR1/Sniffs/Files/SideEffectsSniff.php
@@ -56,7 +56,7 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
+        return $phpcsFile->numTokens;
 
     }//end process()
 
diff --git a/src/Standards/PSR2/Sniffs/Files/EndFileNewlineSniff.php b/src/Standards/PSR2/Sniffs/Files/EndFileNewlineSniff.php
diff --git a/src/Standards/Squiz/Sniffs/Commenting/FileCommentSniff.php b/src/Standards/Squiz/Sniffs/Commenting/FileCommentSniff.php
diff --git a/src/Standards/Squiz/Sniffs/Debug/JSLintSniff.php b/src/Standards/Squiz/Sniffs/Debug/JSLintSniff.php
diff --git a/src/Standards/Squiz/Sniffs/Debug/JavaScriptLintSniff.php b/src/Standards/Squiz/Sniffs/Debug/JavaScriptLintSniff.php
diff --git a/src/Standards/Squiz/Sniffs/Files/FileExtensionSniff.php b/src/Standards/Squiz/Sniffs/Files/FileExtensionSniff.php
diff --git a/src/Standards/Zend/Sniffs/Debug/CodeAnalyzerSniff.php b/src/Standards/Zend/Sniffs/Debug/CodeAnalyzerSniff.php
diff --git a/src/Standards/Zend/Sniffs/Files/ClosingTagSniff.php b/src/Standards/Zend/Sniffs/Files/ClosingTagSniff.php

Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,7 @@ public function register();`
`71`	`71`	`*`
`72`	`72`	`* @return void\|int Optionally returns a stack pointer. The sniff will not be`
`73`	`73`	`* called again on the current file until the returned stack`
`74`		`- * pointer is reached. Return (count($tokens) + 1) to skip`
	`74`	+ * pointer is reached. Return `$phpcsFile->numTokens` to skip
`75`	`75`	`* the rest of the file.`
`76`	`76`	`*/`
`77`	`77`	`public function process(File $phpcsFile, $stackPtr);`
Original file line number	Diff line number	Diff line change
`@@ -150,7 +150,7 @@ public function process(File $phpcsFile, $stackPtr)`
`150`	`150`	`// tag in short open tags and scan run with short_open_tag=Off.`
`151`	`151`	`// Bow out completely as any further detection will be unreliable`
`152`	`152`	`// and create incorrect fixes or cause fixer conflicts.`
`153`		`- return ($phpcsFile->numTokens + 1);`
	`153`	`+ return $phpcsFile->numTokens;`
`154`	`154`	`}`
`155`	`155`
`156`	`156`	`unset($nextNonEmpty, $start);`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public function process(File $phpcsFile, $stackPtr)`
`52`	`52`	`{`
`53`	`53`	`$csslintPath = Config::getExecutablePath('csslint');`
`54`	`54`	`if ($csslintPath === null) {`
`55`		`- return ($phpcsFile->numTokens + 1);`
	`55`	`+ return $phpcsFile->numTokens;`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`$fileName = $phpcsFile->getFilename();`
`@@ -61,7 +61,7 @@ public function process(File $phpcsFile, $stackPtr)`
`61`	`61`	`exec($cmd, $output, $retval);`
`62`	`62`
`63`	`63`	`if (is_array($output) === false) {`
`64`		`- return ($phpcsFile->numTokens + 1);`
	`64`	`+ return $phpcsFile->numTokens;`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`$count = count($output);`
`@@ -90,7 +90,7 @@ public function process(File $phpcsFile, $stackPtr)`
`90`	`90`	`}//end for`
`91`	`91`
`92`	`92`	`// Ignore the rest of the file.`
`93`		`- return ($phpcsFile->numTokens + 1);`
	`93`	`+ return $phpcsFile->numTokens;`
`94`	`94`
`95`	`95`	`}//end process()`
`96`	`96`
Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ public function process(File $phpcsFile, $stackPtr)`
`69`	`69`	`{`
`70`	`70`	`$lintPath = Config::getExecutablePath('gjslint');`
`71`	`71`	`if ($lintPath === null) {`
`72`		`- return ($phpcsFile->numTokens + 1);`
	`72`	`+ return $phpcsFile->numTokens;`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`$fileName = $phpcsFile->getFilename();`
`@@ -79,7 +79,7 @@ public function process(File $phpcsFile, $stackPtr)`
`79`	`79`	`exec($cmd, $output, $retval);`
`80`	`80`
`81`	`81`	`if (is_array($output) === false) {`
`82`		`- return ($phpcsFile->numTokens + 1);`
	`82`	`+ return $phpcsFile->numTokens;`
`83`	`83`	`}`
`84`	`84`
`85`	`85`	`foreach ($output as $finding) {`
`@@ -111,7 +111,7 @@ public function process(File $phpcsFile, $stackPtr)`
`111`	`111`	`}//end foreach`
`112`	`112`
`113`	`113`	`// Ignore the rest of the file.`
`114`		`- return ($phpcsFile->numTokens + 1);`
	`114`	`+ return $phpcsFile->numTokens;`
`115`	`115`
`116`	`116`	`}//end process()`
`117`	`117`
Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ public function process(File $phpcsFile, $stackPtr)`
`60`	`60`	`{`
`61`	`61`	`$eslintPath = Config::getExecutablePath('eslint');`
`62`	`62`	`if ($eslintPath === null) {`
`63`		`- return ($phpcsFile->numTokens + 1);`
	`63`	`+ return $phpcsFile->numTokens;`
`64`	`64`	`}`
`65`	`65`
`66`	`66`	`$filename = $phpcsFile->getFilename();`
`@@ -86,13 +86,13 @@ public function process(File $phpcsFile, $stackPtr)`
`86`	`86`
`87`	`87`	`if ($code <= 0) {`
`88`	`88`	`// No errors, continue.`
`89`		`- return ($phpcsFile->numTokens + 1);`
	`89`	`+ return $phpcsFile->numTokens;`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`$data = json_decode(implode("\n", $stdout));`
`93`	`93`	`if (json_last_error() !== JSON_ERROR_NONE) {`
`94`	`94`	`// Ignore any errors.`
`95`		`- return ($phpcsFile->numTokens + 1);`
	`95`	`+ return $phpcsFile->numTokens;`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`// Data is a list of files, but we only pass a single one.`
`@@ -107,7 +107,7 @@ public function process(File $phpcsFile, $stackPtr)`
`107`	`107`	`}`
`108`	`108`
`109`	`109`	`// Ignore the rest of the file.`
`110`		`- return ($phpcsFile->numTokens + 1);`
	`110`	`+ return $phpcsFile->numTokens;`
`111`	`111`
`112`	`112`	`}//end process()`
`113`	`113`
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public function process(File $phpcsFile, $stackPtr)`
`55`	`55`	`$rhinoPath = Config::getExecutablePath('rhino');`
`56`	`56`	`$jshintPath = Config::getExecutablePath('jshint');`
`57`	`57`	`if ($jshintPath === null) {`
`58`		`- return ($phpcsFile->numTokens + 1);`
	`58`	`+ return $phpcsFile->numTokens;`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`$fileName = $phpcsFile->getFilename();`
`@@ -89,7 +89,7 @@ public function process(File $phpcsFile, $stackPtr)`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`// Ignore the rest of the file.`
`92`		`- return ($phpcsFile->numTokens + 1);`
	`92`	`+ return $phpcsFile->numTokens;`
`93`	`93`
`94`	`94`	`}//end process()`
`95`	`95`
Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ public function process(File $phpcsFile, $stackPtr)`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`// Ignore the rest of the file.`
`79`		`- return ($phpcsFile->numTokens + 1);`
	`79`	`+ return $phpcsFile->numTokens;`
`80`	`80`
`81`	`81`	`}//end process()`
`82`	`82`
Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ public function process(File $phpcsFile, $stackPtr)`
`83`	`83`	`}`
`84`	`84`
`85`	`85`	`// Ignore the rest of the file.`
`86`		`- return ($phpcsFile->numTokens + 1);`
	`86`	`+ return $phpcsFile->numTokens;`
`87`	`87`
`88`	`88`	`}//end process()`
`89`	`89`