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

Alex Deucher alexdeucher at gmail.com
Fri Aug 24 09:27:51 PDT 2012


On Fri, Aug 24, 2012 at 8:52 AM, Michel Dänzer <michel at daenzer.net> 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 the series:

Reviewed-by: Alex Deucher <alexander.deucher 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,
> --
> 1.7.10.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list