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:
> +    case '\n':
> +        for (i = 1; iswspace( str[i] ); i++);
> +        *token = TK_WHITESPACE;
> +        return i;
> +    case '(':
> +        *token = TK_LEFTPAREN;
> +        return 1;
> +    case ')':
> +        *token = TK_RIGHTPAREN;
> +        return 1;
> +    case '[':
> +        if (str[1] != ']')
> +        {
> +            *token = TK_ILLEGAL;
> +            return 1;
> +        }
> What about using the TK_ILLEGAL default below?

```suggestion:-4+0
        if (str[1] != ']') break; /* illegal */
```

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