tests; improved implicit call recognition with dots on the left in the

Author: Zdenko Vujasinovic

lib/coffeescript/rewriter.js

@@ -268,7 +268,7 @@ exports.Rewriter = class Rewriter
       if (tag in IMPLICIT_FUNC and token.spaced or
           tag is '?' and i > 0 and not tokens[i - 1].spaced) and
          (nextTag in IMPLICIT_CALL or 
-         (nextTag is '...' and not @findTagsBackwards(i, ['INDEX_START', '['])) or
+         (nextTag is '...' and @tag(i + 2) in IMPLICIT_CALL and not @findTagsBackwards(i, ['INDEX_START', '['])) or
           nextTag in IMPLICIT_UNSPACED_CALL and
           not nextToken.spaced and not nextToken.newLine)
         tag = token[0] = 'FUNC_EXIST' if tag is '?'

src/rewriter.coffee

@@ -64,6 +64,18 @@ test "#1722: operator precedence in unbounded slice compilation", ->
 test "#2349: inclusive slicing to numeric strings", ->
   arrayEq [0, 1], [0..10][.."1"]
 
+test "#4631: slicing with space before and/or after the dots", ->
+  a = (s) -> s
+  b = [4, 5, 6]
+  c = [7, 8, 9]
+  arrayEq [2, 3, 4], shared[2 ... 5]
+  arrayEq [3, 4, 5], shared[3... 6]
+  arrayEq [4, 5, 6], shared[4 ...7]
+  arrayEq shared[(a b...)...(a c...)]  , shared[(a ...b)...(a ...c)]
+  arrayEq shared[(a b...) ... (a c...)], shared[(a ...b) ... (a ...c)]
+  arrayEq shared[(a b...)... (a c...)] , shared[(a ...b)... (a ...c)]
+  arrayEq shared[(a b...) ...(a c...)] , shared[(a ...b) ...(a ...c)]
+
 
 # Splicing