On 12/22/10 2:17 PM, David Hedberg wrote:
> Hi Jacek,
> On Wed, Dec 22, 2010 at 13:07, Jacek Caban<[email protected]>
wrote:
>>> + if(security_problem) {
>>> + hres =
IUnknown_QueryService((IUnknown*)This->base.protocol_sink,
>> You shouldn't need the cast here.
> I get warnings if I leave the casts out though, and I can't see any
obvious problem with the declarations. Any ideas?
I've read your patch wrongly, sorry. Please avoid using
IUnknown_QueryService. It's an ugly named helper function and it causes
confusions like mine. Explicitly QIing for IServiceProvider and calling
QueryService on it is the way to go.
Cheers,
Jacek
