Mike McCormack : msi: Make opening and closing 4000 handle into two tests, not 20000.

Alexandre Julliard julliard at wine.codeweavers.com
Thu Sep 7 05:07:29 CDT 2006


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

Author: Mike McCormack <mike at codeweavers.com>
Date:   Fri Sep  1 17:30:49 2006 +0900

msi: Make opening and closing 4000 handle into two tests, not 20000.

---

 dlls/msi/tests/db.c |   15 ++++++++-------
 1 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/dlls/msi/tests/db.c b/dlls/msi/tests/db.c
index 52e7fdf..878013b 100644
--- a/dlls/msi/tests/db.c
+++ b/dlls/msi/tests/db.c
@@ -1371,22 +1371,23 @@ static void test_handle_limit(void)
         static char szQueryBuf[256] = "SELECT * from `_Tables`";
         hviews[i] = 0xdeadbeeb;
         r = MsiDatabaseOpenView(hdb, szQueryBuf, &hviews[i]);
-        ok( r == ERROR_SUCCESS, "failed to open query %d\n", i);
-        ok( hviews[i] != 0xdeadbeeb, "no handle set\n");
-        ok( hviews[i] != 0, "%d'th handle is NULL\n", i);
-        if (!hviews[i])
+        if( r != ERROR_SUCCESS || hviews[i] == 0xdeadbeeb || 
+            hviews[i] == 0 || (i && (hviews[i] == hviews[i-1])))
             break;
-        ok( (i == 0 || (hviews[i] != hviews[i-1])),
-            "got handle %p twice\n", (void *) hviews[i] );
     }
 
+    ok( i == MY_NVIEWS, "problem opening views\n");
+
     for (i=0; i<MY_NVIEWS; i++) {
         if (hviews[i] != 0 && hviews[i] != 0xdeadbeeb) {
             r = MsiCloseHandle(hviews[i]);
-            ok( r == ERROR_SUCCESS, "failed to close view handle %d\n", i);
+            if (r != ERROR_SUCCESS)
+                break;
         }
     }
 
+    ok( i == MY_NVIEWS, "problem closing views\n");
+
     r = MsiCloseHandle(hdb);
     ok( r == ERROR_SUCCESS, "failed to close database\n");
 }




More information about the wine-cvs mailing list