Skip to content
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

Wrong lookahead when parsing for loop const variable. #5629

Closed
crelier opened this issue Oct 3, 2012 · 2 comments
Closed

Wrong lookahead when parsing for loop const variable. #5629

crelier opened this issue Oct 3, 2012 · 2 comments
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.
Milestone

Comments

@crelier
Copy link
Contributor

crelier commented Oct 3, 2012

See:

tests/co19/src/Language/11_Statements/05_For/2_For_in_A01_t02.dart
tests/co19/src/Language/11_Statements/05_For/2_For_in_A01_t04.dart

regis@belenos:/dart1/dart/runtime$ out/DebugIA32/dart --ignore-unrecognized-flags /usr/local/google/home/regis/dart1/dart/tests/co19/src/Language/11_Statements/05_For/2_For_in_A01_t02.dart
'file:///usr/local/google/home/regis/dart1/dart/tests/co19/src/Language/11_Statements/05_For/2_For_in_A01_t02.dart': Error: line 32 pos 19: '(' expected
    for (const id in e);
                  ^
regis@belenos:
/dart1/dart/runtime$ out/DebugIA32/dart --ignore-unrecognized-flags /usr/local/google/home/regis/dart1/dart/tests/co19/src/Language/11_Statements/05_For/2_For_in_A01_t04.dart
'file:///usr/local/google/home/regis/dart1/dart/tests/co19/src/Language/11_Statements/05_For/2_For_in_A01_t04.dart': Error: line 38 pos 20: '(' expected
    for ( const id in l ) {
                   ^

@iposva-google
Copy link
Contributor

Added this to the M2 milestone.
Added Accepted label.

@DartBot
Copy link

DartBot commented Oct 4, 2012

This comment was originally written by @mhausner


Fixed this in r13255.

Filed co19 issue #240 (https://code.google.com/p/co19/issues/detail?id=240) since the test is wrong.


Added Fixed label.

@crelier crelier added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. labels Oct 4, 2012
@crelier crelier added this to the M2 milestone Oct 4, 2012
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.
Projects
None yet
Development

No branches or pull requests

3 participants