Nikolay Sivov : user32/combo: Remove redundant argument from text height helper.

Alexandre Julliard julliard at winehq.org
Tue Nov 26 16:21:24 CST 2019


Module: wine
Branch: master
Commit: 14e074b2a428b0f6a4c2ba2f2f0a6ba9ed0dc837
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=14e074b2a428b0f6a4c2ba2f2f0a6ba9ed0dc837

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Tue Nov 26 12:59:22 2019 +0300

user32/combo: Remove redundant argument from text height helper.

Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/user32/combo.c | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/dlls/user32/combo.c b/dlls/user32/combo.c
index d0affc0618..40ff0ef521 100644
--- a/dlls/user32/combo.c
+++ b/dlls/user32/combo.c
@@ -193,9 +193,7 @@ static LRESULT COMBO_NCDestroy( LPHEADCOMBO lphc )
  * This height was determined through experimentation.
  * CBCalcPlacement will add 2*COMBO_YBORDERSIZE pixels for the border
  */
-static INT CBGetTextAreaHeight(
-  HWND        hwnd,
-  LPHEADCOMBO lphc)
+static INT CBGetTextAreaHeight(HEADCOMBO *lphc)
 {
   INT iTextItemHeight;
 
@@ -206,7 +204,7 @@ static INT CBGetTextAreaHeight(
   else
   {
     TEXTMETRICW tm;
-    HDC         hDC       = GetDC(hwnd);
+    HDC         hDC       = GetDC(lphc->self);
     HFONT       hPrevFont = 0;
     INT         baseUnitY;
 
@@ -220,7 +218,7 @@ static INT CBGetTextAreaHeight(
     if( hPrevFont )
       SelectObject( hDC, hPrevFont );
 
-    ReleaseDC(hwnd, hDC);
+    ReleaseDC(lphc->self, hDC);
 
     iTextItemHeight = baseUnitY + 4;
   }
@@ -240,7 +238,7 @@ static INT CBGetTextAreaHeight(
     /*
      * We use the client rect for the width of the item.
      */
-    GetClientRect(hwnd, &clientRect);
+    GetClientRect(lphc->self, &clientRect);
 
     lphc->wState &= ~CBF_MEASUREITEM;
 
@@ -294,7 +292,7 @@ static void CBForceDummyResize(
   RECT windowRect;
   int newComboHeight;
 
-  newComboHeight = CBGetTextAreaHeight(lphc->self,lphc) + 2*COMBO_YBORDERSIZE();
+  newComboHeight = CBGetTextAreaHeight(lphc) + 2*COMBO_YBORDERSIZE();
 
   GetWindowRect(lphc->self, &windowRect);
 
@@ -328,7 +326,7 @@ static void CBCalcPlacement(HEADCOMBO *combo)
     InflateRect(&combo->textRect, -COMBO_XBORDERSIZE(), -COMBO_YBORDERSIZE());
 
     /* Chop off the bottom part to fit with the height of the text area. */
-    combo->textRect.bottom = combo->textRect.top + CBGetTextAreaHeight(combo->self, combo);
+    combo->textRect.bottom = combo->textRect.top + CBGetTextAreaHeight(combo);
 
     /* The button starts the same vertical position as the text area. */
     combo->buttonRect = combo->textRect;
@@ -1455,7 +1453,7 @@ static void COMBO_Size( LPHEADCOMBO lphc )
     GetWindowRect(lphc->self, &rc);
     curComboHeight = rc.bottom - rc.top;
     curComboWidth = rc.right - rc.left;
-    newComboHeight = CBGetTextAreaHeight(lphc->self, lphc) + 2*COMBO_YBORDERSIZE();
+    newComboHeight = CBGetTextAreaHeight(lphc) + 2*COMBO_YBORDERSIZE();
 
     /*
      * Resizing a combobox has another side effect, it resizes the dropped
@@ -2004,7 +2002,7 @@ LRESULT ComboWndProc_common( HWND hwnd, UINT message, WPARAM wParam, LPARAM lPar
 	case CB_GETITEMHEIGHT:
 		if( (INT)wParam >= 0 )	/* listbox item */
                     return SendMessageW(lphc->hWndLBox, LB_GETITEMHEIGHT, wParam, 0);
-                return  CBGetTextAreaHeight(hwnd, lphc);
+                return  CBGetTextAreaHeight(lphc);
 	case CB_RESETCONTENT:
 		SendMessageW(lphc->hWndLBox, LB_RESETCONTENT, 0, 0);
                 if( (lphc->wState & CBF_EDIT) && CB_HASSTRINGS(lphc) )




More information about the wine-cvs mailing list