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

[Adam Martinson]

On 05/26/2011 06:47 AM, Huw Davies wrote:
> 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.
Best to mirror Windows as close as possible.  The AddRef() is done in 
WrapDropTarget().  On failure IDropTarget::Release() is done on the 
wrapper, which removes the ref on the original.