[2/2] msimtf: Add stub implementation for IActiveIMMMessagePumpOwner.
thomas.faber at reactos.org
Wed Jun 24 02:38:41 CDT 2015
On 2015-06-24 09:20, Nikolay Sivov wrote:
> On 24.06.2015 0:48, Thomas Faber wrote:
>> +static HRESULT WINAPI ActiveIMMApp_IActiveIMMMessagePumpOwner_QueryInterface(IActiveIMMMessagePumpOwner* iface,
>> + REFIID iid, LPVOID *ppvOut)
>> + ActiveIMMApp *This = impl_from_IActiveIMMMessagePumpOwner(iface);
>> + return IActiveIMMApp_QueryInterface(&This->IActiveIMMApp_iface, iid, ppvOut);
> It's usually up to author, but I'll still mention this - no need to
> prefix function names with ActiveIMMApp_ because there's no other
> IActiveIMMMessagePumpOwner implementation, so names won't collide.
Makes sense. I'll fix it, those names are way too long as-is. ;)
> It would be nice to have a simple test too, to make sure QI supports
> this interface, but as we don't have tests dir at all for this module it
> could be a separate patch I guess.
I tried to add one but CoCreateInstance for CLSID_CActiveIMM always
seems to return E_FAIL on Windows. Not sure what I have to do to make
More information about the wine-devel