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.