Turn on -Wall -Wextra on Mac, and fix all the warnings that crop up for /usr/bin/g++ and Clang 3.3.

bench/PathIterBench.cpp

Index: bench/PathIterBench.cpp
diff --git a/bench/PathIterBench.cpp b/bench/PathIterBench.cpp
index 8bc8add427cd9b9173ed1897df99d2af7b0e29eb..a81c80283bef04a8b35fc656de6c6898ed866297 100644
--- a/bench/PathIterBench.cpp
+++ b/bench/PathIterBench.cpp
@@ -72,7 +72,7 @@ protected:
                 SkPath::Verb verb;
                 SkPoint      pts[4];
 
-                while ((verb = iter.next(pts)) != SkPath::kDone_Verb);
+                while ((verb = iter.next(pts)) != SkPath::kDone_Verb) {}

[bungeman-skia, 2013/07/22 22:44:12]

super nit: I usually put a space in between, like '{ }' just because I like the
way it looks.

[mtklein, 2013/07/22 22:47:33]

Done.

             }
         } else {
             for (int i = 0; i < N; ++i) {
@@ -80,7 +80,7 @@ protected:
                 SkPath::Verb verb;
                 SkPoint      pts[4];
 
-                while ((verb = iter.next(pts)) != SkPath::kDone_Verb);
+                while ((verb = iter.next(pts)) != SkPath::kDone_Verb) {}
             }
         }
     }