Peter Schauer : comctl32/listview: Fix warning from DPA_GetPtr.

Alexandre Julliard julliard at winehq.org
Tue Apr 21 11:45:46 CDT 2009


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

Author: Peter Schauer <peterschauer at gmx.net>
Date:   Sun Apr 19 13:14:55 2009 +0200

comctl32/listview: Fix warning from DPA_GetPtr.

---

 dlls/comctl32/listview.c |   16 +++++++++-------
 1 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c
index a4899c6..e6fa8ec 100644
--- a/dlls/comctl32/listview.c
+++ b/dlls/comctl32/listview.c
@@ -2654,15 +2654,17 @@ static void ranges_assert(RANGES ranges, LPCSTR desc, const char *func, int line
     assert (ranges);
     assert (DPA_GetPtrCount(ranges->hdpa) >= 0);
     ranges_dump(ranges);
-    prev = DPA_GetPtr(ranges->hdpa, 0);
     if (DPA_GetPtrCount(ranges->hdpa) > 0)
-	assert (prev->lower >= 0 && prev->lower < prev->upper);
-    for (i = 1; i < DPA_GetPtrCount(ranges->hdpa); i++)
     {
-        curr = DPA_GetPtr(ranges->hdpa, i);
-	assert (prev->upper <= curr->lower);
-	assert (curr->lower < curr->upper);
-	prev = curr;
+	prev = DPA_GetPtr(ranges->hdpa, 0);
+	assert (prev->lower >= 0 && prev->lower < prev->upper);
+	for (i = 1; i < DPA_GetPtrCount(ranges->hdpa); i++)
+	{
+	    curr = DPA_GetPtr(ranges->hdpa, i);
+	    assert (prev->upper <= curr->lower);
+	    assert (curr->lower < curr->upper);
+	    prev = curr;
+	}
     }
     TRACE("--- Done checking---\n");
 }




More information about the wine-cvs mailing list