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.y:
> -
>    ;
>
> +expr:
> - TK_LEFTPAREN expr TK_RIGHTPAREN
> -
>    {
> -
>        $$ = $2;
>
> +#if YYBISON >= 30704
> -
>        (void)yysymbol_name; /* avoid unused function warning */
>
> +#endif
> -
>        (void)yynerrs; /* avoid unused variable warning */
> -
>    }
> - | expr TK_AND expr
> -
>    {
> -
>        HRESULT hr;
> -
>        hr = get_boolean_expr( ctx, OP_AND, $1, $3, &$$ );
> -
>        if (FAILED( hr ))
> -
>            YYABORT;

Doesn't seem you need these hr, IMO you could move the call within the FAILED check.