[Piglit] [PATCH] arb_internalformat_query2: Set expected_value in default case.

Brian Paul brianp at vmware.com
Tue Mar 8 01:36:17 UTC 2016


On 03/07/2016 06:02 PM, Vinson Lee wrote:
> Fixes uninitialized scalar defect reported by Coverity.
>
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>   tests/spec/arb_internalformat_query2/format-components.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/tests/spec/arb_internalformat_query2/format-components.c b/tests/spec/arb_internalformat_query2/format-components.c
> index 10d7f5258108..4e89a7b7b152 100644
> --- a/tests/spec/arb_internalformat_query2/format-components.c
> +++ b/tests/spec/arb_internalformat_query2/format-components.c
> @@ -218,6 +218,7 @@ try(const GLenum *targets, unsigned num_targets,
>                                           break;
>                                   default:
>                                           assert("incorrect pname for test");

That assertion can't be right.  The intention was probably !"incorrect 
pname for test"

Maybe you can fix that too.

-Brian


> +                                        expected_value = 0;
>                                           break;
>                                   }
>
>



More information about the Piglit mailing list