Changed linesToIgnore attribute format

phpstan · Jul 25, 2023 · 2862a7b · 2862a7b
1 parent 4c4f9c8
commit 2862a7b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/Analyser/FileAnalyser.php b/src/Analyser/FileAnalyser.php
@@ -242,10 +242,10 @@ private function getLinesToIgnoreFromTokens(array $nodes): array
 			return [];
 		}
 
-		/** @var int[] $tokenLines */
+		/** @var array<int, list<string>|null> $tokenLines */
 		$tokenLines = $nodes[0]->getAttribute('linesToIgnore', []);
 		$lines = [];
-		foreach ($tokenLines as $tokenLine) {
+		foreach (array_keys($tokenLines) as $tokenLine) {
 			$lines[$tokenLine] = true;
 		}
 

diff --git a/src/Parser/RichParser.php b/src/Parser/RichParser.php
@@ -11,10 +11,10 @@
 use PHPStan\File\FileReader;
 use PHPStan\ShouldNotHappenException;
 use function array_filter;
-use function array_values;
 use function is_string;
 use function strpos;
 use function substr_count;
+use const ARRAY_FILTER_USE_KEY;
 use const T_COMMENT;
 use const T_DOC_COMMENT;
 
@@ -78,15 +78,15 @@ public function parseString(string $sourceCode): array
 		}
 
 		foreach ($traitCollectingVisitor->traits as $trait) {
-			$trait->setAttribute('linesToIgnore', array_values(array_filter($linesToIgnore, static fn (int $line): bool => $line >= $trait->getStartLine() && $line <= $trait->getEndLine())));
+			$trait->setAttribute('linesToIgnore', array_filter($linesToIgnore, static fn (int $line): bool => $line >= $trait->getStartLine() && $line <= $trait->getEndLine(), ARRAY_FILTER_USE_KEY));
 		}
 
 		return $nodes;
 	}
 
 	/**
 	 * @param mixed[] $tokens
-	 * @return int[]
+	 * @return array<int, list<string>|null>
 	 */
 	private function getLinesToIgnore(array $tokens): array
 	{
@@ -111,7 +111,7 @@ private function getLinesToIgnore(array $tokens): array
 
 			$line += substr_count($token[1], "\n");
 
-			$lines[] = $line;
+			$lines[$line] = null;
 		}
 
 		return $lines;