Nikolay Sivov : comctl32/rebar: Don' t use local variable with the same name as function parameter (Coverity).

Alexandre Julliard julliard at winehq.org
Fri Feb 25 11:19:45 CST 2011


Module: wine
Branch: master
Commit: 190f6ab158e45b8107953fcdeb1707e4d4b00118
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=190f6ab158e45b8107953fcdeb1707e4d4b00118

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Fri Feb 25 11:50:01 2011 +0300

comctl32/rebar: Don't use local variable with the same name as function parameter (Coverity).

---

 dlls/comctl32/rebar.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/dlls/comctl32/rebar.c b/dlls/comctl32/rebar.c
index 8e9c38b..74ed03d 100644
--- a/dlls/comctl32/rebar.c
+++ b/dlls/comctl32/rebar.c
@@ -1524,9 +1524,10 @@ REBAR_SizeToHeight(REBAR_INFO *infoPtr, int height)
         for (i = prev_visible(infoPtr, infoPtr->uNumBands); i > 0; i = prev_visible(infoPtr, i))
         {
             REBAR_BAND *lpBand = REBAR_GetBand(infoPtr, i);
-            int height = lpBand->rcBand.bottom - lpBand->rcBand.top;
             int cyBreakExtra;  /* additional cy for the rebar after a RBBS_BREAK on this band */
 
+            height = lpBand->rcBand.bottom - lpBand->rcBand.top;
+
             if (infoPtr->dwStyle & RBS_VARHEIGHT)
                 cyBreakExtra = lpBand->cyRowSoFar; /* 'height' => 'lpBand->cyRowSoFar' + 'height'*/
             else




More information about the wine-cvs mailing list