Re: [PATCH v7 0/1] MR6440: win32u: Implement NtGdiDdDDIEnumAdapters.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/win32u/sysparams.c:
>  return status;
>
> }
>
> +/******************************************************************************
> -
> -
>       NtGdiDdDDIEnumAdapters    (win32u.@)
> - */
>
> +NTSTATUS WINAPI NtGdiDdDDIEnumAdapters( D3DKMT_ENUMADAPTERS *desc )
> +{
> - NTSTATUS status;
> - D3DKMT_ENUMADAPTERS2 desc2 = {0};
> -
> - if (!desc) return STATUS_INVALID_PARAMETER;
> -
> - if (status = NtGdiDdDDIEnumAdapters2( &desc2 )) return status;

```suggestion:-0+0
    if ((status = NtGdiDdDDIEnumAdapters2( &desc2 ))) return status;
```

You need an extra paren to tell the compiler this is intentionally an assignment, and not a typoed ==, and avoid a warning.

Looks good otherwise.