Index: tools/render_pictures_main.cpp |
diff --git a/tools/render_pictures_main.cpp b/tools/render_pictures_main.cpp |
index f11eb0b80ee05fed4c4c17f9037e0a9b438b5f1a..352dbf38be5357368358b410943d98d3f85c992f 100644 |
--- a/tools/render_pictures_main.cpp |
+++ b/tools/render_pictures_main.cpp |
@@ -35,8 +35,10 @@ |
DEFINE_bool(writeEncodedImages, false, "Any time the skp contains an encoded image, write it to a " |
"file rather than decoding it. Requires writePath to be set. Skips drawing the full " |
"skp to a file. Not compatible with deferImageDecoding."); |
-DEFINE_string(writeJsonSummaryPath, "", "File to write a JSON summary of image results to."); |
-DEFINE_string2(writePath, w, "", "Directory to write the rendered images into."); |
+DEFINE_string(writeJsonSummaryPath, "", "File to write a JSON summary of image results to. " |
+ "TODO(epoger): Currently, this only works if --writePath is also specified. " |
+ "See https://code.google.com/p/skia/issues/detail?id=2043 ."); |
+DEFINE_string2(writePath, w, "", "Directory to write the rendered images."); |
DEFINE_bool(writeWholeImage, false, "In tile mode, write the entire rendered image to a " |
"file, instead of an image for each tile."); |
DEFINE_bool(validate, false, "Verify that the rendered image contains the same pixels as " |
@@ -339,6 +341,8 @@ |
if (FLAGS_writeWholeImage) { |
sk_tools::force_all_opaque(*bitmap); |
+ // TODO(epoger): It would be better for the filename (without outputDir) to be passed in |
+ // here, and used both for the checksum file and writing into outputDir. |
SkString inputFilename, outputPath; |
sk_tools::get_basename(&inputFilename, inputPath); |
sk_tools::make_filepath(&outputPath, *outputDir, inputFilename); |