From 9552a7f9bf5d8da869efd224acbb3f7019d36715 Mon Sep 17 00:00:00 2001 From: Martin Vogel Date: Thu, 21 May 2026 11:48:22 +0200 Subject: [PATCH] fix(pipeline): drop duplicate `rel` declaration in resolve_worker MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When the genskip commit (aecd861) moved `const char *rel` earlier in the resolve_worker loop body (so the is_generated check could use it before the cross_lsp_eligible computation), the original declaration further down in the loop body wasn't removed — fails to compile with "redefinition of 'rel'". This was caught by the post-commit bench agent and fixed in-place but not landed; now it is. --- src/pipeline/pass_parallel.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/pipeline/pass_parallel.c b/src/pipeline/pass_parallel.c index 92d59f29d..0c6d93ca9 100644 --- a/src/pipeline/pass_parallel.c +++ b/src/pipeline/pass_parallel.c @@ -1830,7 +1830,6 @@ static void resolve_worker(int worker_id, void *ctx_ptr) { continue; } - const char *rel = rc->files[file_idx].rel_path; /* Build import map ONCE (read-only access to main_gbuf). The * same imp_keys/imp_vals feed both the fused cross-file LSP