[Mesa-dev] [PATCH] i965: make brw_context::num_samples unsigned int

Jason Ekstrand jason at jlekstrand.net
Fri Jan 19 16:17:26 UTC 2018


Why?  I think it's probably fine but changing signedness of a variable is 
always a bit tricky.


On January 19, 2018 08:04:24 Emil Velikov <emil.l.velikov at gmail.com> wrote:

> From: Emil Velikov <emil.velikov at collabora.com>
>
> Already used as such through the codebase.
>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
>  src/mesa/drivers/dri/i965/brw_context.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_context.h 
> b/src/mesa/drivers/dri/i965/brw_context.h
> index 79e9f49a389..574a0d9d860 100644
> --- a/src/mesa/drivers/dri/i965/brw_context.h
> +++ b/src/mesa/drivers/dri/i965/brw_context.h
> @@ -977,7 +977,7 @@ struct brw_context
>      * Number of samples in ctx->DrawBuffer, updated by BRW_NEW_NUM_SAMPLES so
>      * that we don't have to reemit that state every time we change FBOs.
>      */
> -   int num_samples;
> +   unsigned int num_samples;
>
>     /* BRW_NEW_URB_ALLOCATIONS:
>      */
> --
> 2.15.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev




More information about the mesa-dev mailing list