[another resend] Improve the hhctrl.ocx stub

Mike Hearn mike at navi.cx
Fri Apr 15 11:19:30 CDT 2005


This one should apply, hopefully.

- Add text dumping of the requested command
- Make the stub message appear for more commands
- Only do unicode conversion in HTML Help control when filename is specified

Index: dlls/hhctrl.ocx/hhctrl.c
===================================================================
RCS file: /home/wine/wine/dlls/hhctrl.ocx/hhctrl.c,v
retrieving revision 1.6
diff -u -p -d -r1.6 hhctrl.c
--- dlls/hhctrl.ocx/hhctrl.c	11 Apr 2005 12:47:40 -0000	1.6
+++ dlls/hhctrl.ocx/hhctrl.c	15 Apr 2005 16:04:50 -0000
@@ -30,16 +30,65 @@
 
 WINE_DEFAULT_DEBUG_CHANNEL(htmlhelp);
 
+static char *command_to_string(UINT command)
+{
+#define X(x) case x: return #x
+    
+    switch (command)
+    {
+        X( HH_DISPLAY_TOPIC );
+        X( HH_DISPLAY_TOC );
+        X( HH_DISPLAY_INDEX );
+        X( HH_DISPLAY_SEARCH );
+        X( HH_SET_WIN_TYPE );
+        X( HH_GET_WIN_TYPE );
+        X( HH_GET_WIN_HANDLE );
+        X( HH_ENUM_INFO_TYPE );
+        X( HH_SET_INFO_TYPE );
+        X( HH_SYNC );
+        X( HH_RESERVED1 );
+        X( HH_RESERVED2 );
+        X( HH_RESERVED3 );
+        X( HH_KEYWORD_LOOKUP );
+        X( HH_DISPLAY_TEXT_POPUP );
+        X( HH_HELP_CONTEXT );
+        X( HH_TP_HELP_CONTEXTMENU );
+        X( HH_TP_HELP_WM_HELP );
+        X( HH_CLOSE_ALL );
+        X( HH_ALINK_LOOKUP );
+        X( HH_GET_LAST_ERROR );
+        X( HH_ENUM_CATEGORY );
+        X( HH_ENUM_CATEGORY_IT );
+        X( HH_RESET_IT_FILTER );
+        X( HH_SET_INCLUSIVE_FILTER );
+        X( HH_SET_EXCLUSIVE_FILTER );
+        X( HH_INITIALIZE );
+        X( HH_UNINITIALIZE );
+        X( HH_PRETRANSLATEMESSAGE );
+        X( HH_SET_GLOBAL_PROPERTY );
+    }
+    
+#undef X    
+}
+
 HWND WINAPI HtmlHelpW(HWND caller, LPCWSTR filename, UINT command, DWORD data)
 {
-    FIXME("(%p, %s, command=%d, data=%ld): stub\n", caller, debugstr_w(filename), command, data);
+    FIXME("(%p, %s, command=%s, data=%ld): stub\n",
+          caller, debugstr_w( filename ),
+          command_to_string( command ), data);
 
-    /* if command is HH_DISPLAY_TOPIC just display an informative message for now */
-    if (command == HH_DISPLAY_TOPIC)
-        MessageBoxA( NULL, "HTML Help functionality is currently unimplemented.\n\n"
-                     "Try installing Internet Explorer, or using a native hhctrl.ocx with the Mozilla ActiveX control.",
-                     "Wine", MB_OK | MB_ICONEXCLAMATION );
-    return 0;
+    switch (command)
+    {
+        case HH_DISPLAY_TOPIC:
+        case HH_DISPLAY_TOC:
+        case HH_DISPLAY_SEARCH:
+        case HH_HELP_CONTEXT:
+            MessageBoxA( NULL, "HTML Help functionality is currently unimplemented.\n\n"
+                         "Try installing Internet Explorer, or using a native hhctrl.ocx with the Mozilla ActiveX control.",
+                         "Wine", MB_OK | MB_ICONEXCLAMATION );
+        default:
+            return 0;
+    }
 }
 
 HWND WINAPI HtmlHelpA(HWND caller, LPCSTR filename, UINT command, DWORD data)



More information about the wine-patches mailing list