On 04/14/2016 01:34 PM, Haixia Shi wrote:
> The function GLboolean piglit_check_gl_error() returns GL_TRUE when there is
> no error, and GL_FALSE when there is an error.
> 
> Signed-off-by: Haixia Shi <[email protected]>
> Cc: Dylan Baker <[email protected]>
> Cc: Ian Romanick <[email protected]>
> ---
>  tests/spec/arb_get_program_binary/retrievable_hint.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/spec/arb_get_program_binary/retrievable_hint.c 
> b/tests/spec/arb_get_program_binary/retrievable_hint.c
> index 8283c5b..9263a89 100644
> --- a/tests/spec/arb_get_program_binary/retrievable_hint.c
> +++ b/tests/spec/arb_get_program_binary/retrievable_hint.c
> @@ -79,7 +79,7 @@ piglit_init(int argc, char **argv)
>        */
>       value = 0xDEADBEEF;
>       glGetProgramiv(prog, GL_PROGRAM_BINARY_RETRIEVABLE_HINT, &value);
> -     got_error = piglit_check_gl_error(0);
> +     got_error = !piglit_check_gl_error(0);
>  
>       if (!got_error) {

I think this would be better as:

        if (piglit_check_gl_error(GL_NO_ERROR)) {

end delete the got_error variable altogether.

I'm really not sure why I didn't do it that way to begin with.

>               if (value == 0xDEADBEEF) {
> 

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

Reply via email to