[Piglit] [PATCH] GL 1.50: Test that UNIFORM_BLOCK_REFERENCED_BY_GEOMETRY_SHADER is accepted by GetActiveUniformBlockiv()

Eric Anholt eric at anholt.net
Tue Oct 8 02:25:50 CEST 2013


Nicholas Mack <nichmack at gmail.com> writes:

> v2: Incorporate this test into referenced-by-shader.c

> @@ -40,10 +41,6 @@ PIGLIT_GL_TEST_CONFIG_BEGIN
>  
>  	config.supports_gl_compat_version = 10;
>  
> -	config.window_width = 10;
> -	config.window_height = 10;
> -	config.window_visual = PIGLIT_GL_VISUAL_RGBA | PIGLIT_GL_VISUAL_DOUBLE;
> -

It looks like this won't ever get run on a 3.2 context on Mesa because
you need at least config.supports_gl_core_version = 31.

> +	bool use_gs = piglit_get_gl_version() >= 32;
> +	const char *header;
> +	char *temp_source;
> +	int num_uniforms_used = 0;
> +
> +	if(use_gs) {

There should always be a space between 'if' and '('

> -	for (i = 0; i < 3; i++) {
> -		GLint ref_vs, ref_fs;
> +	if(use_gs) {
> +		num_uniforms_used = 7;
> +		printf("            v g f\n");
> +	}
> +	else {

Usually we cuddle the braces like "} else {"

With the one functional fix and the little style nits fixed, this would
be:

Reviewed-by: Eric Anholt <eric at anholt.net>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20131007/2535f45b/attachment.pgp>


More information about the Piglit mailing list