[PATCH v2 1/8] d2d1: Add stubs for ID2D1EffectContext.

Ziqing Hui zhui at codeweavers.com
Fri Apr 29 01:45:02 CDT 2022



On 4/29/22 1:54 PM, Nikolay Sivov wrote:
> 
> 
> On 4/28/22 13:40, Ziqing Hui wrote:
>> +void d2d_effect_context_init(struct d2d_effect_context *effect_context)
>> +{
>> +    effect_context->ID2D1EffectContext_iface.lpVtbl = &d2d_effect_context_vtbl;
>> +    effect_context->refcount = 1;
>> +}
> This patch introduces unused code. To avoid that I would move existing device_context->CreateEffect() to effect_context->CreateEffect() right away in the first patch.
> 
> Regarding init helper and patches 2, 4, 7, it seems easier to keep "struct d2d_device_context *" in effect context instead. It provides access to the factory and d3d device.
> 

Thanks for reply.

OK, I'll send a v3 version later following your advice.




More information about the wine-devel mailing list