ole32/ole2: Don't call IDropTarget::QueryInterface() in RegisterDragDrop(). [whitespace fixed]

Huw Davies huw at codeweavers.com
Thu May 26 06:47:04 CDT 2011


On Wed, May 25, 2011 at 12:45:15PM -0500, Adam Martinson wrote:
> On 05/25/2011 12:36 PM, Adam Martinson wrote:
> >The AddRef is done in WrapDropTarget(), seems like the appropriate
> >place for it.  Quite right on the rest though, thanks.
> 
> Oops no, the window prop should be pDropTarget, not the wrapper.
> The wrapper is only used for the marshalling.

That window prop could probably go away, I only put it there because
that's what Windows does, I doubt there's an app that relies on it
though.  However if it stays you do need the AddRef!

Huw.



More information about the wine-devel mailing list