comdlg32: Add a break statement to mark a semantic separtation

Andrew Talbot andrew.talbot at talbotville.com
Sat Sep 24 10:28:28 CDT 2011


Changelog:
    comdlg32: Add a break statement to mark a semantic separtation.

Although this break statement would be technically redundant, it helps to delineate betweeen a shared do-something code path and a separate shared do-nothing code path. The existing fall-through works only because the second code path happens to be a NOP.

diff --git a/dlls/comdlg32/itemdlg.c b/dlls/comdlg32/itemdlg.c
index bc309c9..9e90276 100644
--- a/dlls/comdlg32/itemdlg.c
+++ b/dlls/comdlg32/itemdlg.c
@@ -647,6 +647,7 @@ static void customctrl_resize(FileDialogImpl *This, customctrl *ctrl)
         GetWindowRect(ctrl->hwnd, &rc);
         SetWindowPos(ctrl->wrapper_hwnd, NULL, 0, 0, rc.right-rc.left, rc.bottom-rc.top,
                      SWP_NOZORDER|SWP_NOMOVE|SWP_NOZORDER);
+        break;
     case IDLG_CCTRL_RADIOBUTTONLIST:
     case IDLG_CCTRL_EDITBOX:
     case IDLG_CCTRL_SEPARATOR:



More information about the wine-patches mailing list