Re: [PATCH v3 0/1] MR2957: win32u: Move QueryDisplayConfig from user32.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/win32u/sysparams.c:
> +static BOOL source_mode_exists(const DISPLAYCONFIG_MODE_INFO *modeinfo, UINT32 num_modes,
> -
>                           UINT32 source_id, UINT32 *found_mode_index)
>
> +{
> - UINT32 i;
> -
> - for (i = 0; i < num_modes; i++)
> - {
> -
>    if (modeinfo[i].infoType == DISPLAYCONFIG_MODE_INFO_TYPE_SOURCE &&
> -
>        modeinfo[i].id == source_id)
> -
>    {
> -
>        *found_mode_index = i;
> -
>        return TRUE;
> -
>    }
> - }
> - return FALSE;
>
> +}

```suggestion:-15+0
static BOOL source_mode_exists( const DISPLAYCONFIG_MODE_INFO *modes, UINT32 modes_count,
                                UINT32 source_id, UINT32 *found_mode_index )
{
    UINT32 i;

for (i = 0; i < modes_count; i++)
    {
        if (modes[i].infoType == DISPLAYCONFIG_MODE_INFO_TYPE_SOURCE &&
            modes[i].id == source_id)
        {
            *found_mode_index = i;
            return TRUE;
        }
    }
    return FALSE;
}
```