On 05/23/2016 11:42 PM, mathias.froehl...@gmx.net wrote:
> From: Mathias Fröhlich <mathias.froehl...@web.de>
> 
> Switch over to use the CoordsReplaceBits bitmask.
> 
> Signed-off-by: Mathias Fröhlich <mathias.froehl...@web.de>
> ---
>  src/mesa/drivers/dri/r200/r200_state.c | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/r200/r200_state.c 
> b/src/mesa/drivers/dri/r200/r200_state.c
> index b4acf98..695d511 100644
> --- a/src/mesa/drivers/dri/r200/r200_state.c
> +++ b/src/mesa/drivers/dri/r200/r200_state.c
> @@ -1851,11 +1851,8 @@ static void r200Enable( struct gl_context *ctx, GLenum 
> cap, GLboolean state )
>     case GL_POINT_SPRITE_ARB:
>        R200_STATECHANGE( rmesa, spr );
>        if ( state ) {
> -      int i;
> -      for (i = 0; i < 6; i++) {
> -         rmesa->hw.spr.cmd[SPR_POINT_SPRITE_CNTL] |=
> -             ctx->Point.CoordReplace[i] << (R200_PS_GEN_TEX_0_SHIFT + i);
> -      }
> +      GLuint r = (GLuint)ctx->Point.CoordReplaceBits << 
> R200_PS_GEN_TEX_0_SHIFT;

Since this isn't API facing, just use unsigned.

> +      rmesa->hw.spr.cmd[SPR_POINT_SPRITE_CNTL] |= R200_PS_GEN_TEX_MASK & r;
>        } else {
>        rmesa->hw.spr.cmd[SPR_POINT_SPRITE_CNTL] &= ~R200_PS_GEN_TEX_MASK;
>        }
> 

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to