Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(162)

Issue 10690106: Error produced for missing var, final, const or type on field declarations. (Closed)

Created:
8 years, 5 months ago by Johnni Winther
Modified:
8 years, 4 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Error produced for missing var, final, const or type on field declarations. BUG=2997 TEST=tests/language/field_decl_missing_var_type_test Committed: https://code.google.com/p/dart/source/detail?r=9585

Patch Set 1 #

Total comments: 2

Patch Set 2 : Updated cf. comments. #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -1 line) Patch
M lib/compiler/implementation/scanner/listener.dart View 1 1 chunk +8 lines, -0 lines 1 comment Download
M tests/co19/co19-leg.status View 1 1 chunk +0 lines, -1 line 0 comments Download
A tests/language/field_decl_missing_var_type_test.dart View 1 chunk +15 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Johnni Winther
8 years, 5 months ago (2012-07-09 11:42:18 UTC) #1
Lasse Reichstein Nielsen
lgtm https://chromiumcodereview.appspot.com/10690106/diff/1/tests/co19/co19-leg.status File tests/co19/co19-leg.status (right): https://chromiumcodereview.appspot.com/10690106/diff/1/tests/co19/co19-leg.status#newcode521 tests/co19/co19-leg.status:521: #Language/07_Classes/5_Constructors/1_Generative_Constructors_A01_t02: Fail, OK # @compile-error Remove the line ...
8 years, 5 months ago (2012-07-09 11:49:42 UTC) #2
Johnni Winther
https://chromiumcodereview.appspot.com/10690106/diff/1/tests/co19/co19-leg.status File tests/co19/co19-leg.status (right): https://chromiumcodereview.appspot.com/10690106/diff/1/tests/co19/co19-leg.status#newcode521 tests/co19/co19-leg.status:521: #Language/07_Classes/5_Constructors/1_Generative_Constructors_A01_t02: Fail, OK # @compile-error On 2012/07/09 11:49:42, Lasse ...
8 years, 5 months ago (2012-07-12 09:22:28 UTC) #3
ahe
8 years, 4 months ago (2012-08-02 19:40:24 UTC) #4
Hi Johnni,

Thank you for reducing my critical bug count!

I have reopened the bug so we don't forget the change I suggest, but now it is a
low priority thing.

Cheers,
Peter

https://chromiumcodereview.appspot.com/10690106/diff/4002/lib/compiler/implem...
File lib/compiler/implementation/scanner/listener.dart (right):

https://chromiumcodereview.appspot.com/10690106/diff/4002/lib/compiler/implem...
lib/compiler/implementation/scanner/listener.dart:1533: (p) =>
p.parseVariablesDeclaration(beginToken));
I think we should rework this change to a "recoverable" error generated by the
parser.

Powered by Google App Engine
This is Rietveld 408576698