[PATCH] wincodecs: Add IWICImagingFactory2 stub.

Dmitry Timoshkov dmitry at baikal.ru
Sun Sep 30 03:45:49 CDT 2018


Nikolay Sivov <nsivov at codeweavers.com> wrote:

> >>>>    typedef struct {
> >>>>        IWICComponentFactory IWICComponentFactory_iface;
> >>>> +    IWICImagingFactory2 IWICImagingFactory2_iface;
> >>>>        LONG ref;
> >>>>    } ComponentFactory;
> >>> IWICImagingFactory2 just extends IWICImagingFactory by one element, this
> >>> approach seems to be a bit strange.
> >>>
> >> Both IWICComponentFactory andIWICImagingFactory2extend IWICImagingFactory.
> > If you mean that you have to choose which one should use IWICImagingFactory
> > as a base class then an obvious choice would be IWICImagingFactory2. If you
> > ment something else please try to be less cryptic in your responses.
> >
> If you mean it's better to move existing IWICImagingFactory methods 
> implementation to newly added IWICImagingFactory2, I don't think it's 
> obvious or necessary. It will lead to larger diff without any functional 
> impact. Structure was unfortunately renamed from ImagingFactory, but 
> well what's done is done.

Yes, that's a little bit unfortunate, but IMO it's a good opportunity
to fix it now while moving to a newer version of the interface instead
of keeping this mess forever.

-- 
Dmitry.



More information about the wine-devel mailing list