Skip the TIMESTAMP file.

tools/skimage_main.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "gm_expectations.h"
 #include "SkBitmap.h"
 #include "SkColorPriv.h"
@@ skipping 471 matching lines @@
  *  If directory is non null and does not end with a path separator, append one.
  *  @param directory SkString representing the path to a directory. If the last character is not a
  *      path separator (specific to the current OS), append one.
  */
 static void append_path_separator_if_necessary(SkString* directory) {
     if (directory != NULL && directory->c_str()[directory->size() - 1] != SkPATH_SEPARATOR) {
         directory->appendf("%c", SkPATH_SEPARATOR);
     }
 }
 
+/**
+ *  Return true if the filename represents an image.
+ */
+static bool is_image_file(const char* filename) {
+    const char* gImageExtensions[] = {
+        ".png", ".PNG", ".jpg", ".JPG", ".jpeg", ".JPEG", ".bmp", ".BMP",
+        ".webp", ".WEBP", ".ico", ".ICO", ".wbmp", ".WBMP", ".gif", ".GIF"
+    };

[borenet, 2013/06/28 20:02:02]

I agree that this is cumbersome, but I think it's better than listing the
timestamp file by name.

+    for (size_t i = 0; i < SK_ARRAY_COUNT(gImageExtensions); ++i) {
+        if (SkStrEndsWith(filename, gImageExtensions[i])) {
+            return true;
+        }
+    }
+    return false;
+}
+
 int tool_main(int argc, char** argv);
 int tool_main(int argc, char** argv) {
     SkCommandLineFlags::SetUsage("Decode files, and optionally write the results to files.");
     SkCommandLineFlags::Parse(argc, argv);
 
     if (FLAGS_readPath.count() < 1) {
         SkDebugf("Folder(s) or image(s) to decode are required.\n");
         return -1;
     }
 
@@ skipping 19 matching lines @@
     for (int i = 0; i < FLAGS_readPath.count(); i++) {
         const char* readPath = FLAGS_readPath[i];
         if (strlen(readPath) < 1) {
             break;
         }
         if (sk_isdir(readPath)) {
             const char* dir = readPath;
             SkOSFile::Iter iter(dir);
             SkString filename;
             while (iter.next(&filename)) {
+                if (!is_image_file(filename.c_str())) {
+                    continue;
+                }
                 SkString fullname = SkOSPath::SkPathJoin(dir, filename.c_str());
                 decodeFileAndWrite(fullname.c_str(), outDirPtr);
             }
-        } else if (sk_exists(readPath)) {
+        } else if (sk_exists(readPath) && is_image_file(readPath)) {
             decodeFileAndWrite(readPath, outDirPtr);
         }
     }
 
     if (!FLAGS_createExpectationsPath.isEmpty()) {
         // Use an empty value for everything besides expectations, since the reader only cares
         // about the expectations.
         Json::Value nullValue;
         Json::Value root = skiagm::CreateJsonTree(gExpectationsToWrite, nullValue, nullValue,
                                                   nullValue, nullValue);
@@ skipping 16 matching lines @@
     }
 
     return failed ? -1 : 0;
 }
 
 #if !defined SK_BUILD_FOR_IOS
 int main(int argc, char * const argv[]) {
     return tool_main(argc, (char**) argv);
 }
 #endif