Merge pull request #68 from mvz/compatible-rescue-with-multiple-assig…

…nment

Handle multiple assignment with RHS that is a block

lib/ripper_ruby_parser/sexp_handlers/assignment.rb

@@ -35,7 +35,7 @@ def process_massign(exp)
         when :mrhs
           _, right = right
         else
-          right = s(:to_ary, right)
+          right = s(:to_ary, unwrap_begin(right))
         end
 
         s(:masgn, left, right)

lib/ripper_ruby_parser/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RipperRubyParser
-  VERSION = '1.5.1'
+  VERSION = '1.6.0.pre1'
 end

test/ripper_ruby_parser/sexp_handlers/assignment_test.rb

@@ -311,6 +311,13 @@
                               s(:to_ary, s(:call, nil, :baz)))
       end
 
+      it 'works with blocks' do
+        'foo, bar = begin; baz; end'.
+          must_be_parsed_as s(:masgn,
+                              s(:array, s(:lasgn, :foo), s(:lasgn, :bar)),
+                              s(:to_ary, s(:call, nil, :baz)))
+      end
+
       it 'works with a rescue modifier' do
         'foo, bar = baz rescue qux'.
           must_be_parsed_as s(:rescue,

test/samples/misc.rb

@@ -163,6 +163,13 @@ def barfoo
     puts err
   end
 
+  # begin/rescue with multiple assignment
+  foo, bar = begin
+    baz
+  rescue qux
+    quuz
+  end
+
   # alias
   alias foo bar
   alias :foo bar