Silence unused var warning for variables starting with _ (#469)

Akuli · Dec 19, 2023 · 799bfce · 799bfce
1 parent 31cc68c
commit 799bfce
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/simplify_cfg.c b/src/simplify_cfg.c
@@ -454,6 +454,7 @@ static void remove_unused_variables(CfGraph *cfg)
         if (
             used[i] == WRITE
             && cfg->locals.ptr[i]->name[0] != '\0'
+            && cfg->locals.ptr[i]->name[0] != '_'
             && strcmp(cfg->locals.ptr[i]->name, "return") != 0
         ) {
             show_warning(write_locations[i], "variable '%s' is never used", cfg->locals.ptr[i]->name);

diff --git a/tests/should_succeed/unused_variable.jou b/tests/should_succeed/unused_variable.jou
@@ -1,3 +1,6 @@
 def main() -> int:
+    # You get no warning if variable name starts with '_'
+    _ = 123
     x = 4  # Warning: variable 'x' is never used
+    _foo = 7
     return 0