Remove a useless check

mame · mame · commit a32981b6b89c · 2024-11-29T12:27:04.000+09:00
Here `nb` should never be NULL. If it were, the following `nb->buffer_list` would be strange. A follow-up to ddd8da4
diff --git a/node.c b/node.c
@@ -225,7 +225,7 @@ free_ast_value(rb_ast_t *ast, void *ctx, NODE *node)
 static void
 rb_node_buffer_free(rb_ast_t *ast, node_buffer_t *nb)
 {
-    if (nb && nb->tokens) {
+    if (nb->tokens) {
         parser_tokens_free(ast, nb->tokens);
     }
     iterate_node_values(ast, &nb->buffer_list, free_ast_value, NULL);

Original file line number	Diff line number	Diff line change
`@@ -225,7 +225,7 @@ free_ast_value(rb_ast_t ast, void ctx, NODE *node)`
`225`	`225`	`static void`
`226`	`226`	`rb_node_buffer_free(rb_ast_t ast, node_buffer_t nb)`
`227`	`227`	`{`
`228`		`- if (nb && nb->tokens) {`
	`228`	`+ if (nb->tokens) {`
`229`	`229`	`parser_tokens_free(ast, nb->tokens);`
`230`	`230`	`}`
`231`	`231`	`iterate_node_values(ast, &nb->buffer_list, free_ast_value, NULL);`