From 5404ca723c266503e9ae76fed4a5dda73b42ae03 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adri=C3=A0=20Arrufat?= Date: Fri, 20 Mar 2026 10:18:16 +0900 Subject: [PATCH] SemanticTree: move NodeData initialization closer to usage --- src/SemanticTree.zig | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/src/SemanticTree.zig b/src/SemanticTree.zig index 94a63616..6366890f 100644 --- a/src/SemanticTree.zig +++ b/src/SemanticTree.zig @@ -196,18 +196,6 @@ fn walk( name = null; } - var data = NodeData{ - .id = cdp_node.id, - .axn = axn, - .role = role, - .name = name, - .value = value, - .options = options, - .xpath = xpath, - .is_interactive = is_interactive, - .node_name = node_name, - }; - var should_visit = true; if (self.interactive_only) { var keep = false; @@ -239,6 +227,18 @@ fn walk( var did_visit = false; var should_walk_children = true; + var data: NodeData = .{ + .id = cdp_node.id, + .axn = axn, + .role = role, + .name = name, + .value = value, + .options = options, + .xpath = xpath, + .is_interactive = is_interactive, + .node_name = node_name, + }; + if (should_visit) { should_walk_children = try visitor.visit(node, &data); did_visit = true; // Always true if should_visit was true, because visit() executed and opened structures