[PATCH v2 3/8] shell32/autocomplete: Forward to the edit control outside of the switch to simplify the code
Gabriel Ivăncescu
gabrielopcode at gmail.com
Thu Sep 20 06:55:36 CDT 2018
Signed-off-by: Gabriel Ivăncescu <gabrielopcode at gmail.com>
---
Instead of using the default case to forward it, use the case outside of the
switch for this purpose. It's a minor simplification now, but it will be very
helpful in future patches (next series), when more messages will be handled.
dlls/shell32/autocomplete.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/dlls/shell32/autocomplete.c b/dlls/shell32/autocomplete.c
index 40e0866..ef835b9 100644
--- a/dlls/shell32/autocomplete.c
+++ b/dlls/shell32/autocomplete.c
@@ -348,13 +348,13 @@ static LRESULT APIENTRY ACEditSubclassProc(HWND hwnd, UINT uMsg, WPARAM wParam,
case CB_SHOWDROPDOWN:
if (This->options & ACO_AUTOSUGGEST)
ShowWindow(This->hwndListBox, SW_HIDE);
- break;
+ return 0;
case WM_KILLFOCUS:
if ((This->options & ACO_AUTOSUGGEST) && ((HWND)wParam != This->hwndListBox))
{
ShowWindow(This->hwndListBox, SW_HIDE);
}
- return CallWindowProcW(This->wpOrigEditProc, hwnd, uMsg, wParam, lParam);
+ break;
case WM_KEYDOWN:
return ACEditSubclassProc_KeyDown(This, hwnd, uMsg, wParam, lParam);
case WM_CHAR:
@@ -382,11 +382,8 @@ static LRESULT APIENTRY ACEditSubclassProc(HWND hwnd, UINT uMsg, WPARAM wParam,
destroy_autocomplete_object(This);
return CallWindowProcW(proc, hwnd, uMsg, wParam, lParam);
}
- default:
- return CallWindowProcW(This->wpOrigEditProc, hwnd, uMsg, wParam, lParam);
}
-
- return 0;
+ return CallWindowProcW(This->wpOrigEditProc, hwnd, uMsg, wParam, lParam);
}
static LRESULT APIENTRY ACLBoxSubclassProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
--
1.9.1
More information about the wine-devel
mailing list