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

Unified Diff: dm/DM.cpp

Issue 989083002: DM: show skipped and --blacklist notes only in verbose mode. (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dm/DM.cpp
diff --git a/dm/DM.cpp b/dm/DM.cpp
index 4ab5da975ee7785b2085957ab72e114cc4eac7e5..9f3dbaba5eda2ca9489a26683622c5d141e8ff18 100644
--- a/dm/DM.cpp
+++ b/dm/DM.cpp
@@ -333,7 +333,7 @@ struct Task {
task->src.tag,
name.c_str(),
err.c_str()));
- } else {
+ } else if (FLAGS_verbose) {
name.appendf(" (skipped: %s)", err.c_str());
}
done(timer.fWall, task->sink.tag, task->src.tag, name, log);
@@ -378,7 +378,7 @@ struct Task {
}
}
timer.end();
- if (!whyBlacklisted.isEmpty()) {
+ if (FLAGS_verbose && !whyBlacklisted.isEmpty()) {
name.appendf(" (--blacklist, %s)", whyBlacklisted.c_str());
}
done(timer.fWall, task->sink.tag, task->src.tag, name, log);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698