diff options
| author | 卜部昌平 <shyouhei@ruby-lang.org> | 2024-04-25 10:08:39 +0200 |
|---|---|---|
| committer | 卜部昌平 <shyouhei@ruby-lang.org> | 2024-04-25 10:12:47 +0200 |
| commit | 64bd8e41df859fc61f373fcab431e1516148ee57 (patch) | |
| tree | ed81690e0c4e8088229242db432d7913d0a36655 | |
| parent | 03f8477566d544c240cd1949fd56d8e42d686aa8 (diff) | |
false is not a pointer
This function returned VALUE before. False made sense back then.
Now that it returns a pointer. NULL should be used instead.
| -rw-r--r-- | parse.y | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -12833,13 +12833,13 @@ literal_concat0(struct parser_params *p, rb_parser_string_t *head, rb_parser_str static rb_parser_string_t * string_literal_head(struct parser_params *p, enum node_type htype, NODE *head) { - if (htype != NODE_DSTR) return false; + if (htype != NODE_DSTR) return NULL; if (RNODE_DSTR(head)->nd_next) { head = RNODE_LIST(RNODE_LIST(RNODE_DSTR(head)->nd_next)->as.nd_end)->nd_head; - if (!head || !nd_type_p(head, NODE_STR)) return false; + if (!head || !nd_type_p(head, NODE_STR)) return NULL; } rb_parser_string_t *lit = RNODE_DSTR(head)->string; - ASSUME(lit != false); + ASSUME(lit); return lit; } |
