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

Issue 1813973002: Don't upgrade the severity of *everything* to an error in strong mode. (Closed)

Created:
4 years, 9 months ago by Bob Nystrom
Modified:
4 years, 9 months ago
CC:
reviews_dartlang.org
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Don't upgrade the severity of *everything* to an error in strong mode. Oops! Fixes #26026. R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/a602585097aec147fc5cd3cbf93fc26b38b79dbf

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -9 lines) Patch
M pkg/analyzer/lib/source/error_processor.dart View 1 chunk +6 lines, -8 lines 2 comments Download
M pkg/analyzer/test/source/error_processor_test.dart View 2 chunks +14 lines, -1 line 0 comments Download

Messages

Total messages: 8 (3 generated)
Bob Nystrom
4 years, 9 months ago (2016-03-17 19:48:26 UTC) #3
Brian Wilkerson
LGTM https://codereview.chromium.org/1813973002/diff/1/pkg/analyzer/lib/source/error_processor.dart File pkg/analyzer/lib/source/error_processor.dart (right): https://codereview.chromium.org/1813973002/diff/1/pkg/analyzer/lib/source/error_processor.dart#newcode103 pkg/analyzer/lib/source/error_processor.dart:103: processors = processors.toList(); It looks like it should ...
4 years, 9 months ago (2016-03-17 19:54:14 UTC) #4
Bob Nystrom
https://codereview.chromium.org/1813973002/diff/1/pkg/analyzer/lib/source/error_processor.dart File pkg/analyzer/lib/source/error_processor.dart (right): https://codereview.chromium.org/1813973002/diff/1/pkg/analyzer/lib/source/error_processor.dart#newcode103 pkg/analyzer/lib/source/error_processor.dart:103: processors = processors.toList(); On 2016/03/17 19:54:14, Brian Wilkerson wrote: ...
4 years, 9 months ago (2016-03-17 19:58:25 UTC) #5
Bob Nystrom
Committed patchset #1 (id:1) manually as a602585097aec147fc5cd3cbf93fc26b38b79dbf (presubmit successful).
4 years, 9 months ago (2016-03-17 19:58:31 UTC) #7
pquitslund
4 years, 9 months ago (2016-03-17 20:16:41 UTC) #8
Message was sent while issue was closed.
Late to the party, but LGTM!

Powered by Google App Engine
This is Rietveld 408576698