[Mesa-dev] [PATCH 1/4] radeonsi: Drop extraneous semicolons from pm4 state macro definitions.

Christian König deathsimple at vodafone.de
Fri Aug 24 09:12:49 PDT 2012


On 24.08.2012 14:52, Michel Dänzer wrote:
> From: Michel Dänzer <michel.daenzer at amd.com>
>
> Could cause build failures if trying to use the macros in certain constructs.
>
> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
For this series:

Reviewed-by: Christian König <christian.koenig at amd.com>

> ---
>   src/gallium/drivers/radeonsi/si_state.h |    6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_state.h b/src/gallium/drivers/radeonsi/si_state.h
> index 87b5370..f53ecb7 100644
> --- a/src/gallium/drivers/radeonsi/si_state.h
> +++ b/src/gallium/drivers/radeonsi/si_state.h
> @@ -104,7 +104,7 @@ union si_state {
>   #define si_pm4_bind_state(rctx, member, value) \
>   	do { \
>   		(rctx)->queued.named.member = (value); \
> -	} while(0);
> +	} while(0)
>   
>   #define si_pm4_delete_state(rctx, member, value) \
>   	do { \
> @@ -113,7 +113,7 @@ union si_state {
>   		} \
>   		si_pm4_free_state(rctx, (struct si_pm4_state *)(value), \
>   				  si_pm4_block_idx(member)); \
> -	} while(0);
> +	} while(0)
>   
>   #define si_pm4_set_state(rctx, member, value) \
>   	do { \
> @@ -123,7 +123,7 @@ union si_state {
>   				si_pm4_block_idx(member)); \
>   			(rctx)->queued.named.member = (value); \
>   		} \
> -	} while(0);
> +	} while(0)
>   
>   /* si_state.c */
>   bool si_is_format_supported(struct pipe_screen *screen,



More information about the mesa-dev mailing list