Merge pull request #3441 from Earlopain/parser-translator-rescue-semicolon

Fix parser translator rescue location with semicolon body

Author: kddnewton

lib/prism/translation/parser/compiler.rb

@@ -203,7 +203,14 @@ def visit_begin_node(node)
           if (rescue_clause = node.rescue_clause)
             begin
               find_start_offset = (rescue_clause.reference&.location || rescue_clause.exceptions.last&.location || rescue_clause.keyword_loc).end_offset
-              find_end_offset = (rescue_clause.statements&.location&.start_offset || rescue_clause.subsequent&.location&.start_offset || (find_start_offset + 1))
+              find_end_offset = (
+                rescue_clause.statements&.location&.start_offset ||
+                rescue_clause.subsequent&.location&.start_offset ||
+                node.else_clause&.location&.start_offset ||
+                node.ensure_clause&.location&.start_offset ||
+                node.end_keyword_loc&.start_offset ||
+                find_start_offset + 1
+              )
 
               rescue_bodies << builder.rescue_body(
                 token(rescue_clause.keyword_loc),

test/prism/fixtures/begin_rescue.txt

@@ -2,6 +2,12 @@ begin; a; rescue; b; else; c; end
 
 begin; a; rescue; b; else; c; ensure; d; end
 
+begin; rescue ; end
+
+begin; rescue ; ensure ; end
+
+begin; rescue ; else ; end
+
 begin
 a
 end