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

Issue 1574213005: Report missing semicolons after function declarations (Closed)

Created:
4 years, 11 months ago by hausner
Modified:
4 years, 11 months ago
Reviewers:
regis
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Report missing semicolons after function declarations Also take the opportunity to add the ability to report an error after a given token position. This way, the missing semicolon gets reported where it should: at the end of the line rather than the first token on the following line, when the error is actually detected. BUG=23761 R=regis@google.com Committed: https://github.com/dart-lang/sdk/commit/782f7b268bda32e907714492ac50b706493027d3

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+100 lines, -31 lines) Patch
M runtime/vm/class_finalizer.cc View 1 2 3 chunks +4 lines, -3 lines 0 comments Download
M runtime/vm/dart_api_impl_test.cc View 1 1 chunk +3 lines, -3 lines 0 comments Download
M runtime/vm/flow_graph_builder.cc View 1 1 chunk +1 line, -0 lines 0 comments Download
M runtime/vm/flow_graph_compiler.cc View 1 1 chunk +1 line, -0 lines 0 comments Download
M runtime/vm/intermediate_language.cc View 1 1 chunk +1 line, -0 lines 0 comments Download
M runtime/vm/object.h View 1 2 chunks +6 lines, -1 line 0 comments Download
M runtime/vm/object.cc View 1 2 11 chunks +17 lines, -1 line 0 comments Download
M runtime/vm/parser.h View 5 chunks +6 lines, -1 line 0 comments Download
M runtime/vm/parser.cc View 1 2 12 chunks +31 lines, -10 lines 0 comments Download
M runtime/vm/raw_object.h View 1 1 chunk +1 line, -0 lines 0 comments Download
M runtime/vm/raw_object_snapshot.cc View 1 2 chunks +2 lines, -0 lines 0 comments Download
M runtime/vm/report.h View 1 3 chunks +7 lines, -1 line 0 comments Download
M runtime/vm/report.cc View 1 2 7 chunks +16 lines, -9 lines 0 comments Download
M runtime/vm/report_test.cc View 1 2 chunks +4 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (4 generated)
hausner
I am not terribly happy about the naming of the new parameters. I want to ...
4 years, 11 months ago (2016-01-13 21:16:57 UTC) #3
regis
lgtm You could maybe declare 2 boolean constants in Token class: Token::kPointAt (=false) Token::kPointAfter (=true) ...
4 years, 11 months ago (2016-01-13 21:57:11 UTC) #4
hausner
4 years, 11 months ago (2016-01-19 17:31:38 UTC) #7
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
782f7b268bda32e907714492ac50b706493027d3 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698