Dart2js fails to resolve optional functional parameter with default value.

[lrhn]

Example:
 foo(x, y, [int z(a,b) = bar]) => z(x,y);
reports:

funcarg.dart:3:12: Internal error: expected optional parameters
foo(x, y, [int z(a,b) = bar]) => z(x,y);
           ^^^^^^^^^^^^^^^^^

The error is triggered in resolver.dart line 2242:

­5 SignatureResolver.analyzeNodes (file:///.../dart/lib/compiler/implementation/resolver.dart:2242:24)

(Dart2js still don't support functions as compile time constants, but this error seems unrelated to that).

[peter-ahe-google]

This is one of the things I'm willing to let slip, but please raise the priority if you disagree.

---

Added this to the M1 milestone.
Removed Priority-Medium label.
Added Priority-Low label.

[kasperl]

Changed the title to: "Dart2js fails to resolve optional functional parameter with default value.".

[floitschG]

Issue #5026 has been merged into this issue.

[kasperl]

Removed this from the M1 milestone.
Added this to the M2 milestone.

[alan-knight]

I ran into this today. I wouldn't suggest trying to squeeze it into M1 at this point, but calling it "Low" doesn't seem right.

[kasperl]

Removed Priority-Low label.
Added Priority-Medium label.

[lrhn]

Marked this as blocking #5870.

[kasperl]

Removed this from the M2 milestone.

[kasperl]

Added this to the M2 milestone.

[peter-ahe-google]

Set owner to @peter-ahe-google.

[anders-sandholm]

Confirmed/reproduced in Dart SDK version 0.2.2.1_r14458

[peter-ahe-google]

Issue #6729 has been merged into this issue.

[peter-ahe-google]

Bumping priority: this is the issue I have with most stars.

---

Removed Priority-Medium label.
Added Priority-High label.

[peter-ahe-google]

Removed this from the M2 milestone.
Added this to the M3 milestone.

[kasperl]

Johnni, if you've got a few cycles maybe you can help Peter with this?

---

cc @johnniwinther.

[peter-ahe-google]

Removed this from the M3 milestone.
Added this to the M4 milestone.

[peter-ahe-google]

Set owner to [email protected].
Added Started label.

[DartBot]

This comment was originally written by @aam

---

Fixed in r19849.

---

Added Fixed label.