Alexandre Julliard : rpcrt4/tests: Remove tests for specific values of bit fields.

Alexandre Julliard julliard at winehq.org
Fri Oct 9 09:24:19 CDT 2009


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

Author: Alexandre Julliard <julliard at winehq.org>
Date:   Fri Oct  9 16:09:20 2009 +0200

rpcrt4/tests: Remove tests for specific values of bit fields.

---

 dlls/rpcrt4/tests/ndr_marshall.c |   19 ++-----------------
 1 files changed, 2 insertions(+), 17 deletions(-)

diff --git a/dlls/rpcrt4/tests/ndr_marshall.c b/dlls/rpcrt4/tests/ndr_marshall.c
index f4ef592..9781115 100644
--- a/dlls/rpcrt4/tests/ndr_marshall.c
+++ b/dlls/rpcrt4/tests/ndr_marshall.c
@@ -1184,30 +1184,15 @@ static void test_common_stub_data( const char *prefix, const MIDL_STUB_MESSAGE *
         "%s: pAsyncMsg should have been set to zero instead of %d\n", prefix, stubMsg->PointerLength );
     TEST_ZERO(fInDontFree, "%d");
     TEST_ZERO(fDontCallFreeInst, "%d");
-    ok(stubMsg->fInOnlyParam == 0 ||
-       stubMsg->fInOnlyParam == -1, /* Vista */
-       "%s: fInOnlyParam should have been set to 0 or -1 instead of %d\n", prefix, stubMsg->fInOnlyParam);
-    ok( stubMsg->fHasReturn == 0 ||
-        broken(stubMsg->fHasReturn == -1), /* win9x, nt4 */
+    ok( stubMsg->fHasReturn == 0 || broken(stubMsg->fHasReturn), /* win9x, nt4 */
         "%s: fHasReturn should have been set to zero instead of %d\n", prefix, stubMsg->fHasReturn );
     TEST_ZERO(fHasExtensions, "%d");
     TEST_ZERO(fHasNewCorrDesc, "%d");
-    ok(stubMsg->fIsIn == 0 ||
-       broken(stubMsg->fIsIn == -1), /* win9x, nt4 */
+    ok(stubMsg->fIsIn == 0 || broken(stubMsg->fIsIn), /* win9x, nt4 */
        "%s: fIsIn should have been set to 0 instead of %d\n", prefix, stubMsg->fIsIn);
-    ok(stubMsg->fIsOut == 0 ||
-       stubMsg->fIsOut == -1, /* XP-SP3 */
-       "%s: fIsOut should have been set to 0 or -1 instead of %d\n", prefix, stubMsg->fIsOut);
     TEST_ZERO(fIsOicf, "%d");
     ok(stubMsg->fBufferValid == 0,
        "%s: fBufferValid should have been set to 0 instead of %d\n", prefix, stubMsg->fBufferValid);
-    ok(stubMsg->fHasMemoryValidateCallback == 0 ||
-       stubMsg->fHasMemoryValidateCallback == -1, /* XP-SP3 */
-       "%s: fHasMemoryValidateCallback should have been set to 0 or -1 instead of %d\n",
-       prefix, stubMsg->fHasMemoryValidateCallback);
-    ok(stubMsg->fInFree == 0 ||
-       stubMsg->fInFree == -1, /* XP-SP3 */
-       "%s: fInFree should have been set to 0 or -1 instead of %d\n", prefix, stubMsg->fInFree);
     TEST_ZERO(fNeedMCCP, "%d");
     ok(stubMsg->fUnused == 0 ||
        stubMsg->fUnused == -2, /* Vista */




More information about the wine-cvs mailing list