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