Skip to content

Commit 27a427b

Browse files
committed
Fix up void stmt generation in ripper translation
1 parent f4f7f8f commit 27a427b

File tree

2 files changed

+1
-11
lines changed

2 files changed

+1
-11
lines changed

lib/prism/translation/ripper.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1114,7 +1114,7 @@ def visit_def_node(node)
11141114

11151115
bodystmt =
11161116
if node.equal_loc.nil?
1117-
visit_body_node(node.body&.location || node.end_keyword_loc, node.body)
1117+
visit_body_node(node.rparen_loc || node.end_keyword_loc, node.body)
11181118
else
11191119
body = visit(node.body.body.first)
11201120

test/prism/ripper_test.rb

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -57,10 +57,6 @@ class RipperTest < TestCase
5757
seattlerb/case_in.txt
5858
seattlerb/case_in_else.txt
5959
seattlerb/class_comments.txt
60-
seattlerb/defn_arg_forward_args.txt
61-
seattlerb/defn_args_forward_args.txt
62-
seattlerb/defn_forward_args.txt
63-
seattlerb/defn_kwarg_lvar.txt
6460
seattlerb/defn_oneliner_eq2.txt
6561
seattlerb/defn_oneliner_rescue.txt
6662
seattlerb/defs_oneliner_eq2.txt
@@ -179,16 +175,11 @@ class RipperTest < TestCase
179175
whitequark/dedenting_non_interpolating_heredoc_line_continuation.txt
180176
whitequark/def.txt
181177
whitequark/empty_stmt.txt
182-
whitequark/forward_arg.txt
183178
whitequark/forward_args_legacy.txt
184-
whitequark/forwarded_argument_with_kwrestarg.txt
185-
whitequark/forwarded_argument_with_restarg.txt
186-
whitequark/forwarded_kwrestarg.txt
187179
whitequark/forwarded_kwrestarg_with_additional_kwarg.txt
188180
whitequark/forwarded_restarg.txt
189181
whitequark/if_elsif.txt
190182
whitequark/kwbegin_compstmt.txt
191-
whitequark/kwoptarg_with_kwrestarg_and_forwarded_args.txt
192183
whitequark/lvar_injecting_match.txt
193184
whitequark/masgn.txt
194185
whitequark/masgn_attr.txt
@@ -214,7 +205,6 @@ class RipperTest < TestCase
214205
whitequark/send_self.txt
215206
whitequark/slash_newline_in_heredocs.txt
216207
whitequark/string_concat.txt
217-
whitequark/trailing_forward_arg.txt
218208
]
219209

220210
relatives.each do |relative|

0 commit comments

Comments
 (0)