oleaut: Add missing SysFreeString to the Var{Mul,Add,Cmp} tests

Michael Stefaniuc mstefani at redhat.de
Tue Jan 17 16:03:24 CST 2006


Hello Alexandre,

i forgot to add SysFreeString's to the Var{Add,Cmp,Mul} tests in my
previous patches. On the one hand it's cleaner this way on the other
hand the tests are short lived and memory is freed anyway when the
winetest run terminates.

bye
	michael
-- 
Michael Stefaniuc               Tel.: +49-711-96437-199
Sr. Network Engineer            Fax.: +49-711-96437-111
Red Hat GmbH                    Email: mstefani at redhat.com
Hauptstaetterstr. 58            http://www.redhat.de/
D-70178 Stuttgart
-------------- next part --------------
oleaut: Add missing SysFreeString to the VarMul, VarAdd and VarCmp tests.

---

 dlls/oleaut32/tests/vartest.c |   17 +++++++++++++++++
 1 files changed, 17 insertions(+), 0 deletions(-)

9b795a4ce062f6290e1c52c951ddb0e346362aee
diff --git a/dlls/oleaut32/tests/vartest.c b/dlls/oleaut32/tests/vartest.c
index 85fb76e..65c5cfd 100644
--- a/dlls/oleaut32/tests/vartest.c
+++ b/dlls/oleaut32/tests/vartest.c
@@ -4731,6 +4731,9 @@ static void test_VarMul(void)
     ok(hres == S_OK && V_VT(&result) == VT_DECIMAL, "VarMul: expected coerced type VT_DECIMAL, got %s!\n", vtstr(V_VT(&result)));
     hres = VarR8FromDec(&V_DECIMAL(&result), &r);
     ok(hres == S_OK && EQ_DOUBLE(r, 46.2), "VarMul: DECIMAL value %f, expected %f\n", r, (double)46.2);
+
+    SysFreeString(lbstr);
+    SysFreeString(rbstr);
 }
 
 static HRESULT (WINAPI *pVarAdd)(LPVARIANT,LPVARIANT,LPVARIANT);
@@ -4928,6 +4931,9 @@ static void test_VarAdd(void)
     ok(hres == S_OK && V_VT(&result) == VT_DECIMAL, "VarAdd: expected coerced type VT_DECIMAL, got %s!\n", vtstr(V_VT(&result)));
     hres = VarR8FromDec(&V_DECIMAL(&result), &r);
     ok(hres == S_OK && EQ_DOUBLE(r, -15.2), "VarAdd: DECIMAL value %f, expected %f\n", r, (double)-15.2);
+
+    SysFreeString(lbstr);
+    SysFreeString(rbstr);
 }
 
 static HRESULT (WINAPI *pVarCmp)(LPVARIANT,LPVARIANT,LCID,ULONG);
@@ -5180,6 +5186,17 @@ static void test_VarCmp(void)
     VARCMP(R4,1,BSTR,bstr1few,VARCMP_LT); /* bstr1few == 1+1e-8 */
     SETDEC(dec,8,0,0,0x5F5E101);          /* 1+1e-8 */
     VARCMP(R4,1,DECIMAL,dec,VARCMP_LT);
+
+    SysFreeString(bstrhuh);
+    SysFreeString(bstrempty);
+    SysFreeString(bstr0);
+    SysFreeString(bstr1);
+    SysFreeString(bstr7);
+    SysFreeString(bstr42);
+    SysFreeString(bstr1neg);
+    SysFreeString(bstr666neg);
+    SysFreeString(bstr2cents);
+    SysFreeString(bstr1few);
 }
 
 START_TEST(vartest)
-- 
1.1.1


More information about the wine-patches mailing list