Re: [PATCH 0/7] MR576: win32u: Move display placement logic out of graphics drivers.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/win32u/sysparams.c:
> -
>    if (!adapter->mode_count) continue;
> -
>    size += sizeof(DEVMODEW) + adapter->modes[0].dmDriverExtra;
>
> }
> -
>
> /* allocate an extra mode for easier iteration */
> -
>
> if (!(displays = calloc( 1, size + sizeof(DEVMODEW) ))) goto done;
> -
>
> mode = displays;
> -
> -
>
> LIST_FOR_EACH_ENTRY(adapter, &adapters, struct adapter, entry)
> -
>
> {
> -
>    if (!adapter->mode_count) continue;
> -
> -
>    mode->dmSize = sizeof(DEVMODEW);
> -
>    if (devname)
> -
>    {
> -
>        if (!wcsicmp( devname, adapter->dev.device_name ))

I don't remember if it's from the D3D code. I think curly braces should be added when any of the branches contain more than one line.