Re: Patch 1/2 - [scrrun] - Add initial support for Dictionary - try 2

[Vijay Kiran Kamuju]

On Fri, Jun 8, 2012 at 11:00 AM, Jacek Caban <jacek(a)codeweavers.com> wrote:
> Hi Vijay,
>
> On 06/08/12 03:15, Vijay Kiran Kamuju wrote:
> > +    *ppvObject = NULL;
> > +    if ( IsEqualGUID( riid, &IID_IDictionary ) ||
> > +         IsEqualGUID( riid, &IID_IPersist ) ||
> > +         IsEqualGUID( riid, &IID_IDispatch ) ||
> > +         IsEqualGUID( riid, &IID_IUnknown ) )
> > +    {
> > +        *ppvObject = This;
> > +    }
> > Attaching the new dictionary.c with a bit more functionality, please review
>
> That's not the right way to cast object to interface. See
> http://wiki.winehq.org/COMGuideline
>
> Cheers,
> Jacek