New version of the bbh shootout

tools/PictureRenderer.h

Index: tools/PictureRenderer.h
diff --git a/tools/PictureRenderer.h b/tools/PictureRenderer.h
index 67392774ba192b421b9b5432474961e5d3598221..d2e9f3bfa3729a915d21c4478fb6aa25817b840b 100644
--- a/tools/PictureRenderer.h
+++ b/tools/PictureRenderer.h
@@ -83,7 +83,10 @@ public:
         kQuadTree_BBoxHierarchyType,
         kRTree_BBoxHierarchyType,
         kTileGrid_BBoxHierarchyType,
+
+        kLast_BBoxHierarchyType = kTileGrid_BBoxHierarchyType,
     };
+    static const int kBBoxHierarchyTypeCount = kLast_BBoxHierarchyType + 1;

[reed1, 2014/03/07 13:39:49]

stylistically, I'm not a fan of these in public headers.

[mtklein, 2014/03/07 13:41:15]

If you can get away with it, I'd recommend using only the Count.  Having
kLast_Thing can make compilers warnings get annoying about non-exhaustive
switches.

[iancottrell, 2014/03/07 14:52:47]

Removed the count.
kLast seemed to be the standard across all of skia, I though all compilers had
been fixed for those exhaustive messages now?

 
     // this uses SkPaint::Flags as a base and adds additional flags
     enum DrawFilterFlags {