1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[PRISM] Account for RescueNodes with no statements

Browse Source 
We need a PUTNIL if a RescueNode has no statements.
This commit is contained in:
Jemma Issroff 2023-12-01 13:00:20 -05:00
parent d6584a0201
commit 2a8d9c59ff
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
prism_compile.c
View File

@ -3658,9 +3658,14 @@ pm_compile_node(rb_iseq_t *iseq, const pm_node_t *node, LINK_ANCHOR *const ret,
ADD_GETLOCAL(ret, &dummy_line_node, LVAR_ERRINFO, 0); ADD_GETLOCAL(ret, &dummy_line_node, LVAR_ERRINFO, 0);
PM_COMPILE((pm_node_t *)rescue_node->reference); PM_COMPILE((pm_node_t *)rescue_node->reference);
} }
if (rescue_node->statements) { if (rescue_node->statements) {
PM_COMPILE((pm_node_t *)rescue_node->statements); PM_COMPILE((pm_node_t *)rescue_node->statements);
} }
else {
PM_PUTNIL;
}
ADD_INSN(ret, &dummy_line_node, leave); ADD_INSN(ret, &dummy_line_node, leave);
ADD_LABEL(ret, rescue_end); ADD_LABEL(ret, rescue_end);

6
test/ruby/test_compile_prism.rb
View File

@ -792,6 +792,12 @@ module Prism
rescue StandardError => e rescue StandardError => e
end end
CODE CODE
assert_prism_eval(<<~CODE)
begin
raise StandardError
rescue StandardError => e
end
CODE
assert_prism_eval(<<~CODE) assert_prism_eval(<<~CODE)
begin begin
1 1