Execution context for `func arr...`

[TrevorBurnham]

Currently on master, func arr... compiles to func.apply(func, arr). In 0.9.4, it was func.apply(this, arr). However, I believe that the proper output should be func.apply(null, arr), as this example illustrates:

arr = [1, 2, 3]
`property = 'huh'; // global`
func = -> console.log @property
func arr               # 'huh'
func.apply(null, arr)  # 'huh
func arr...            # undefined

[satyr]

Agreed.

[michaelficarra]

agreed, +1

[devongovett]

This one screwed up my code when I upgraded to 0.9.5. I had been taking advantage of the fact that it set this as the function context. The problem with it being func is that if the function you are calling is a class, the context should be that class. The problem with this is that if you aren't calling the function from within the same class or object, you will get an unexpected context. Thus, I agree that null should be the expected context in this case since there is no way to get the real desired context anyway.

[michaelficarra]

Wanted to note: I've opened pull request 876, which fixes this.

[michaelficarra]

closed by 63cbb64; fix partially inspired by satyr/coco 3e37cf32