Francois Gouget : ole32/tests: Replace some '#if 0's with 'if (0)'s.

Alexandre Julliard julliard at wine.codeweavers.com
Thu Dec 14 14:24:00 CST 2006


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

Author: Francois Gouget <fgouget at free.fr>
Date:   Thu Dec 14 18:11:28 2006 +0100

ole32/tests: Replace some '#if 0's with 'if (0)'s.

Fix the code so it compiles.

---

 dlls/ole32/tests/marshal.c |   23 ++++++++++++++---------
 1 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/dlls/ole32/tests/marshal.c b/dlls/ole32/tests/marshal.c
index e99983a..fc733ff 100644
--- a/dlls/ole32/tests/marshal.c
+++ b/dlls/ole32/tests/marshal.c
@@ -1456,9 +1456,11 @@ static void test_proxybuffer(REFIID riid
     ok(refs == 1, "Ref count of outer unknown should have been 1 instead of %d\n", refs);
 
     refs = IPSFactoryBuffer_Release(psfb);
-#if 0 /* not reliable on native. maybe it leaks references! */
-    ok(refs == 0, "Ref-count leak of %ld on IPSFactoryBuffer\n", refs);
-#endif
+    if (0)
+    {
+    /* not reliable on native. maybe it leaks references! */
+    ok(refs == 0, "Ref-count leak of %d on IPSFactoryBuffer\n", refs);
+    }
 
     refs = IUnknown_Release((IUnknown *)lpvtbl);
     ok(refs == 0, "Ref-count leak of %d on IRpcProxyBuffer\n", refs);
@@ -1487,9 +1489,11 @@ static void test_stubbuffer(REFIID riid)
     ok_ole_success(hr, IPSFactoryBuffer_CreateStub);
 
     refs = IPSFactoryBuffer_Release(psfb);
-#if 0 /* not reliable on native. maybe it leaks references */
-    ok(refs == 0, "Ref-count leak of %ld on IPSFactoryBuffer\n", refs);
-#endif
+    if (0)
+    {
+    /* not reliable on native. maybe it leaks references */
+    ok(refs == 0, "Ref-count leak of %d on IPSFactoryBuffer\n", refs);
+    }
 
     ok_more_than_one_lock();
 
@@ -1732,11 +1736,12 @@ static void test_freethreadedmarshaler(v
 
 /* native doesn't allow us to unmarshal or release the stream data,
  * presumably because it wants us to call CoMarshalInterface instead */
-#if 0
+    if (0)
+    {
     /* local normal marshaling */
 
     IStream_Seek(pStream, llZero, STREAM_SEEK_SET, NULL);
-    hr = IMarshal_MarshalInterface(pFTMarshal, pStream, IID_IClassFactory, (IUnknown*)&Test_ClassFactory, MSHCTX_LOCAL, NULL, MSHLFLAGS_NORMAL);
+    hr = IMarshal_MarshalInterface(pFTMarshal, pStream, &IID_IClassFactory, (IUnknown*)&Test_ClassFactory, MSHCTX_LOCAL, NULL, MSHLFLAGS_NORMAL);
     ok_ole_success(hr, IMarshal_MarshalInterface);
 
     ok_more_than_one_lock();
@@ -1748,7 +1753,7 @@ static void test_freethreadedmarshaler(v
     ok_ole_success(hr, IMarshal_ReleaseMarshalData);
 
     ok_no_locks();
-#endif
+    }
 
     /* inproc table-strong marshaling */
 




More information about the wine-cvs mailing list