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

[Rémi Bernon (＠rbernon)]

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?