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.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.

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