Fix parsing of paren following '/' operator without space

adambeynon · adambeynon · commit 03f28902f244 · 2015-05-05T11:16:02.000+01:00
Fixes #728.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,8 @@
 ## edge (upcoming 0.8.0)
 
+*   Parser: Fix parsing of parens following divide operator without a
+    space.
+
 *   Parser: Fix bug where keyword arguments could not be parsed if
     method definition did not have parens around arguments.
 
diff --git a/lib/opal/parser/lexer.rb b/lib/opal/parser/lexer.rb
@@ -879,13 +879,17 @@ def yylex
           elsif scan(/\=/)
             @lex_state = :expr_beg
             return new_op_asgn('/')
-          elsif after_operator?
-            @lex_state = :expr_arg
-          elsif arg?
+          end
+
+          if arg?
             if !check(/\s/) && @space_seen
               self.strterm = new_strterm(STR_REGEXP, '/', '/')
               return :tREGEXP_BEG
             end
+          end
+
+          if after_operator?
+            @lex_state = :expr_arg
           else
             @lex_state = :expr_beg
           end
diff --git a/spec/lib/parser/call_spec.rb b/spec/lib/parser/call_spec.rb
@@ -144,3 +144,12 @@
                                        [:arglist, [:sym, :+]]]
   end
 end
+
+describe 'Operator calls followed by parens' do
+  it 'correctly parses parens' do
+    parsed('1/(2)').should == [:call, [:int, 1], '/'.to_sym, [:arglist, [:paren, [:int, 2]]]]
+    parsed('1*(2)').should == [:call, [:int, 1], '*'.to_sym, [:arglist, [:paren, [:int, 2]]]]
+    parsed('1+(2)').should == [:call, [:int, 1], '+'.to_sym, [:arglist, [:paren, [:int, 2]]]]
+    parsed('1-(2)').should == [:call, [:int, 1], '-'.to_sym, [:arglist, [:paren, [:int, 2]]]]
+  end
+end
