[Piglit] [PATCH] cl-api-create-context-from-type: Remove unused variables.

Tom Stellard tom at stellard.net
Wed Jun 11 12:41:12 PDT 2014


On Sun, Jun 01, 2014 at 08:32:09PM -0700, Vinson Lee wrote:
> This patch fixes these GCC warnings.
> 
> create-context-from-type.c: In function ‘piglit_cl_test’:
> create-context-from-type.c:173:24: warning: variable ‘invalid_platform_context_properties’ set but not used [-Wunused-but-set-variable]
>   cl_context_properties invalid_platform_context_properties[] = {
>                         ^
> create-context-from-type.c:156:13: warning: unused variable ‘cl_ctx’ [-Wunused-variable]
>   cl_context cl_ctx;
>              ^
> create-context-from-type.c:155:9: warning: unused variable ‘errNo’ [-Wunused-variable]
>   cl_int errNo;
>          ^
> 
Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  tests/cl/api/create-context-from-type.c | 8 --------
>  1 file changed, 8 deletions(-)
> 
> diff --git a/tests/cl/api/create-context-from-type.c b/tests/cl/api/create-context-from-type.c
> index 9c31fff..46095d6 100644
> --- a/tests/cl/api/create-context-from-type.c
> +++ b/tests/cl/api/create-context-from-type.c
> @@ -152,8 +152,6 @@ piglit_cl_test(const int argc,
>  	char test_str[1024];
>  
>  	int i, mask;
> -	cl_int errNo;
> -	cl_context cl_ctx;
>  	cl_device_type mixed_device_types;
>  
>  	bool found_invalid_platform = false;
> @@ -170,10 +168,6 @@ piglit_cl_test(const int argc,
>  		CL_DEVICE_NAME, (cl_context_properties)env->platform_id,
>  		0
>  	};
> -	cl_context_properties invalid_platform_context_properties[] = {
> -		CL_CONTEXT_PLATFORM, (cl_context_properties)NULL,
> -		0
> -	};
>  	cl_context_properties multiple_platform_context_properties[] = {
>  		CL_CONTEXT_PLATFORM, (cl_context_properties)env->platform_id,
>  		CL_CONTEXT_PLATFORM, (cl_context_properties)env->platform_id,
> @@ -197,8 +191,6 @@ piglit_cl_test(const int argc,
>  		}
>  	}
>  	free(platform_ids);
> -	invalid_platform_context_properties[1] =
> -		(cl_context_properties)invalid_platform_id;
>  
>  	/*** Normal usage ***/
>  
> -- 
> 1.8.3.2
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list