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

Issue 35543006: Fix type errors detected with --error-on-bad-type. (Closed)

Created:
7 years, 2 months ago by Siggi Cherem (dart-lang)
Modified:
7 years, 2 months ago
Reviewers:
nweiz, regis
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

Fix type errors detected with --error-on-bad-type. R=nweiz@google.com, regis@google.com Committed: https://code.google.com/p/dart/source/detail?r=29047

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -2 lines) Patch
M pkg/barback/lib/src/barback.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/barback/lib/src/phase_output.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/barback/lib/src/transform.dart View 1 chunk +2 lines, -0 lines 2 comments Download
M pkg/barback/lib/src/transform_node.dart View 1 chunk +1 line, -0 lines 0 comments Download
M runtime/lib/collection_patch.dart View 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/_internal/pub/lib/src/barback/load_transformers.dart View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Siggi Cherem (dart-lang)
I had no idea about this flag, it's really nice. Nathan, most changes are in ...
7 years, 2 months ago (2013-10-22 22:59:21 UTC) #1
regis
LGTM Did you try --error_on_bad_override? It reports an error as well.
7 years, 2 months ago (2013-10-22 23:03:41 UTC) #2
nweiz
One comment, otherwise lgtm. https://codereview.chromium.org/35543006/diff/1/pkg/barback/lib/src/transform.dart File pkg/barback/lib/src/transform.dart (right): https://codereview.chromium.org/35543006/diff/1/pkg/barback/lib/src/transform.dart#newcode9 pkg/barback/lib/src/transform.dart:9: import 'package:source_maps/span.dart' show Span; Style ...
7 years, 2 months ago (2013-10-22 23:18:23 UTC) #3
Siggi Cherem (dart-lang)
Committed patchset #1 manually as r29047 (presubmit successful).
7 years, 2 months ago (2013-10-22 23:41:13 UTC) #4
Siggi Cherem (dart-lang)
7 years, 2 months ago (2013-10-22 23:41:18 UTC) #5
Message was sent while issue was closed.
https://codereview.chromium.org/35543006/diff/1/pkg/barback/lib/src/transform...
File pkg/barback/lib/src/transform.dart (right):

https://codereview.chromium.org/35543006/diff/1/pkg/barback/lib/src/transform...
pkg/barback/lib/src/transform.dart:9: import 'package:source_maps/span.dart'
show Span;
On 2013/10/22 23:18:23, nweiz wrote:
> Style nit: add a newline before this. Also, we tend not to use "show" unless
> it's specifically necessary in a file. Same goes fro transform_node.dart.

Done.

Powered by Google App Engine
This is Rietveld 408576698