Skip to content

dartc does not check for _ in named parameters #2493

New issue

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

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

Already on GitHub? # to your account

Closed
DartBot opened this issue Apr 9, 2012 · 6 comments
Closed

dartc does not check for _ in named parameters #2493

DartBot opened this issue Apr 9, 2012 · 6 comments
Labels
legacy-area-analyzer Use area-devexp instead.
Milestone

Comments

@DartBot
Copy link

DartBot commented Apr 9, 2012

This issue was originally filed by zundel@google.com


It is a compile time error if a named formal parameter begins with an _

See co19 test Language/06_Functions/2_Formal_Parameters/2_Named_Optional_Formals_A03_t01

See also issue #2137

@kasperl
Copy link

kasperl commented Apr 17, 2012

Removed Area-Compiler label.
Added Area-Analyzer label.

@danrubel
Copy link

danrubel commented Jun 4, 2012

Added this to the M1 milestone.

@DartBot
Copy link
Author

DartBot commented Jun 7, 2012

This comment was originally written by zundel@google.com


Set owner to zundel@google.com.

@DartBot
Copy link
Author

DartBot commented Jun 11, 2012

This comment was originally written by zundel@google.com


Filed issue #3496 as tthere is at least one method in the dartium library that uses an underscore, causing compiling lots of existing code to break


Marked this as being blocked by #3496.

@DartBot
Copy link
Author

DartBot commented Jun 11, 2012

This comment was originally written by zundel@google.com


https://chromiumcodereview.appspot.com/10541095


Added Started label.

@DartBot
Copy link
Author

DartBot commented Jun 13, 2012

This comment was originally written by zundel@google.com


r8631


Added Fixed label.

@DartBot DartBot added Type-Defect legacy-area-analyzer Use area-devexp instead. labels Jun 13, 2012
@DartBot DartBot added this to the M1 milestone Jun 13, 2012
This issue was closed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
legacy-area-analyzer Use area-devexp instead.
Projects
None yet
Development

No branches or pull requests

3 participants