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:
> +    }
> +    *token = TK_ILLEGAL;
> +    return 1;
> +}
> +
> +UINT aqs_lex( void *p, struct aqs_parser *parser )
> +{
> +    aqs_token_kind_t token = -1;
> +    struct string *str = p;
> +
> +    do
> +    {
> +        parser->idx += parser->len;
> +        if (!parser->query[parser->idx]) return 0;
> +        if (!(parser->len = get_token( &parser->query[parser->idx], &token )))
> +            break;
> Can this ever return 0? Should we return 0 on NUL char or invalid token maybe?

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