summaryrefslogtreecommitdiff
path: root/parse.y
diff options
context:
space:
mode:
authorPeter Zhu <peter@peterzhu.ca>2023-06-29 16:31:35 -0400
committerPeter Zhu <peter@peterzhu.ca>2023-06-30 09:13:31 -0400
commit58386814a7c7275f66ffa111175fca2fe307a1b5 (patch)
tree56bfd1daec3a6d83dfda64b569de1b9fbbb4d23c /parse.y
parent37a893d12915b8860f6880d6a0c2859e096ab4ff (diff)
Don't check for null pointer in calls to free
According to the C99 specification section 7.20.3.2 paragraph 2: > If ptr is a null pointer, no action occurs. So we do not need to check that the pointer is a null pointer.
Notes
Notes: Merged: https://github.com/ruby/ruby/pull/8004
Diffstat (limited to 'parse.y')
-rw-r--r--parse.y2
1 files changed, 1 insertions, 1 deletions
diff --git a/parse.y b/parse.y
index 329b578f36..13cb3842d5 100644
--- a/parse.y
+++ b/parse.y
@@ -13740,7 +13740,7 @@ rb_ruby_parser_free(void *ptr)
ruby_sized_xfree(p->tokenbuf, p->toksiz);
}
for (local = p->lvtbl; local; local = prev) {
- if (local->vars) xfree(local->vars);
+ xfree(local->vars);
prev = local->prev;
xfree(local);
}