Array.forEach facade produces invalid code

[mhevery]

var a = [];
a.forEach((k, v) => { stmt1(); });

produces in dart

a.forEach((k, v) => { stmt1(); });

this is incorrect, since => and {} can not be used together. Valid outputs would either of these:

a.forEach((k, v) { stmt1(); });
a.forEach((k, v) => stmt1();});

[vicb]

Not sure what's going wrong here, there is no special case for this one (i.e. any arrow function should be translated by ts2d whatever the context).

But you probably want to have a single argument for the cb - the item.

[jsayol]

Just FYI, ts2dart already produces the correct output now ( a.forEach((k, v) { stmt1(); }); ) so this issue can be closed.

[mprobst]

Thanks for pointing this out.