hhctrl.ocx: Use the SendMessage instead of ListView_GetItem.
authorMichael Stefaniuc <mstefani@redhat.de>
Tue, 9 Feb 2010 22:02:03 +0000 (23:02 +0100)
committerAlexandre Julliard <julliard@winehq.org>
Wed, 10 Feb 2010 10:33:57 +0000 (11:33 +0100)
SendMessage() is more explicit and also gets rid of three gcc -Wextra
  warning: value computed is not used

dlls/hhctrl.ocx/help.c

index 680911a0f019f61dadd961f9d47e93453ac2a6bb..30cb008bd27a7ef05765529db5ec8fa2b48a5c77 100644 (file)
@@ -550,7 +550,7 @@ static LRESULT CALLBACK Child_WndProc(HWND hWnd, UINT message, WPARAM wParam, LP
 
                 lvItem.iItem = (int) SendMessageW(hwndList, LVM_GETSELECTIONMARK, 0, 0);
                 lvItem.mask = TVIF_PARAM;
-                ListView_GetItemW(hwndList, &lvItem);
+                SendMessageW(hwndList, LVM_GETITEMW, 0, (LPARAM)&lvItem);
                 OnTopicChange(info, (void*) lvItem.lParam);
                 return 0;
             }
@@ -578,7 +578,7 @@ static LRESULT CALLBACK Child_WndProc(HWND hWnd, UINT message, WPARAM wParam, LP
 
                     lvItem.iItem = (int) SendMessageW(hwndList, LVM_GETSELECTIONMARK, 0, 0);
                     lvItem.mask = TVIF_PARAM;
-                    ListView_GetItemW(hwndList, &lvItem);
+                    SendMessageW(hwndList, LVM_GETITEMW, 0, (LPARAM)&lvItem);
                     OnTopicChange(info, (void*) lvItem.lParam);
                     return 0;
                 }
@@ -1095,7 +1095,7 @@ static LRESULT CALLBACK PopupChild_WndProc(HWND hWnd, UINT message, WPARAM wPara
 
             lvItem.iItem = (int) SendMessageW(info->popup.hwndList, LVM_GETSELECTIONMARK, 0, 0);
             lvItem.mask = TVIF_PARAM;
-            ListView_GetItemW(info->popup.hwndList, &lvItem);
+            SendMessageW(info->popup.hwndList, LVM_GETITEMW, 0, (LPARAM)&lvItem);
             iter = (IndexSubItem*) lvItem.lParam;
             NavigateToChm(info, info->index->merge.chm_file, iter->local);
             ShowWindow(info->popup.hwndPopup, SW_HIDE);