windows: in DnD, always set pdwEffect

It appears other apps (like Chromium) set pdwEffect on Drop too: https://github.com/chromium/chromium/blob/61a391b86bd946d6e1105412539e77ba9fb2a6b3/ui/base/dragdrop/drop_target_win.cc
rust-windowing · Jan 17, 2025 · 2153865 · 2153865
1 parent 59dd7ed
commit 2153865
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/platform_impl/windows/drop_handler.rs b/src/platform_impl/windows/drop_handler.rs
@@ -127,6 +127,7 @@ impl FileDropHandler {
         unsafe {
             *pdwEffect = drop_handler.cursor_effect;
         }
+
         S_OK
     }
 
@@ -147,7 +148,7 @@ impl FileDropHandler {
         pDataObj: *const IDataObject,
         _grfKeyState: u32,
         pt: POINTL,
-        _pdwEffect: *mut u32,
+        pdwEffect: *mut u32,
     ) -> HRESULT {
         let drop_handler = unsafe { Self::from_interface(this) };
         if drop_handler.valid {
@@ -168,6 +169,9 @@ impl FileDropHandler {
                 }
             }
         }
+        unsafe {
+            *pdwEffect = drop_handler.cursor_effect;
+        }
 
         S_OK
     }