Fix fuzzer's bools to be 0 or 1 only

fuzz/FuzzGradients.cpp

Index: fuzz/FuzzGradients.cpp
diff --git a/fuzz/FuzzGradients.cpp b/fuzz/FuzzGradients.cpp
index d24bdbe0579a1a4fba23559bb2b39679945c3bd4..98026b3e0da21b0c5c9c7d9f26a43567bd1539d9 100644
--- a/fuzz/FuzzGradients.cpp
+++ b/fuzz/FuzzGradients.cpp
@@ -80,8 +80,8 @@ void fuzzLinearGradient(Fuzz* fuzz) {
                 !fuzz->next<SkScalar>(&b)          ||
                 !fuzz->next<SkScalar>(&c)          ||
                 !fuzz->next<SkScalar>(&d)          ||
-                !fuzz->next<bool>(&useLocalMatrix) ||
-                !fuzz->next<bool>(&useGlobalMatrix)) {
+                !fuzz->next(&useLocalMatrix) ||

[mtklein_C, 2016/10/24 17:45:10]

This code reads inconsistently, before this CL and more now.  If we're going to
line up ||, let's line them all up, and while we're at it, line up all the
!fuzz->next too.

Again, it'd be nice to either always use the type or never use the type when
calling next().  I'd recommend never using the type... it can be inferred in
every single one of these cases.

If you want to use the type, instead of overloading next(T*) with next(bool*),
we can make an explicit template function specialization.  It'd work very
similarly except that next<bool>(...) would pick up the specialized version. 
The downside is that there are some more restrictions on where you can define
it, so let's not get into that unless you really like the <Type> all over the
place.

[kjlubick, 2016/10/24 18:27:01]

All the types removed and the || are lined up as well as !fuzz->next

+                !fuzz->next(&useGlobalMatrix)) {
             return;
         }
         SkPoint pts[2] = {SkPoint::Make(a,b), SkPoint::Make(c, d)};
@@ -127,8 +127,8 @@ void fuzzRadialGradient(Fuzz* fuzz) {
         if (!fuzz->next<SkScalar>(&a)          ||
                 !fuzz->next<SkScalar>(&b)          ||
                 !fuzz->next<SkScalar>(&radius)     ||
-                !fuzz->next<bool>(&useLocalMatrix) ||
-                !fuzz->next<bool>(&useGlobalMatrix)) {
+                !fuzz->next(&useLocalMatrix) ||
+                !fuzz->next(&useGlobalMatrix)) {
             return;
         }
         SkPoint center = SkPoint::Make(a,b);
@@ -178,8 +178,8 @@ void fuzzTwoPointConicalGradient(Fuzz* fuzz) {
                 !fuzz->next<SkScalar>(&c)           ||
                 !fuzz->next<SkScalar>(&d)           ||
                 !fuzz->next<SkScalar>(&endRadius)   ||
-                !fuzz->next<bool>(&useLocalMatrix)  ||
-                !fuzz->next<bool>(&useGlobalMatrix)) {
+                !fuzz->next(&useLocalMatrix)  ||
+                !fuzz->next(&useGlobalMatrix)) {
             return;
         }
         SkPoint start = SkPoint::Make(a, b);
@@ -225,8 +225,8 @@ void fuzzSweepGradient(Fuzz* fuzz) {
         bool useLocalMatrix, useGlobalMatrix;
         if (!fuzz->next<SkScalar>(&cx)         ||
                 !fuzz->next<SkScalar>(&cy)         ||
-                !fuzz->next<bool>(&useLocalMatrix) ||
-                !fuzz->next<bool>(&useGlobalMatrix)) {
+                !fuzz->next(&useLocalMatrix) ||
+                !fuzz->next(&useGlobalMatrix)) {
             return;
         }