Use skiplist-based FastCallSet within trace::CallSet

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

index 291b2da9425d71e4e075bec9d66fb049fd018daa..689898512acd75ff1e23b06ce2af2f5b09352d24 100755 (executable)
--- a/scripts/snapdiff.py
+++ b/scripts/snapdiff.py
@@ -154,6 +154,10 @@ def main():
  
      optparser = optparse.OptionParser(
          usage="\n\t%prog [options] <ref_prefix> <src_prefix>")
+    optparser.add_option(
+        '-v', '--verbose',
+        action="store_true", dest="verbose", default=False,
+        help="verbose output")
      optparser.add_option(
          '-o', '--output', metavar='FILE',
          type="string", dest="output", default='index.html',
@@ -170,6 +174,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 +200,45 @@ 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)
-
+            if options.verbose:
+                sys.stdout.write('Comparing %s and %s ...' % (ref_image, src_image))
+            comparer = Comparer(ref_image, src_image, options.alpha)
+            match = comparer.ae(fuzz=options.fuzz) == 0
+            if match:
+                result = 'MATCH'
+                bgcolor = '#20ff20'
+            else:
+                result = 'MISMATCH'
+                failures += 1
+                bgcolor = '#ff2020'
+            if options.verbose:
+                sys.stdout.write(' %s\n' % (result,))
              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()