Issue #894: Strange interaction between class instantiation and splats

lib/nodes.js

@@ -520,7 +520,11 @@
     __extends(Call, Base);
     Call.prototype.children = ['variable', 'args'];
     Call.prototype.newInstance = function() {
-      this.isNew = true;
+      if (this.variable.base instanceof Call) {
+        this.variable.base.newInstance();
+      } else {
+        this.isNew = true;
+      }
       return this;
     };
     Call.prototype.superReference = function(o) {

src/nodes.coffee

@@ -433,7 +433,10 @@ exports.Call = class Call extends Base
 
   # Tag this invocation as creating a new instance.
   newInstance: ->
-    @isNew = true
+    if @variable.base instanceof Call
+      @variable.base.newInstance()
+    else
+      @isNew = true
     this
 
   # Grab the reference to the superclass's implementation of the current

test/test_functions.coffee

@@ -346,6 +346,7 @@ eq ok, new ->
   ### Should `return` implicitly   ###
   ### even with trailing comments. ###
 
+
 #855: execution context for `func arr...` should be `null`
 (->
   global = @
@@ -355,3 +356,14 @@ eq ok, new ->
   contextTest.apply null, array
   contextTest array...
 )()
+
+
+# #894: Splatting against constructor-chained functions.
+x = null
+
+class Foo
+  bar: (y) -> x = y
+
+new Foo().bar([101]...)
+
+eq x, 101