On Tue, 9 Sep 2008, Rob Shearman wrote: >> -static HRESULT Registrar_create(const IUnknown *pUnkOuter, REFIID riid, >> void **ppvObject) >> +static HRESULT Registrar_create(REFIID riid, void **ppvObject) > A test needs to be added to see whether or not the Registrar class > factory supports aggregation. If it does then a FIXME should be > emitted. If not then an ERR may be emitted and CLASS_E_NOAGGREGATION > returned. Just removing pUnkOuter isn't the right thing to do.
Fair point! Sadly, this is beyond my expertise in this area and I hope that someone with that expertise is going to help with that? Is this something you could look at? (Based on your analysis consider my patch redrawn...) Gerald