JS Null and Undefined objects are not treated correctly in .call and .appl about ironjs HOT 8 CLOSED

That particular version is the "command line version", which has external dependencies on Node.js

The coffee script core compiler: http://jashkenas.github.com/coffee-script/extras/coffee-script.js
Has all of the require'd stuff built in.

Could you try the core compiler version?

from ironjs.

The two urls have the same content, the core compiler.

http://jashkenas.github.com/coffee-script/extras/coffee-script.js
https://github.com/jashkenas/coffee-script/raw/master/extras/coffee-script.js

But I made sure, same result with both versions.

from ironjs.

I'll look into this and see if I can re-produce it over the weekend, to busy with work this week to have time right now, sadly.

from ironjs.

This looks like an issue with the Function.prototype.call method. (Native.Function.fs, lines 131 though 133)

We are generating a delegate to call, based on the types of the parameters passed in.
However, we are always using the CLR value of the parameters.
The CLR Value of a JS undefined object is null, and we are then calling TypeUtils.getType on null, which is causing the exception.

For example (min repro):

var foo = function (a, b, c, d, e, f) { print('bleh'); };
var me = "";
foo.call(me, 1, 'a', undefined, null);

results in:
CLR Values: { 1, "a", null, null, null, null }
CLR Types: { Int32, String, error, ... }

Whereas it should probably result in:
CLR Types: { Int32, String, BoxedValue, BoxedValue, BoxedValue, BoxedValue}

from ironjs.

I'm not 100% sure what the fix should be, here.

It seems to me that the TC.ToClrObject should maybe return the BoxedValue for anything that is not IsClr, but I'm not sure if that is the desired semantics of this method. (i.e., I don't know whether a JS Null should return a CLR null here.)

Otherwise, we need to change line 132 to read:

let argsClr = args |> Array.map (fun i -> if not i.IsNull && i.IsClr then TC.ToClrObject i else i :> obj)

This does, in fact, allow the above example to work.

from ironjs.

I opted to use the latter definition, in order to preserve all previous usages of TC.ToClrObject.

from ironjs.

The fix lets the code run w/o errors, that said, the results aren't correct yet.

Compiling this CoffeeScript fragment foo -> 1 with IronJS gives:

var ;
 foo(function() {
var ;
return 1;
});

but the correct result looks like

foo(function() {
  return 1;
});

from ironjs.

from ironjs.