https://bugs.winehq.org/show_bug.cgi?id=37113
--- Comment #6 from graben info@translucide.ca --- I did a bit of debugging and it does not seem to me that the patch in question is the problem.
First, in the latest git, the patch is reversed and trackerInfo->returnValue = IDropTarget_Drop(...) is back again as it was before.
But the problem persist in the latest git.
I can confirm that the function IDropTarget_Drop(...) is called in the case DRAGDROP_S_DROP clause in dlls/ole2.c around line 2411
I am not familiar with wine codebase nor windows internals so I don't know what is the problem here, but the drag n drop operation gets cancelled somehow and DRAGDROP_S_CANCEL case part in the same file around line 2433 is not executed.
When testing here is the values for the meaningful variables I have observed :
trackerInfo->returnValue = DRAGDROP_S_DROP (0x40100) trackerInfo->pdwEffect = 0x02, which equates to move effect
I have not found where IDropTarget_Drop is defined in the wine codebase, the search for it did not return the function definition... I suspect it might have something to do with it, but then again I'm not familiar with wine nor Windows internals and I might be missing something obvious here...