apitrace diff-images: Print one line for each file being compared.

[apitrace] / scripts / snapdiff.py
diff --git a/scripts/snapdiff.py b/scripts/snapdiff.py

index 291b2da9425d71e4e075bec9d66fb049fd018daa..7ae3a433fa125724f83c82548bbff4da9e7886c8 100755 (executable)
--- a/scripts/snapdiff.py
+++ b/scripts/snapdiff.py
@@ -170,6 +170,10 @@ def main():
          '--overwrite',
          action="store_true", dest="overwrite", default=False,
          help="overwrite images")
+    optparser.add_option(
+        '--show-all',
+        action="store_true", dest="show_all", default=False,
+        help="show all images, including similar ones")
  
      (options, args) = optparser.parse_args(sys.argv[1:])
  
@@ -192,31 +196,40 @@ def main():
      html.write('  <body>\n')
      html.write('    <table border="1">\n')
      html.write('      <tr><th>File</th><th>%s</th><th>%s</th><th>&Delta;</th></tr>\n' % (ref_prefix, src_prefix))
+    failures = 0
      for image in images:
          ref_image = ref_prefix + image
          src_image = src_prefix + image
          root, ext = os.path.splitext(src_image)
          delta_image = "%s.diff.png" % (root, )
          if os.path.exists(ref_image) and os.path.exists(src_image):
-            if options.overwrite \
-               or not os.path.exists(delta_image) \
-               or (os.path.getmtime(delta_image) < os.path.getmtime(ref_image) \
-                   and os.path.getmtime(delta_image) < os.path.getmtime(src_image)):
-
-                comparer = Comparer(ref_image, src_image, options.alpha)
-                comparer.write_diff(delta_image, fuzz=options.fuzz)
-
+            print "Comparing %s and %s\n" % (ref_image, src_image)
+            comparer = Comparer(ref_image, src_image, options.alpha)
+            match = comparer.ae(fuzz=options.fuzz) == 0
+            if match:
+                bgcolor = '#20ff20'
+            else:
+                failures += 1
+                bgcolor = '#ff2020'
              html.write('      <tr>\n')
-            html.write('        <td>%s</td>\n' % (image,))
-            surface(html, ref_image)
-            surface(html, src_image)
-            surface(html, delta_image)
+            html.write('        <td bgcolor="%s"><a href="%s">%s<a/></td>\n' % (bgcolor, ref_image, image))
+            if not match or options.show_all:
+                if options.overwrite \
+                   or not os.path.exists(delta_image) \
+                   or (os.path.getmtime(delta_image) < os.path.getmtime(ref_image) \
+                       and os.path.getmtime(delta_image) < os.path.getmtime(src_image)):
+                    comparer.write_diff(delta_image, fuzz=options.fuzz)
+                surface(html, ref_image)
+                surface(html, src_image)
+                surface(html, delta_image)
              html.write('      </tr>\n')
              html.flush()
      html.write('    </table>\n')
      html.write('  </body>\n')
      html.write('</html>\n')
  
+    if failures:
+        sys.exit(1)
  
  if __name__ == '__main__':
      main()