Re: ole32/ole2: Don't call IDropTarget::QueryInterface() in RegisterDragDrop(). (try 4 resend)

[Michael Stefaniuc]

Hello Adam,

Adam Martinson wrote:
> This is to work around apps which break COM rules by not implementing

you need to follow the Wine COM guidelines:
http://wiki.winehq.org/COMGuideline
> IDropTarget::QueryInterface().  Windows doesn't expose this because it
> doesn't call CoMarshallInterface() in RegisterDragDrop().
> Fixes bug 23861.
>
> dlls/ole32/ole2.c           |  122
> +++++++++++++++++++++++++++++++++++++++---
>  dlls/ole32/tests/dragdrop.c |    2 +-
>  2 files changed, 114 insertions(+), 10 deletions(-)

ole32/ole2: Don't call IDropTarget::QueryInterface() in RegisterDragDrop(). (try 4 resend)

[Adam Martinson]

On 06/06/2011 10:58 AM, Michael Stefaniuc wrote:
> Hello Adam,
>
> Adam Martinson wrote:
> > This is to work around apps which break COM rules by not implementing
>
> you need to follow the Wine COM guidelines:
> http://wiki.winehq.org/COMGuideline
> > IDropTarget::QueryInterface().  Windows doesn't expose this because it
> > doesn't call CoMarshallInterface() in RegisterDragDrop().
> > Fixes bug 23861.
> >
> > dlls/ole32/ole2.c           |  122
> > +++++++++++++++++++++++++++++++++++++++---
> >   dlls/ole32/tests/dragdrop.c |    2 +-
> >   2 files changed, 114 insertions(+), 10 deletions(-)

Thanks, I'll take a look.