winex11.drv: Use bitwise NOT not logical NOT

Marcus Meissner marcus at jet.franken.de
Sat Feb 2 13:03:55 CST 2008


On Sat, Feb 02, 2008 at 05:43:27PM +0000, Andrew Talbot wrote:
> Changelog:
>     winex11.drv: Use bitwise NOT not logical NOT.
> 
> diff --git a/dlls/winex11.drv/settings.c b/dlls/winex11.drv/settings.c
> index d68b674..2cd36c7 100644
> --- a/dlls/winex11.drv/settings.c
> +++ b/dlls/winex11.drv/settings.c
> @@ -369,10 +369,10 @@ LONG X11DRV_ChangeDisplaySettingsEx( LPCWSTR devname, LPDEVMODEW devmode,
>                devmode->dmBitsPerPel,devmode->dmDisplayFrequency, handler_name);
>  
>          dwBpp = devmode->dmBitsPerPel;
> -        if (devmode->dmFields & DM_BITSPERPEL) def_mode &= !dwBpp;
> -        if (devmode->dmFields & DM_PELSWIDTH)  def_mode &= !devmode->dmPelsWidth;
> -        if (devmode->dmFields & DM_PELSHEIGHT) def_mode &= !devmode->dmPelsHeight;
> -        if (devmode->dmFields & DM_DISPLAYFREQUENCY) def_mode &= !devmode->dmDisplayFrequency;
> +        if (devmode->dmFields & DM_BITSPERPEL) def_mode &= ~dwBpp;
> +        if (devmode->dmFields & DM_PELSWIDTH)  def_mode &= ~devmode->dmPelsWidth;
> +        if (devmode->dmFields & DM_PELSHEIGHT) def_mode &= ~devmode->dmPelsHeight;
> +        if (devmode->dmFields & DM_DISPLAYFREQUENCY) def_mode &= ~devmode->dmDisplayFrequency;

I do not think this is right, def_mode is a boolean and we actually check for
not-0-being of various struct members.

Ciao, Marcus



More information about the wine-devel mailing list