Re: [PATCH v12 0/3] MR551: win32u: Complete support for display modes in nulldrv.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/win32u/sysparams.c:
> if (!(primary = find_adapter_device_by_id( 0 ))) primary_name[0] = 0;
>      else wcscpy( primary_name, primary->device_name );
>
> -    ret = user_driver->pChangeDisplaySettings( displays, primary_name, hwnd, flags, lparam );
> +    if ((ret = user_driver->pChangeDisplaySettings( displays, primary_name, hwnd, flags, lparam )))

In your earlier revisions, you checked ret against E_NOTIMPL. Now you're going for the fallback anytime the user driver fails to change display modes? why did you change it? I think the earlier versions are correct.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/551#note_12776