gdi32: Simplify bitwise and and xor with 0 (PVS-Studio)

Huw Davies huw at codeweavers.com
Wed Jul 29 06:39:32 CDT 2015


On 29 Jul 2015, at 12:32, Michael Stefaniuc wrote:
> ---
> Same sha1 for primitives.o with and without patch.
> 
> 
> 
> dlls/gdi32/dibdrv/primitives.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/dlls/gdi32/dibdrv/primitives.c b/dlls/gdi32/dibdrv/primitives.c
> index c7b48ed..9da08ad 100644
> --- a/dlls/gdi32/dibdrv/primitives.c
> +++ b/dlls/gdi32/dibdrv/primitives.c
> @@ -5655,8 +5655,8 @@ static void create_dither_masks_1(const dib_info *dib, int rop2, COLORREF color,
>             }
>             else
>             {
> -                rop_mask.and = (0x00 & codes.a1) ^ codes.a2;
> -                rop_mask.xor = (0x00 & codes.x1) ^ codes.x2;
> +                rop_mask.and = codes.a2;
> +                rop_mask.xor = codes.x2;
>             }
>             *and_bits |= (rop_mask.and & pixel_masks_1[x]);
>             *xor_bits |= (rop_mask.xor & pixel_masks_1[x]);


It's clearer as it was, especially given the lines just above this hunk.

Huw.




More information about the wine-devel mailing list