[PATCH v3 4/8] dllhost: Implement IClassFactory::CreateInstance().

[Dmitry Timoshkov]

Huw Davies <huw at codeweavers.com> wrote:

> On Tue, Mar 08, 2022 at 01:10:15PM +0300, Dmitry Timoshkov wrote:
> > Huw Davies <huw at codeweavers.com> wrote:
> > > Shouldn't this ask for IID_IClassFactory, from which CreateInstance is
> > > called?  You can also pass punkouter to the inproc CreateInstance; see
> > > the link you sent me:
> > > 
> > > https://docs.microsoft.com/en-us/windows/win32/com/writing-a-custom-surrogate
> > 
> > Actually this patch could be dropped because according to the results I get
> > with this patch set IClassFactory::CreateInstance() is never called under Wine
> > during the surrogate invocation, and if really necessary could be implemented
> > the way you (and MSDN) suggest. Other patches in the series should apply cleanly
> > without this patch.
> 
> Hi Dmitry,
> 
> Does it all go via IMarshal then?

Looks like that, yes. At least that's true for the application I'm working
on, and everything seems to work just fine.

> I guess we can leave this out in that case.  Since I've signed off
> the first three patches, could you wait for those to go in and then
> send 5-8?

Sure, sound good. Thanks.

-- 
Dmitry.