Re: [PATCH v4 0/7] MR5262: win32u: Refactor monitor cache to only store the necessary information.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/win32u/sysparams.c:
> {
>          if (!is_monitor_primary( monitor )) continue;
>          if (should_enumerate_monitor( monitor, &origin, &limit, &enum_info[count].rect ))
> +        {
>              enum_info[count++].handle = monitor->handle;
> +            break;

This break should not be inside the if condition. There is only one primary monitor. You want to break out of the loop even if should_enumerate_monitor() returns FALSE.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5262#note_64678