Skip to content

Optional-named params do not work with function expressions. #697

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
DartBot opened this issue Dec 5, 2011 · 2 comments
Closed

Optional-named params do not work with function expressions. #697

DartBot opened this issue Dec 5, 2011 · 2 comments
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-cannot-reproduce Closed as we were unable to reproduce the reported issue

Comments

@DartBot
Copy link

DartBot commented Dec 5, 2011

This issue was originally filed by [email protected]


The following error occurs in the VM and cannot be replicated on http://try.dartlang.org/

The following code emits this error (unexpected token ':'):

void f([num x = 0, num y = 0]) {}
main() => f(x:12, y:13);

This variant works just fine:

void f([num x = 0, num y = 0]) {}
main() {
  f(x:12, y:13);
}

@DartBot
Copy link
Author

DartBot commented Dec 6, 2011

This comment was originally written by [email protected]


Added Triaged label.

@iposva-google
Copy link
Contributor

Cannot reproduce any longer.


Added CannotReproduce label.

@DartBot DartBot added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-cannot-reproduce Closed as we were unable to reproduce the reported issue labels Mar 13, 2012
copybara-service bot pushed a commit that referenced this issue Jun 1, 2022
Changes:
```
> git log --format="%C(auto) %h %s" 2c9b418..2993ea5
 https://dart.googlesource.com/http.git/+/2993ea5 Implement the ability to run a particular Client implementation in a Zone (#697)
 https://dart.googlesource.com/http.git/+/fa0af10 Fix head, get, post, etc. links in Client docs (#703)
 https://dart.googlesource.com/http.git/+/6b3e3f3 Use spawnHybrid for the stub server (#700)
 https://dart.googlesource.com/http.git/+/74a4371 Separate the `post` description for "String" (#695)
 https://dart.googlesource.com/http.git/+/87d4379 Switch from homepage to repository in pubspec  (#687)

```

Diff: https://dart.googlesource.com/http.git/+/2c9b418f5086f999c150d18172d2eec1f963de7b~..2993ea5dff5ffb066b4a35c707e7a2b8dcfa17c2/
Change-Id: I121641dc9f75d8ccb6d62fd9baec20b951bdbcde
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/246761
Commit-Queue: Devon Carew <[email protected]>
Reviewed-by: Nate Bosch <[email protected]>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. closed-cannot-reproduce Closed as we were unable to reproduce the reported issue
Projects
None yet
Development

No branches or pull requests

2 participants