From cdd07781b00b41b4f1f25fe9b12cd43d60a92834 Mon Sep 17 00:00:00 2001 From: TSUYUSATO Kitsune Date: Tue, 21 Nov 2023 12:20:22 +0900 Subject: [PATCH] [ruby/prism] Remove TODO https://github.com/ruby/prism/commit/d6d718487d --- prism/prism.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/prism/prism.c b/prism/prism.c index 4918e8a992..56b565a38d 100644 --- a/prism/prism.c +++ b/prism/prism.c @@ -16782,10 +16782,7 @@ parse_expression(pm_parser_t *parser, pm_binding_power_t binding_power, pm_diagn PM_NODE_TYPE_P(node, PM_PRE_EXECUTION_NODE) || PM_NODE_TYPE_P(node, PM_POST_EXECUTION_NODE) || PM_NODE_TYPE_P(node, PM_ALIAS_GLOBAL_VARIABLE_NODE) || PM_NODE_TYPE_P(node, PM_ALIAS_METHOD_NODE) || PM_NODE_TYPE_P(node, PM_UNDEF_NODE); - // TODO: the right condition should `pm_binding_powers[parser->current.type].left > PM_BINDING_POWER_MODIFIER_RESCUE` instead. - // However, it does not work because of the `rescue` modifier's binding power trick. - // After getting to merge #1879, this TODO can be removed. - if (is_statement && pm_binding_powers[parser->current.type].right > PM_BINDING_POWER_MODIFIER_RESCUE + 1) { + if (is_statement && pm_binding_powers[parser->current.type].left > PM_BINDING_POWER_MODIFIER_RESCUE) { return node; }