[Piglit] [PATCH 6/6] Convert some raw glReadPixels to use piglit probes
Matt Turner
mattst88 at gmail.com
Mon Aug 27 11:26:54 PDT 2012
On Sun, Aug 26, 2012 at 12:55 PM, Paul Berry <stereotype441 at gmail.com> wrote:
> Minor nit-pick: Our usual convention is to make "pass" a bool, and to write
> this as "pass = piglit_probe_pixel_rgba(...) && pass", just so that it's
> clear that there aren't any bitwise shenanigans going on. Note that it
> would probably make sense to also change the return type of DoTest(), and
> the type of "pass" in piglit_display(), to bool as well. I won't be a
> stickler about it though.
As it turns out, we've got a good mixture of 'pass = ... && pass',
'pass = pass && ...", and as I learned today 'pass &= ...'. 445, 351,
and 281 instances respectively. Ugh.
More information about the Piglit
mailing list