Stop relying on parent in throw flagging visitor

Author: ssnepenthe

src/FeatureCounter/NodeVisitor/ThrowExpressionFlaggingNodeVisitor.php

@@ -5,39 +5,18 @@
 use PhpParser\Node;
 use PhpParser\Node\Expr\Throw_;
 use PhpParser\Node\Stmt\Expression;
+use PhpParser\NodeVisitor;
 use PhpParser\NodeVisitorAbstract;
 
-use function array_pop;
-use function count;
-
 final class ThrowExpressionFlaggingNodeVisitor extends NodeVisitorAbstract {
-    /**
-     * @var Node[]
-     */
-    private array $stack = [];
-
-    public function beforeTraverse(array $nodes) {
-        $this->stack = [];
-
-        return null;
-    }
-
     public function enterNode(Node $node) {
-        if ($node instanceof Throw_ && ! empty($this->stack)) {
-            $parent = $this->stack[count($this->stack) - 1];
-
-            if (! $parent instanceof Expression) {
-                $node->setAttribute('is_throw_expression', true);
-            }
+        if ($node instanceof Expression && $node->expr instanceof Throw_) {
+            return NodeVisitor::DONT_TRAVERSE_CHILDREN;
         }
 
-        $this->stack[] = $node;
-
-        return null;
-    }
-
-    public function leaveNode(Node $node) {
-        array_pop($this->stack);
+        if ($node instanceof Throw_) {
+            $node->setAttribute('is_throw_expression', true);
+        }
 
         return null;
     }