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

Unified Diff: bench/benchmain.cpp

Issue 47513017: More Windows 64b compilation warning fixes (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Fix linux & mac builds Created 7 years, 1 month 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 | include/core/SkUtils.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: bench/benchmain.cpp
===================================================================
--- bench/benchmain.cpp (revision 12322)
+++ bench/benchmain.cpp (working copy)
@@ -450,13 +450,13 @@
// Find the longest name of the benches we're going to run to make the output pretty.
Iter names;
SkBenchmark* bench;
- int longestName = 0;
+ size_t longestName = 0;
while ((bench = names.next()) != NULL) {
SkAutoTUnref<SkBenchmark> benchUnref(bench);
if (SkCommandLineFlags::ShouldSkip(FLAGS_match, bench->getName())) {
continue;
}
- const int length = strlen(bench->getName());
+ const size_t length = strlen(bench->getName());
longestName = length > longestName ? length : longestName;
}
@@ -546,7 +546,7 @@
loggedBenchName = true;
SkString str;
str.printf("running bench [%3d %3d] %*s ",
- dim.fX, dim.fY, longestName, bench->getName());
+ dim.fX, dim.fY, (int)longestName, bench->getName());
logger.logProgress(str);
}
« no previous file with comments | « no previous file | include/core/SkUtils.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698