[PATCH 0/1] MR9648: dlls/wmp: Implement the OLEIVERB_SHOW needed by the software Alldup

[pan hui (＠panhui)]

The software "Alldup" uses OLEIVERB_SHOW, but OLEIVERB_SHOW is not implemented in oleobj.c, resulting in errors when running the Alldup via Wine.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9648

[PATCH 1/1] dlls/wmp: Implement the OLEIVERB_SHOW needed by the software Alldup

[panhui]

From: panhui <panhui(a)uniontech.com>

---
 dlls/wmp/oleobj.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/dlls/wmp/oleobj.c b/dlls/wmp/oleobj.c
index 5309194f289..725ab3dd7e6 100644
--- a/dlls/wmp/oleobj.c
+++ b/dlls/wmp/oleobj.c
@@ -421,7 +421,14 @@ static HRESULT WINAPI OleObject_DoVerb(IOleObject *iface, LONG iVerb, LPMSG lpms
 
         ShowWindow(This->hwnd, SW_HIDE);
         return S_OK;
+    case OLEIVERB_SHOW:
+        if(!This->hwnd) {
+            FIXME("No window to show\n");
+            return E_UNEXPECTED;
+        }
 
+        ShowWindow(This->hwnd, SW_SHOW);
+        return S_OK;
     default:
         FIXME("Unsupported iVerb %ld\n", iVerb);
     }
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/9648

[Elizabeth Figura (＠zfigura)]

Can we please add some tests in test_wmp()? They should look similar to the tests for OLEIVERB_HIDE.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9648#note_124740