Const variable declarations accepted in the parser.

Const variable declarations accepted in the parser.

TEST=language/const_var_test.dart

Committed: https://code.google.com/p/dart/source/detail?r=10731

[Johnni Winther, 2012-08-15 12:58:21]

http://codereview.chromium.org/10829334/diff/1/lib/compiler/implementation/sc...
File lib/compiler/implementation/scanner/parser.dart (right):

http://codereview.chromium.org/10829334/diff/1/lib/compiler/implementation/sc...
lib/compiler/implementation/scanner/parser.dart:834: Token
peekIdentifierAfterTypeOpt(Token token) {
Rename to peekIdentifierAfterOptionalType

http://codereview.chromium.org/10829334/diff/1/lib/compiler/implementation/sc...
lib/compiler/implementation/scanner/parser.dart:898: // We are looking at "const
type identifier" followed by '=', ';', ','.
';', or ','

http://codereview.chromium.org/10829334/diff/1/tests/language/const_var_test....
File tests/language/const_var_test.dart (right):

http://codereview.chromium.org/10829334/diff/1/tests/language/const_var_test....
tests/language/const_var_test.dart:14: }
Add:
 const []
 const {}
 const <int>[]
 const <int>{}
 const List()
 const List<int>()
 const foo.List()
 const foo.List<int>()

[ahe, 2012-08-15 12:59:14]

LGTM if you address the comments.

[Johnni Winther, 2012-08-15 13:59:18]

http://codereview.chromium.org/10829334/diff/1/lib/compiler/implementation/sc...
File lib/compiler/implementation/scanner/parser.dart (right):

http://codereview.chromium.org/10829334/diff/1/lib/compiler/implementation/sc...
lib/compiler/implementation/scanner/parser.dart:834: Token
peekIdentifierAfterTypeOpt(Token token) {
On 2012/08/15 12:58:21, Johnni Winther wrote:
> Rename to peekIdentifierAfterOptionalType

Done.

http://codereview.chromium.org/10829334/diff/1/lib/compiler/implementation/sc...
lib/compiler/implementation/scanner/parser.dart:898: // We are looking at "const
type identifier" followed by '=', ';', ','.
On 2012/08/15 12:58:21, Johnni Winther wrote:
> ';', or ','

Done.

http://codereview.chromium.org/10829334/diff/1/tests/language/const_var_test....
File tests/language/const_var_test.dart (right):

http://codereview.chromium.org/10829334/diff/1/tests/language/const_var_test....
tests/language/const_var_test.dart:14: }
On 2012/08/15 12:58:21, Johnni Winther wrote:
> Add:
>  const []
>  const {}
>  const <int>[]
>  const <int>{}
>  const List()
>  const List<int>()
>  const foo.List()
>  const foo.List<int>()

Done.