Do not ignore errors reported by PHPStan

sebastianbergmann · sebastianbergmann · commit e1d94111f699 · 2024-12-08T08:12:31.000+01:00
diff --git a/phpstan.neon b/phpstan.neon
@@ -4,53 +4,3 @@ parameters:
         - src
         - tests/tests
         - tests/bootstrap.php
-    ignoreErrors:
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\CodeUnitFindingVisitor\\:\\:enterNode\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|int\\|PhpParser\\\\Node\\|null\\) of method PhpParser\\\\NodeVisitor\\:\\:enterNode\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/CodeUnitFindingVisitor.php
-
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\CodeUnitFindingVisitor\\:\\:enterNode\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|int\\|PhpParser\\\\Node\\|null\\) of method PhpParser\\\\NodeVisitorAbstract\\:\\:enterNode\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/CodeUnitFindingVisitor.php
-
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\ExecutableLinesFindingVisitor\\:\\:afterTraverse\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|null\\) of method PhpParser\\\\NodeVisitor\\:\\:afterTraverse\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/ExecutableLinesFindingVisitor.php
-
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\ExecutableLinesFindingVisitor\\:\\:afterTraverse\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|null\\) of method PhpParser\\\\NodeVisitorAbstract\\:\\:afterTraverse\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/ExecutableLinesFindingVisitor.php
-
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\ExecutableLinesFindingVisitor\\:\\:enterNode\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|int\\|PhpParser\\\\Node\\|null\\) of method PhpParser\\\\NodeVisitor\\:\\:enterNode\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/ExecutableLinesFindingVisitor.php
-
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\ExecutableLinesFindingVisitor\\:\\:enterNode\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|int\\|PhpParser\\\\Node\\|null\\) of method PhpParser\\\\NodeVisitorAbstract\\:\\:enterNode\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/ExecutableLinesFindingVisitor.php
-
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\IgnoredLinesFindingVisitor\\:\\:enterNode\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|int\\|PhpParser\\\\Node\\|null\\) of method PhpParser\\\\NodeVisitor\\:\\:enterNode\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/IgnoredLinesFindingVisitor.php
-
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\IgnoredLinesFindingVisitor\\:\\:enterNode\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|int\\|PhpParser\\\\Node\\|null\\) of method PhpParser\\\\NodeVisitorAbstract\\:\\:enterNode\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/IgnoredLinesFindingVisitor.php
-
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\CodeUnitFindingVisitor\\:\\:leaveNode\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|int\\|PhpParser\\\\Node\\|null\\) of method PhpParser\\\\NodeVisitor\\:\\:leaveNode\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/CodeUnitFindingVisitor.php
-
-        -
-            message: "#^Return type \\(void\\) of method SebastianBergmann\\\\CodeCoverage\\\\StaticAnalysis\\\\CodeUnitFindingVisitor\\:\\:leaveNode\\(\\) should be compatible with return type \\(array\\<PhpParser\\\\Node\\>\\|int\\|PhpParser\\\\Node\\|null\\) of method PhpParser\\\\NodeVisitorAbstract\\:\\:leaveNode\\(\\)$#"
-            count: 1
-            path: src/StaticAnalysis/CodeUnitFindingVisitor.php
diff --git a/src/StaticAnalysis/CodeUnitFindingVisitor.php b/src/StaticAnalysis/CodeUnitFindingVisitor.php
@@ -67,15 +67,15 @@ public function __construct(string $file)
         $this->file = $file;
     }
 
-    public function enterNode(Node $node): void
+    public function enterNode(Node $node): null
     {
         if ($node instanceof Interface_) {
             $this->processInterface($node);
         }
 
         if ($node instanceof Class_) {
             if ($node->isAnonymous()) {
-                return;
+                return null;
             }
 
             $this->processClass($node);
@@ -86,20 +86,22 @@ public function enterNode(Node $node): void
         }
 
         if (!$node instanceof Function_) {
-            return;
+            return null;
         }
 
         $this->processFunction($node);
+
+        return null;
     }
 
