Re: [PATCH 0/3] MR8697: combase: Don't marshal objects marked as IAgileObject in RoGetAgileReference.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/combase/roapi.c:
> impl->option = option;
>      impl->ref = 1;
>
> -    if (option == AGILEREFERENCE_DEFAULT)
> +    hr = IUnknown_QueryInterface(obj, &IID_IAgileObject, (void **)&unknown);
> +    if (SUCCEEDED(hr))
> +    {
> +        impl->obj = obj;
> +        impl->is_agile = TRUE;
> +    }

I would rather combine this with AGILEREFERENCE_DELAYEDMARSHAL block, using additional helper like "bool is_object_agile()". Right now you need to look twice to see why "unknown" is not released.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8697#note_113708