Re: [PATCH v3 0/1] MR2957: win32u: Move QueryDisplayConfig from user32.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/win32u/sysparams.c:
> - if (!(devmode->dmFields & DM_DISPLAYFLAGS))
> -
>    return DISPLAYCONFIG_SCANLINE_ORDERING_UNSPECIFIED;
> - else if (devmode->dmDisplayFlags & DM_INTERLACED)
> -
>    return DISPLAYCONFIG_SCANLINE_ORDERING_INTERLACED;
> - else
> -
>    return DISPLAYCONFIG_SCANLINE_ORDERING_PROGRESSIVE;
>
> +}
> -
>
> +static DISPLAYCONFIG_PIXELFORMAT get_dc_pixelformat(DWORD dmBitsPerPel)
> +{
> - if ((dmBitsPerPel == 8) || (dmBitsPerPel == 16) ||
> -
>    (dmBitsPerPel == 24) || (dmBitsPerPel == 32))
> -
>    return dmBitsPerPel / 8;
> - else
> -
>    return DISPLAYCONFIG_PIXELFORMAT_NONGDI;
>
> +}

```suggestion:-7+0
static DISPLAYCONFIG_PIXELFORMAT get_dc_pixelformat( DWORD bits )
{
    if ((bits == 8) || (bits == 16) || (bits == 24) || (bits == 32))
        return bits / 8;
    else
        return DISPLAYCONFIG_PIXELFORMAT_NONGDI;
}
```