[SNDVOL32] Deduplicate dialog control retrieval code

Author: SigmaTel71

base/applications/sndvol32/dialog.c

@@ -765,22 +765,18 @@ LoadDialogCtrls(
     }
 }
 
-VOID
-UpdateDialogLineSwitchControl(
+HWND
+GetLineDialogControl(
     PPREFERENCES_CONTEXT PrefContext,
     LPMIXERLINE Line,
-    LONG fValue)
+    DWORD dwDialogID)
 {
-    DWORD Index;
-    DWORD wID;
-    HWND hDlgCtrl;
-    WCHAR LineName[MIXER_LONG_NAME_CHARS];
-
     /* find the index of this line */
-    for (Index = 0; Index < PrefContext->MixerWindow->DialogCount; Index++)
+    for (DWORD Index = 0; Index < PrefContext->MixerWindow->DialogCount; Index++)
     {
         /* get id */
-        wID = (Index + 1) * IDC_LINE_NAME;
+        WCHAR LineName[MIXER_LONG_NAME_CHARS];
+        DWORD wID = (Index + 1) * IDC_LINE_NAME;
 
         if (GetDlgItemText(PrefContext->MixerWindow->hWnd, wID, LineName, MIXER_LONG_NAME_CHARS) == 0)
         {
@@ -792,69 +788,50 @@ UpdateDialogLineSwitchControl(
         if (!_wcsicmp(LineName, Line->szName))
         {
             /* found matching line name */
-            wID = (Index + 1) * IDC_LINE_SWITCH;
-
-            /* get dialog control */
-            hDlgCtrl = GetDlgItem(PrefContext->MixerWindow->hWnd, wID);
+            wID = (Index + 1) * dwDialogID;
 
-            if (hDlgCtrl != NULL)
-            {
-                /* check state */
-                if (SendMessageW(hDlgCtrl, BM_GETCHECK, 0, 0) != fValue)
-                {
-                    /* update control state */
-                    SendMessageW(hDlgCtrl, BM_SETCHECK, (WPARAM)fValue, 0);
-                }
-            }
-            break;
+            return GetDlgItem(PrefContext->MixerWindow->hWnd, wID);
         }
     }
+
+    return NULL;
 }
 
 VOID
-UpdateDialogLineSliderControl(
+UpdateDialogLineSwitchControl(
     PPREFERENCES_CONTEXT PrefContext,
     LPMIXERLINE Line,
-    DWORD dwDialogID,
-    DWORD Position)
+    LONG fValue)
 {
-    DWORD Index;
-    DWORD wID;
-    HWND hDlgCtrl;
-    WCHAR LineName[MIXER_LONG_NAME_CHARS];
+    HWND hDlgCtrl = GetLineDialogControl(PrefContext, Line, IDC_LINE_SWITCH);
 
-    /* find the index of this line */
-    for (Index = 0; Index < PrefContext->MixerWindow->DialogCount; Index++)
-    {
-        /* get id */
-        wID = (Index + 1) * IDC_LINE_NAME;
+    if (hDlgCtrl != NULL && SendMessageW(hDlgCtrl, BM_GETCHECK, 0, 0) != fValue)
+        SendMessageW(hDlgCtrl, BM_SETCHECK, (WPARAM)fValue, 0);
+}
 
-        if (GetDlgItemText(PrefContext->MixerWindow->hWnd, wID, LineName, MIXER_LONG_NAME_CHARS) == 0)
-        {
-            /* failed to retrieve id */
-            continue;
-        }
+DWORD
+GetDialogLineSliderCurrentPosition(
+    PPREFERENCES_CONTEXT PrefContext,
+    LPMIXERLINE Line,
+    DWORD dwDialogID)
+{
+    HWND hDlgCtrl = GetLineDialogControl(PrefContext, Line, dwDialogID);
 
-        /* check if the line name matches */
-        if (!_wcsicmp(LineName, Line->szName))
-        {
-            /* found matching line name */
-            wID = (Index + 1) * dwDialogID;
+    return hDlgCtrl != NULL
+        ? SendMessageW(hDlgCtrl, TBM_GETPOS, 0, 0)
+        : 0;
+}
 
-            /* get dialog control */
-            hDlgCtrl = GetDlgItem(PrefContext->MixerWindow->hWnd, wID);
+VOID
+UpdateDialogLineSliderControl(
+    PPREFERENCES_CONTEXT PrefContext,
+    LPMIXERLINE Line,
+    DWORD dwDialogID,
+    DWORD Position)
+{
+    HWND hDlgCtrl = GetLineDialogControl(PrefContext, Line, dwDialogID);
+    BOOL isPositionChanged = GetDialogLineSliderCurrentPosition(PrefContext, Line, dwDialogID) != Position;
 
-            if (hDlgCtrl != NULL)
-            {
-                /* check state */
-                LRESULT OldPosition = SendMessageW(hDlgCtrl, TBM_GETPOS, 0, 0);
-                if (OldPosition != Position)
-                {
-                    /* update control state */
-                    SendMessageW(hDlgCtrl, TBM_SETPOS, (WPARAM)TRUE, Position);
-                }
-            }
-            break;
-        }
-    }
-}
+    if (hDlgCtrl != NULL && isPositionChanged)
+        SendMessageW(hDlgCtrl, TBM_SETPOS, (WPARAM)TRUE, Position);
+}