Re: [PATCH v7 0/6] MR8890: windows.devices.enumeration: Add support for parsing AQS filter strings.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/windows.devices.enumeration/aqs.c:
> +                if (str[i+1] && str[i+1] == '\"')
> +                {
> +                    i++;
> +                    continue;
> +                }
> +                break;
> +            }
> +        }
> +        if (str[i])
> +            i++;
> +        *token = TK_STRING;
> +        return i;
> +    case '0': case '1': case '2': case '3': case '4':
> +    case '5': case '6': case '7': case '8': case '9':
> +        *token = TK_INTEGER;
> +        for (i = 1; iswdigit( str[i] ); i++);
        for (i = 1; iswdigit( str[i] ); i++) /* nothing */;

I think we usually add a comment there to avoid missing the empty loop.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8890#note_115221