Eric Pouech : msi: Don't call PropVariantClear on uninitialized variants.

Alexandre Julliard julliard at wine.codeweavers.com
Thu Nov 9 06:08:56 CST 2006


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

Author: Eric Pouech <eric.pouech at wanadoo.fr>
Date:   Wed Nov  8 21:47:09 2006 +0100

msi: Don't call PropVariantClear on uninitialized variants.

---

 dlls/msi/tests/suminfo.c |    8 --------
 1 files changed, 0 insertions(+), 8 deletions(-)

diff --git a/dlls/msi/tests/suminfo.c b/dlls/msi/tests/suminfo.c
index 0122936..869495d 100644
--- a/dlls/msi/tests/suminfo.c
+++ b/dlls/msi/tests/suminfo.c
@@ -299,50 +299,42 @@ static void test_create_database_binary(
     r = IPropertyStorage_SetClass( ps, &FMTID_SummaryInformation );
     ok( r == S_OK, "failed to set class\n");
 
-    PropVariantClear( &propvar[0] );
     propspec[0].ulKind = PRSPEC_PROPID;
     U(propspec[0]).propid = PID_TITLE;
     propvar[0].vt = VT_LPSTR;
     U(propvar[0]).pszVal = LOSE_CONST("test title");
 
-    PropVariantClear( &propvar[1] );
     propspec[1].ulKind = PRSPEC_PROPID;
     U(propspec[1]).propid = PID_SUBJECT;
     propvar[1].vt = VT_LPSTR;
     U(propvar[1]).pszVal = LOSE_CONST("msi suminfo / property storage test");
 
-    PropVariantClear( &propvar[2] );
     propspec[2].ulKind = PRSPEC_PROPID;
     U(propspec[2]).propid = PID_AUTHOR;
     propvar[2].vt = VT_LPSTR;
     U(propvar[2]).pszVal = LOSE_CONST("mike_m");
 
-    PropVariantClear( &propvar[3] );
     propspec[3].ulKind = PRSPEC_PROPID;
     U(propspec[3]).propid = PID_TEMPLATE;
     propvar[3].vt = VT_LPSTR;
     U(propvar[3]).pszVal = LOSE_CONST(";1033");  /* actually the string table's codepage */
 
-    PropVariantClear( &propvar[4] );
     propspec[4].ulKind = PRSPEC_PROPID;
     U(propspec[4]).propid = PID_REVNUMBER;
     propvar[4].vt = VT_LPSTR;
     U(propvar[4]).pszVal = LOSE_CONST("{913B8D18-FBB6-4CAC-A239-C74C11E3FA74}");
 
-    PropVariantClear( &propvar[5] );
     propspec[5].ulKind = PRSPEC_PROPID;
     U(propspec[5]).propid = PID_PAGECOUNT;
     propvar[5].vt = VT_I4;
     U(propvar[5]).lVal = 100;
 
-    PropVariantClear( &propvar[6] );
     propspec[6].ulKind = PRSPEC_PROPID;
     U(propspec[6]).propid = PID_WORDCOUNT;
     propvar[6].vt = VT_I4;
     U(propvar[6]).lVal = 0;
 
     /* MSDN says that PID_LASTPRINTED should be a VT_FILETIME... */
-    PropVariantClear( &propvar[7] );
     propspec[7].ulKind = PRSPEC_PROPID;
     U(propspec[7]).propid = PID_LASTPRINTED;
     propvar[7].vt = VT_LPSTR;




More information about the wine-cvs mailing list