-    public function leaveNode(Node $node): void
+    public function leaveNode(Node $node): null
     {
         if (!$node instanceof Class_) {
-            return;
+            return null;
         }
 
         if ($node->isAnonymous()) {
-            return;
+            return null;
         }
 
         $traits = [];
@@ -111,7 +113,7 @@ public function leaveNode(Node $node): void
         }
 
         if (empty($traits)) {
-            return;
+            return null;
         }
 
         $namespacedClassName = $node->namespacedName->toString();
@@ -130,6 +132,8 @@ public function leaveNode(Node $node): void
             $traits,
             $this->classes[$namespacedClassName]->methods(),
         );
+
+        return null;
     }
 
     /**
diff --git a/src/StaticAnalysis/ExecutableLinesFindingVisitor.php b/src/StaticAnalysis/ExecutableLinesFindingVisitor.php
@@ -54,7 +54,7 @@ public function __construct(string $source)
         $this->source = $source;
     }
 
-    public function enterNode(Node $node): void
+    public function enterNode(Node $node): null
     {
         foreach ($node->getComments() as $comment) {
             $commentLine = $comment->getStartLine();
@@ -80,15 +80,15 @@ public function enterNode(Node $node): void
                 }
             }
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Stmt\Interface_) {
             foreach (range($node->getStartLine(), $node->getEndLine()) as $line) {
                 $this->unsets[$line] = true;
             }
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Stmt\Declare_ ||
@@ -113,7 +113,7 @@ public function enterNode(Node $node): void
             $node instanceof Node\Name ||
             $node instanceof Node\Param ||
             $node instanceof Node\Scalar) {
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Expr\Match_) {
@@ -125,13 +125,13 @@ public function enterNode(Node $node): void
                 );
             }
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Stmt\Expression && $node->expr instanceof Node\Expr\Throw_) {
             $this->setLineBranch($node->expr->expr->getEndLine(), $node->expr->expr->getEndLine(), ++$this->nextBranch);
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Stmt\Enum_ ||
@@ -176,7 +176,7 @@ public function enterNode(Node $node): void
             }
 
             if ($isConcreteClassLike) {
-                return;
+                return null;
             }
 
             $hasEmptyBody = [] === $node->stmts ||
@@ -188,15 +188,15 @@ public function enterNode(Node $node): void
 
             if ($hasEmptyBody) {
                 if ($node->getEndLine() === $node->getStartLine() && isset($this->executableLinesGroupedByBranch[$node->getStartLine()])) {
-                    return;
+                    return null;
                 }
 
                 $this->setLineBranch($node->getEndLine(), $node->getEndLine(), ++$this->nextBranch);
 
-                return;
+                return null;
             }
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Expr\ArrowFunction) {
@@ -208,12 +208,12 @@ public function enterNode(Node $node): void
             $endLine = $node->expr->getEndLine();
 
             if ($endLine < $startLine) {
-                return;
+                return null;
             }
 
             $this->setLineBranch($startLine, $endLine, ++$this->nextBranch);
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Expr\Ternary) {
@@ -226,22 +226,22 @@ public function enterNode(Node $node): void
                 $this->setLineBranch($node->else->getStartLine(), $node->else->getEndLine(), ++$this->nextBranch);
             }
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Expr\BinaryOp\Coalesce) {
             if ($node->getStartLine() !== $node->getEndLine()) {
                 $this->setLineBranch($node->getEndLine(), $node->getEndLine(), ++$this->nextBranch);
             }
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Stmt\If_ ||
             $node instanceof Node\Stmt\ElseIf_ ||
             $node instanceof Node\Stmt\Case_) {
             if (null === $node->cond) {
-                return;
+                return null;
             }
 
             $this->setLineBranch(
@@ -250,7 +250,7 @@ public function enterNode(Node $node): void
                 ++$this->nextBranch,
             );
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Stmt\For_) {
@@ -292,7 +292,7 @@ public function enterNode(Node $node): void
             }
 
             if (null === $startLine || null === $endLine) {
-                return;
+                return null;
             }
 
             $this->setLineBranch(
@@ -301,7 +301,7 @@ public function enterNode(Node $node): void
                 ++$this->nextBranch,
             );
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Stmt\Foreach_) {
@@ -311,7 +311,7 @@ public function enterNode(Node $node): void
                 ++$this->nextBranch,
             );
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Stmt\While_ ||
@@ -322,7 +322,7 @@ public function enterNode(Node $node): void
                 ++$this->nextBranch,
             );
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Stmt\Catch_) {
@@ -337,7 +337,7 @@ public function enterNode(Node $node): void
                 ++$this->nextBranch,
             );
 
-            return;
+            return null;
         }
 
         if ($node instanceof Node\Expr\CallLike) {
@@ -349,17 +349,19 @@ public function enterNode(Node $node): void
 
             $this->setLineBranch($node->getStartLine(), $node->getEndLine(), $branch);
 
-            return;
+            return null;
         }
 
         if (isset($this->executableLinesGroupedByBranch[$node->getStartLine()])) {
-            return;
+            return null;
         }
 
         $this->setLineBranch($node->getStartLine(), $node->getEndLine(), ++$this->nextBranch);
+
+        return null;
     }
 
-    public function afterTraverse(array $nodes): void
+    public function afterTraverse(array $nodes): null
     {
         $lines = explode("\n", $this->source);
 
@@ -379,6 +381,8 @@ public function afterTraverse(array $nodes): void
             $this->executableLinesGroupedByBranch,
             $this->unsets,
         );
+
+        return null;
     }
 
     /**
diff --git a/src/StaticAnalysis/IgnoredLinesFindingVisitor.php b/src/StaticAnalysis/IgnoredLinesFindingVisitor.php
@@ -39,7 +39,7 @@ public function __construct(bool $useAnnotationsForIgnoringCode, bool $ignoreDep
         $this->ignoreDeprecated              = $ignoreDeprecated;
     }
 
-    public function enterNode(Node $node): void
+    public function enterNode(Node $node): null
     {
         if (!$node instanceof Class_ &&
             !$node instanceof Trait_ &&
@@ -48,11 +48,11 @@ public function enterNode(Node $node): void
             !$node instanceof ClassMethod &&
             !$node instanceof Function_ &&
             !$node instanceof Attribute) {
-            return;
+            return null;
         }
 
         if ($node instanceof Class_ && $node->isAnonymous()) {
-            return;
+            return null;
         }
 
         if ($node instanceof Class_ ||
@@ -68,11 +68,11 @@ public function enterNode(Node $node): void
         }
 
         if (!$this->useAnnotationsForIgnoringCode) {
-            return;
+            return null;
         }
 
         if ($node instanceof Interface_) {
-            return;
+            return null;
         }
 
         if ($node instanceof Attribute &&
@@ -84,10 +84,12 @@ public function enterNode(Node $node): void
                 $this->ignoredLines[] = $line;
             }
 
-            return;
+            return null;
         }
 
         $this->processDocComment($node);
+
+        return null;
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -67,15 +67,15 @@ public function __construct(string $file)`
`67`	`67`	`$this->file = $file;`
`68`	`68`	`}`
`69`	`69`
`70`		`- public function enterNode(Node $node): void`
	`70`	`+ public function enterNode(Node $node): null`
`71`	`71`	`{`
`72`	`72`	`if ($node instanceof Interface_) {`
`73`	`73`	`$this->processInterface($node);`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`if ($node instanceof Class_) {`
`77`	`77`	`if ($node->isAnonymous()) {`
`78`		`- return;`
	`78`	`+ return null;`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`$this->processClass($node);`
`@@ -86,20 +86,22 @@ public function enterNode(Node $node): void`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`if (!$node instanceof Function_) {`
`89`		`- return;`
	`89`	`+ return null;`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`$this->processFunction($node);`
	`93`	`+`
	`94`	`+ return null;`
`93`	`95`	`}`
`94`	`96`
`95`		`- public function leaveNode(Node $node): void`
	`97`	`+ public function leaveNode(Node $node): null`
`96`	`98`	`{`
`97`	`99`	`if (!$node instanceof Class_) {`
`98`		`- return;`
	`100`	`+ return null;`
`99`	`101`	`}`
`100`	`102`
`101`	`103`	`if ($node->isAnonymous()) {`
`102`		`- return;`
	`104`	`+ return null;`
`103`	`105`	`}`
`104`	`106`
`105`	`107`	`$traits = [];`
`@@ -111,7 +113,7 @@ public function leaveNode(Node $node): void`
`111`	`113`	`}`
`112`	`114`
`113`	`115`	`if (empty($traits)) {`
`114`		`- return;`
	`116`	`+ return null;`
`115`	`117`	`}`
`116`	`118`
`117`	`119`	`$namespacedClassName = $node->namespacedName->toString();`
`@@ -130,6 +132,8 @@ public function leaveNode(Node $node): void`
`130`	`132`	`$traits,`
`131`	`133`	`$this->classes[$namespacedClassName]->methods(),`
`132`	`134`	`);`
	`135`	`+`
	`136`	`+ return null;`
`133`	`137`	`}`
`134`	`138`
`135`	`139`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ public function __construct(string $source)`
`54`	`54`	`$this->source = $source;`
`55`	`55`	`}`
`56`	`56`
`57`		`- public function enterNode(Node $node): void`
	`57`	`+ public function enterNode(Node $node): null`
`58`	`58`	`{`
`59`	`59`	`foreach ($node->getComments() as $comment) {`
`60`	`60`	`$commentLine = $comment->getStartLine();`
`@@ -80,15 +80,15 @@ public function enterNode(Node $node): void`
`80`	`80`	`}`
`81`	`81`	`}`
`82`	`82`
`83`		`- return;`
	`83`	`+ return null;`
`84`	`84`	`}`
`85`	`85`
`86`	`86`	`if ($node instanceof Node\Stmt\Interface_) {`
`87`	`87`	`foreach (range($node->getStartLine(), $node->getEndLine()) as $line) {`
`88`	`88`	`$this->unsets[$line] = true;`
`89`	`89`	`}`
`90`	`90`
`91`		`- return;`
	`91`	`+ return null;`
`92`	`92`	`}`
`93`	`93`
`94`	`94`	`if ($node instanceof Node\Stmt\Declare_ \|\|`
`@@ -113,7 +113,7 @@ public function enterNode(Node $node): void`
`113`	`113`	`$node instanceof Node\Name \|\|`
`114`	`114`	`$node instanceof Node\Param \|\|`
`115`	`115`	`$node instanceof Node\Scalar) {`
`116`		`- return;`
	`116`	`+ return null;`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`if ($node instanceof Node\Expr\Match_) {`
`@@ -125,13 +125,13 @@ public function enterNode(Node $node): void`
`125`	`125`	`);`
`126`	`126`	`}`
`127`	`127`
`128`		`- return;`
	`128`	`+ return null;`
`129`	`129`	`}`
`130`	`130`
`131`	`131`	`if ($node instanceof Node\Stmt\Expression && $node->expr instanceof Node\Expr\Throw_) {`
`132`	`132`	`$this->setLineBranch($node->expr->expr->getEndLine(), $node->expr->expr->getEndLine(), ++$this->nextBranch);`
`133`	`133`
`134`		`- return;`
	`134`	`+ return null;`
`135`	`135`	`}`
`136`	`136`
`137`	`137`	`if ($node instanceof Node\Stmt\Enum_ \|\|`
`@@ -176,7 +176,7 @@ public function enterNode(Node $node): void`
`176`	`176`	`}`
`177`	`177`
`178`	`178`	`if ($isConcreteClassLike) {`
`179`		`- return;`
	`179`	`+ return null;`
`180`	`180`	`}`
`181`	`181`
`182`	`182`	`$hasEmptyBody = [] === $node->stmts \|\|`
`@@ -188,15 +188,15 @@ public function enterNode(Node $node): void`
`188`	`188`
`189`	`189`	`if ($hasEmptyBody) {`
`190`	`190`	`if ($node->getEndLine() === $node->getStartLine() && isset($this->executableLinesGroupedByBranch[$node->getStartLine()])) {`
`191`		`- return;`
	`191`	`+ return null;`
`192`	`192`	`}`
`193`	`193`
`194`	`194`	`$this->setLineBranch($node->getEndLine(), $node->getEndLine(), ++$this->nextBranch);`
`195`	`195`
`196`		`- return;`
	`196`	`+ return null;`
`197`	`197`	`}`
`198`	`198`
`199`		`- return;`
	`199`	`+ return null;`
`200`	`200`	`}`
`201`	`201`
`202`	`202`	`if ($node instanceof Node\Expr\ArrowFunction) {`
`@@ -208,12 +208,12 @@ public function enterNode(Node $node): void`
`208`	`208`	`$endLine = $node->expr->getEndLine();`
`209`	`209`
`210`	`210`	`if ($endLine < $startLine) {`
`211`		`- return;`
	`211`	`+ return null;`
`212`	`212`	`}`
`213`	`213`
`214`	`214`	`$this->setLineBranch($startLine, $endLine, ++$this->nextBranch);`
`215`	`215`
`216`		`- return;`
	`216`	`+ return null;`
`217`	`217`	`}`
`218`	`218`
`219`	`219`	`if ($node instanceof Node\Expr\Ternary) {`
`@@ -226,22 +226,22 @@ public function enterNode(Node $node): void`
`226`	`226`	`$this->setLineBranch($node->else->getStartLine(), $node->else->getEndLine(), ++$this->nextBranch);`
`227`	`227`	`}`
`228`	`228`
`229`		`- return;`
	`229`	`+ return null;`
`230`	`230`	`}`
`231`	`231`
`232`	`232`	`if ($node instanceof Node\Expr\BinaryOp\Coalesce) {`
`233`	`233`	`if ($node->getStartLine() !== $node->getEndLine()) {`
`234`	`234`	`$this->setLineBranch($node->getEndLine(), $node->getEndLine(), ++$this->nextBranch);`
`235`	`235`	`}`
`236`	`236`
`237`		`- return;`
	`237`	`+ return null;`
`238`	`238`	`}`
`239`	`239`
`240`	`240`	`if ($node instanceof Node\Stmt\If_ \|\|`
`241`	`241`	`$node instanceof Node\Stmt\ElseIf_ \|\|`
`242`	`242`	`$node instanceof Node\Stmt\Case_) {`
`243`	`243`	`if (null === $node->cond) {`
`244`		`- return;`
	`244`	`+ return null;`
`245`	`245`	`}`
`246`	`246`
`247`	`247`	`$this->setLineBranch(`
`@@ -250,7 +250,7 @@ public function enterNode(Node $node): void`
`250`	`250`	`++$this->nextBranch,`
`251`	`251`	`);`
`252`	`252`
`253`		`- return;`
	`253`	`+ return null;`
`254`	`254`	`}`
`255`	`255`
`256`	`256`	`if ($node instanceof Node\Stmt\For_) {`
`@@ -292,7 +292,7 @@ public function enterNode(Node $node): void`
`292`	`292`	`}`
`293`	`293`
`294`	`294`	`if (null === $startLine \|\| null === $endLine) {`
`295`		`- return;`
	`295`	`+ return null;`
`296`	`296`	`}`
`297`	`297`
`298`	`298`	`$this->setLineBranch(`
`@@ -301,7 +301,7 @@ public function enterNode(Node $node): void`
`301`	`301`	`++$this->nextBranch,`
`302`	`302`	`);`
`303`	`303`
`304`		`- return;`
	`304`	`+ return null;`
`305`	`305`	`}`
`306`	`306`
`307`	`307`	`if ($node instanceof Node\Stmt\Foreach_) {`
`@@ -311,7 +311,7 @@ public function enterNode(Node $node): void`
`311`	`311`	`++$this->nextBranch,`
`312`	`312`	`);`
`313`	`313`
`314`		`- return;`
	`314`	`+ return null;`
`315`	`315`	`}`
`316`	`316`
`317`	`317`	`if ($node instanceof Node\Stmt\While_ \|\|`
`@@ -322,7 +322,7 @@ public function enterNode(Node $node): void`
`322`	`322`	`++$this->nextBranch,`
`323`	`323`	`);`
`324`	`324`
`325`		`- return;`
	`325`	`+ return null;`
`326`	`326`	`}`
`327`	`327`
`328`	`328`	`if ($node instanceof Node\Stmt\Catch_) {`
`@@ -337,7 +337,7 @@ public function enterNode(Node $node): void`
`337`	`337`	`++$this->nextBranch,`
`338`	`338`	`);`
`339`	`339`
`340`		`- return;`
	`340`	`+ return null;`
`341`	`341`	`}`
`342`	`342`
`343`	`343`	`if ($node instanceof Node\Expr\CallLike) {`
`@@ -349,17 +349,19 @@ public function enterNode(Node $node): void`
`349`	`349`
`350`	`350`	`$this->setLineBranch($node->getStartLine(), $node->getEndLine(), $branch);`
`351`	`351`
`352`		`- return;`
	`352`	`+ return null;`
`353`	`353`	`}`
`354`	`354`
`355`	`355`	`if (isset($this->executableLinesGroupedByBranch[$node->getStartLine()])) {`
`356`		`- return;`
	`356`	`+ return null;`
`357`	`357`	`}`
`358`	`358`
`359`	`359`	`$this->setLineBranch($node->getStartLine(), $node->getEndLine(), ++$this->nextBranch);`
	`360`	`+`
	`361`	`+ return null;`
`360`	`362`	`}`
`361`	`363`
`362`		`- public function afterTraverse(array $nodes): void`
	`364`	`+ public function afterTraverse(array $nodes): null`
`363`	`365`	`{`
`364`	`366`	`$lines = explode("\n", $this->source);`
`365`	`367`
`@@ -379,6 +381,8 @@ public function afterTraverse(array $nodes): void`
`379`	`381`	`$this->executableLinesGroupedByBranch,`
`380`	`382`	`$this->unsets,`
`381`	`383`	`);`
	`384`	`+`
	`385`	`+ return null;`
`382`	`386`	`}`
`383`	`387`
`384`	`388`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public function __construct(bool $useAnnotationsForIgnoringCode, bool $ignoreDep`
`39`	`39`	`$this->ignoreDeprecated = $ignoreDeprecated;`
`40`	`40`	`}`
`41`	`41`
`42`		`- public function enterNode(Node $node): void`
	`42`	`+ public function enterNode(Node $node): null`
`43`	`43`	`{`
`44`	`44`	`if (!$node instanceof Class_ &&`
`45`	`45`	`!$node instanceof Trait_ &&`
`@@ -48,11 +48,11 @@ public function enterNode(Node $node): void`
`48`	`48`	`!$node instanceof ClassMethod &&`
`49`	`49`	`!$node instanceof Function_ &&`
`50`	`50`	`!$node instanceof Attribute) {`
`51`		`- return;`
	`51`	`+ return null;`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`if ($node instanceof Class_ && $node->isAnonymous()) {`
`55`		`- return;`
	`55`	`+ return null;`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`if ($node instanceof Class_ \|\|`
`@@ -68,11 +68,11 @@ public function enterNode(Node $node): void`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`if (!$this->useAnnotationsForIgnoringCode) {`
`71`		`- return;`
	`71`	`+ return null;`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`if ($node instanceof Interface_) {`
`75`		`- return;`
	`75`	`+ return null;`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`if ($node instanceof Attribute &&`
`@@ -84,10 +84,12 @@ public function enterNode(Node $node): void`
`84`	`84`	`$this->ignoredLines[] = $line;`
`85`	`85`	`}`
`86`	`86`
`87`		`- return;`
	`87`	`+ return null;`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`$this->processDocComment($node);`
	`91`	`+`
	`92`	`+ return null;`
`91`	`93`	`}`
`92`	`94`
`93`	`95`	`/**`