Chromium Code Reviews
DescriptionZero 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 #
Messages
Total messages: 3 (1 generated)
|
|||||||||||||||||||||||||||||||||||||