Merge branch 'master' into 4.x

Author: jrfnl

src/Files/File.php

@@ -159,7 +159,7 @@ class File
      *
      * @var array<string, int>
      */
-    private $firstRunCounts;
+    private $firstRunCounts = [];
 
     /**
      * The current total number of errors that can be fixed.
@@ -542,7 +542,7 @@ public function process()
             StatusWriter::write('*** END SNIFF PROCESSING REPORT ***', 1);
         }
 
-        if (isset($this->firstRunCounts) === false) {
+        if (empty($this->firstRunCounts) === true) {
             $this->firstRunCounts = [
                 'error'          => $this->errorCount,
                 'warning'        => $this->warningCount,

src/Reports/Junit.php

@@ -114,7 +114,7 @@ public function generate(
         $tests   = 0;
         $matches = [];
         preg_match_all('/tests="([0-9]+)"/', $cachedData, $matches);
-        if (isset($matches[1]) === true) {
+        if (empty($matches[1]) === false) {
             foreach ($matches[1] as $match) {
                 $tests += $match;
             }

src/Standards/Generic/Sniffs/ControlStructures/DisallowYodaConditionsSniff.php

@@ -109,9 +109,7 @@ public function process(File $phpcsFile, int $stackPtr)
                 if ($prev === false) {
                     return;
                 }
-            } elseif ($tokens[$beforeOpeningParenthesisIndex]['code'] === T_ARRAY
-                && $this->isArrayStatic($phpcsFile, $beforeOpeningParenthesisIndex) === false
-            ) {
+            } elseif ($this->isArrayStatic($phpcsFile, $beforeOpeningParenthesisIndex) === false) {
                 return;
             }
         }

src/Standards/PEAR/Sniffs/Formatting/MultiLineAssignmentSniff.php

@@ -61,7 +61,7 @@ public function process(File $phpcsFile, int $stackPtr)
         }
 
         // Make sure it is the first thing on the line, otherwise we ignore it.
-        $prev = $phpcsFile->findPrevious(T_WHITESPACE, ($stackPtr - 1), false, true);
+        $prev = $phpcsFile->findPrevious(T_WHITESPACE, ($stackPtr - 1), null, true);
         if ($prev === false) {
             // Bad assignment.
             return;

src/Standards/PEAR/Sniffs/Functions/FunctionDeclarationSniff.php

@@ -55,8 +55,6 @@ public function process(File $phpcsFile, int $stackPtr)
 
         if (isset($tokens[$stackPtr]['parenthesis_opener']) === false
             || isset($tokens[$stackPtr]['parenthesis_closer']) === false
-            || $tokens[$stackPtr]['parenthesis_opener'] === null
-            || $tokens[$stackPtr]['parenthesis_closer'] === null
         ) {
             return;
         }

src/Standards/PSR12/Sniffs/Functions/ReturnTypeDeclarationSniff.php

@@ -46,8 +46,6 @@ public function process(File $phpcsFile, int $stackPtr)
 
         if (isset($tokens[$stackPtr]['parenthesis_opener']) === false
             || isset($tokens[$stackPtr]['parenthesis_closer']) === false
-            || $tokens[$stackPtr]['parenthesis_opener'] === null
-            || $tokens[$stackPtr]['parenthesis_closer'] === null
         ) {
             return;
         }

src/Standards/Squiz/Sniffs/Commenting/VariableCommentSniff.php

@@ -77,9 +77,8 @@ public function processMemberVar(File $phpcsFile, int $stackPtr)
             break;
         }
 
-        if ($commentEnd === false
-            || ($tokens[$commentEnd]['code'] !== T_DOC_COMMENT_CLOSE_TAG
-            && $tokens[$commentEnd]['code'] !== T_COMMENT)
+        if ($tokens[$commentEnd]['code'] !== T_DOC_COMMENT_CLOSE_TAG
+            && $tokens[$commentEnd]['code'] !== T_COMMENT
         ) {
             $phpcsFile->addError('Missing member variable doc comment', $stackPtr, 'Missing');
             return;

src/Standards/Squiz/Sniffs/Functions/FunctionDeclarationArgumentSpacingSniff.php

@@ -69,8 +69,6 @@ public function process(File $phpcsFile, int $stackPtr)
 
         if (isset($tokens[$stackPtr]['parenthesis_opener']) === false
             || isset($tokens[$stackPtr]['parenthesis_closer']) === false
-            || $tokens[$stackPtr]['parenthesis_opener'] === null
-            || $tokens[$stackPtr]['parenthesis_closer'] === null
         ) {
             return;
         }