[Mesa-dev] [PATCH 11/14] radeonsi: Improve assert info out of si_set_framebuffer_state()

Marek Olšák maraeo at gmail.com
Sun Mar 27 17:19:26 UTC 2016


On Sun, Mar 27, 2016 at 4:38 AM, Edward O'Callaghan
<eocallaghan at alterapraxis.com> wrote:
> Lets give the developer a little hand if we are going to assert
> on a zero literal at the end of a branch.
>
> Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
> ---
>  src/gallium/drivers/radeonsi/si_state.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c
> index 8b6144a..72f9e0c 100644
> --- a/src/gallium/drivers/radeonsi/si_state.c
> +++ b/src/gallium/drivers/radeonsi/si_state.c
> @@ -2623,6 +2623,8 @@ static void si_set_framebuffer_state(struct pipe_context *ctx,
>                         constbuf.user_buffer = sctx->b.sample_locations_16x;
>                         break;
>                 default:
> +                       R600_ERR("Requested a invalid number of samples %i.\n",

"an invalid"

With that fixed:
Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek


More information about the mesa-dev mailing list