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

Issue 1182323009: Zero warnings from the analyzer! (Closed)

Created:
5 years, 6 months ago by Hixie
Modified:
5 years, 6 months ago
Reviewers:
abarth-chromium
CC:
abarth-chromium, gregsimon, jackson_old, mojo-reviews_chromium.org, qsr+mojo_chromium.org
Base URL:
https://github.com/domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Zero warnings from the analyzer! Ok, not really zero. Actually 72 if you include warnings from non-sky packages and if you include bogus warnings that can be filtered out. If you pipe the analyzer's output through the following perl script, though, you get zero warnings on the stock app with this patch: while (defined($_ = <STDIN>)) { s|out/android_Debug/gen/dart-pkg/sky/|sky/sdk/|gos; next if m|/dart-pkg/|os; # other packages next if m|^\[warning] Missing concrete implementation of 'RenderObject.toString'|os; # https://github.com/dart-lang/sdk/issues/23606 print; } R=abarth@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/a0841841cf01b51f58afa4df3e25a795c11a5063

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -12 lines) Patch
M sky/sdk/lib/rendering/box.dart View 1 chunk +1 line, -1 line 0 comments Download
M sky/sdk/lib/widgets/ink_well.dart View 1 chunk +9 lines, -8 lines 0 comments Download
M sky/sdk/lib/widgets/widget.dart View 2 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (1 generated)
abarth-chromium
lgtm
5 years, 6 months ago (2015-06-18 20:58:35 UTC) #2
Hixie
5 years, 6 months ago (2015-06-18 23:11:33 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
a0841841cf01b51f58afa4df3e25a795c11a5063 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698