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

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

Committed: http://code.google.com/p/skia/source/detail?r=10255

[bungeman-skia, 2013-07-22 22:43:52]

lgtm

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

My one super nit.

https://codereview.chromium.org/19569012/diff/1/bench/PathIterBench.cpp
File bench/PathIterBench.cpp (right):

https://codereview.chromium.org/19569012/diff/1/bench/PathIterBench.cpp#newco...
bench/PathIterBench.cpp:75: while ((verb = iter.next(pts)) !=
SkPath::kDone_Verb) {}
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]

https://codereview.chromium.org/19569012/diff/1/bench/PathIterBench.cpp
File bench/PathIterBench.cpp (right):

https://codereview.chromium.org/19569012/diff/1/bench/PathIterBench.cpp#newco...
bench/PathIterBench.cpp:75: while ((verb = iter.next(pts)) !=
SkPath::kDone_Verb) {}
On 2013/07/22 22:44:12, bungeman1 wrote:
> super nit: I usually put a space in between, like '{ }' just because I like
the
> way it looks.

Done.

[commit-bot: I haz the power, 2013-07-23 02:05:43]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/mtklein@google.com/19569012/6001

[commit-bot: I haz the power, 2013-07-23 11:14:00]

Change committed as 10255