Re: [PATCH 0/3] MR9010: cfgmgr32: Device query API part 4: Implement support for logical boolean operators in DEVPROP_FILTER_EXPRESSION.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/cfgmgr32/main.c:
> cmp = op & DEVPROP_OPERATOR_MODIFIER_IGNORE_CASE ? wcsicmp( prop->Buffer, cmp_prop->Buffer )
>                                                                   : wcscmp( prop->Buffer, cmp_prop->Buffer );
>                  break;
> +            case DEVPROP_TYPE_GUID:
> +                /* Any other comparison operator other than DEVPROP_OPERATOR_EQUALS with GUIDs evaluates to false. */
> +                if (!(op & DEVPROP_OPERATOR_EQUALS)) break;

Wouldn't that get inverted by the `DEVPROP_OPERATOR_MODIFIER_NOT` check at the bottom? Is it expected?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9010#note_116435