Chad Versace <[email protected]> writes:

> +static void
> +validate_supported_apis(const struct piglit_gl_test_config *test_config)
> +{
> +     int supported_apis = !!test_config->supports_gl_core_version
> +                        + !!test_config->supports_gl_compat_version
> +                        + !!test_config->supports_gl_es1
> +                        + !!test_config->supports_gl_es2;
> +
> +     if (supported_apis == 0) {

That's a very strange way of saying

       if (!test_config->supports_gl_core_version &&
           !test_config->supports_gl_compat_version &&
           !test_config->supports_gl_es1 &&
           !test_config->supports_gl_es2) {

:P

So, I've scanned through the series.  Other than the malloc wrappers,
it gets my acked-by.

Attachment: pgp5pPLSJEgeO.pgp
Description: PGP signature

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to