ole32/tests: Remove unused argument from expect() helper.
Michael Stefaniuc
mstefani at redhat.de
Fri Nov 28 07:38:28 CST 2014
---
Supersedes "ole32/tests: Simplify ok() check"
dlls/ole32/tests/propvariant.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/dlls/ole32/tests/propvariant.c b/dlls/ole32/tests/propvariant.c
index 1c9c447..08496de 100644
--- a/dlls/ole32/tests/propvariant.c
+++ b/dlls/ole32/tests/propvariant.c
@@ -130,7 +130,7 @@ static const char* wine_vtypes[VT_CLSID+1] =
};
-static void expect(HRESULT hr, VARTYPE vt, BOOL copy)
+static void expect(HRESULT hr, VARTYPE vt)
{
int idx = vt & VT_TYPEMASK;
BYTE flags;
@@ -158,7 +158,7 @@ static void expect(HRESULT hr, VARTYPE vt, BOOL copy)
}
if(flags == PROP_INV)
- ok(hr == copy ? DISP_E_BADVARTYPE : STG_E_INVALIDPARAMETER, "%s (%s): got %08x\n", wine_vtypes[idx], modifier, hr);
+ ok(hr == STG_E_INVALIDPARAMETER, "%s (%s): got %08x\n", wine_vtypes[idx], modifier, hr);
else if(flags == PROP_V0)
ok(hr == S_OK, "%s (%s): got %08x\n", wine_vtypes[idx], modifier, hr);
else if(flags & PROP_TODO)
@@ -210,7 +210,7 @@ static void test_validtypes(void)
vt = propvar.vt = i;
memset(©, 0x77, sizeof(copy));
hr = PropVariantCopy(©, &propvar);
- expect(hr, vt, TRUE);
+ expect(hr, vt);
if (hr == S_OK)
{
ok(copy.vt == propvar.vt, "expected %d, got %d\n", propvar.vt, copy.vt);
@@ -224,7 +224,7 @@ static void test_validtypes(void)
ok(!ret || broken(ret) /* win2000 */, "%d: copy should stay unchanged\n", i);
}
hr = PropVariantClear(&propvar);
- expect(hr, vt, FALSE);
+ expect(hr, vt);
ok(propvar.vt == 0, "expected 0, got %d\n", propvar.vt);
ok(U(propvar).uhVal.QuadPart == 0, "%u: expected 0, got %#x/%#x\n",
i, U(propvar).uhVal.u.LowPart, U(propvar).uhVal.u.HighPart);
@@ -234,7 +234,7 @@ static void test_validtypes(void)
vt = propvar.vt = i | VT_ARRAY;
memset(©, 0x77, sizeof(copy));
hr = PropVariantCopy(©, &propvar);
- expect(hr, vt, TRUE);
+ expect(hr, vt);
if (hr == S_OK)
{
ok(copy.vt == propvar.vt, "expected %d, got %d\n", propvar.vt, copy.vt);
@@ -247,7 +247,7 @@ static void test_validtypes(void)
ok(!ret || broken(ret) /* win2000 */, "%d: copy should stay unchanged\n", i);
}
hr = PropVariantClear(&propvar);
- expect(hr, vt, FALSE);
+ expect(hr, vt);
ok(propvar.vt == 0, "expected 0, got %d\n", propvar.vt);
ok(U(propvar).uhVal.QuadPart == 0, "%u: expected 0, got %#x/%#x\n",
i, U(propvar).uhVal.u.LowPart, U(propvar).uhVal.u.HighPart);
@@ -258,7 +258,7 @@ static void test_validtypes(void)
vt = propvar.vt = i | VT_VECTOR;
memset(©, 0x77, sizeof(copy));
hr = PropVariantCopy(©, &propvar);
- expect(hr, vt, TRUE);
+ expect(hr, vt);
if (hr == S_OK)
{
ok(copy.vt == propvar.vt, "expected %d, got %d\n", propvar.vt, copy.vt);
@@ -271,7 +271,7 @@ static void test_validtypes(void)
ok(!ret || broken(ret) /* win2000 */, "%d: copy should stay unchanged\n", i);
}
hr = PropVariantClear(&propvar);
- expect(hr, vt, FALSE);
+ expect(hr, vt);
ok(propvar.vt == 0, "expected 0, got %d\n", propvar.vt);
ok(U(propvar).uhVal.QuadPart == 0, "%u: expected 0, got %#x/%#x\n",
i, U(propvar).uhVal.u.LowPart, U(propvar).uhVal.u.HighPart);
@@ -281,7 +281,7 @@ static void test_validtypes(void)
vt = propvar.vt = i | VT_BYREF;
memset(©, 0x77, sizeof(copy));
hr = PropVariantCopy(©, &propvar);
- expect(hr, vt, TRUE);
+ expect(hr, vt);
if (hr == S_OK)
{
ok(copy.vt == propvar.vt, "expected %d, got %d\n", propvar.vt, copy.vt);
@@ -295,7 +295,7 @@ static void test_validtypes(void)
ok(!ret || broken(ret) /* win2000 */, "%d: copy should stay unchanged\n", i);
}
hr = PropVariantClear(&propvar);
- expect(hr, vt, FALSE);
+ expect(hr, vt);
ok(propvar.vt == 0, "expected 0, got %d\n", propvar.vt);
ok(U(propvar).uhVal.QuadPart == 0, "%u: expected 0, got %#x/%#x\n",
i, U(propvar).uhVal.u.LowPart, U(propvar).uhVal.u.HighPart);
--
1.9.3
More information about the wine-patches
mailing list