On 2 August 2016 at 17:22, Eric Engestrom <eric.engest...@imgtec.com> wrote:
> This field was added in 2d94601582e4f0fcaf8c ("Add DRI3+Present loader"),
> which
> was a v2. Guessing it was used in v1 and Keith forgot to remove it in v2.
>
> Signed-off-by: Eric Engestrom <eric.engest...@imgtec.com>
> ---
>
> Fair warning: This patch will conflict with Jan Ziak's "[PATCH] dri: use
> 'bool'
> instead of 'int' for boolean variables", depending on which is applied first.
> Whoever picks one should pick the other and deal with the conflict right way
> :)
>
> ---
>
> src/glx/dri3_priv.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/src/glx/dri3_priv.h b/src/glx/dri3_priv.h
> index 0822377..31b2b98 100644
> --- a/src/glx/dri3_priv.h
> +++ b/src/glx/dri3_priv.h
> @@ -78,7 +78,6 @@ struct dri3_display
> int dri3Minor;
>
> /* Present bits */
> - int hasPresent;
> int presentMajor;
> int presentMinor;
The variable is never set actually. The other two (and maybe others)
are set-but-unused.
Can you do a quick grep through the struct and purge all the unused
variables in one go ?
Thanks
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
