devenum/tests: Fix return value check for IEnumMoniker_Next.
Thomas Faber
thomas.faber at reactos.org
Tue Jun 13 09:03:05 CDT 2017
IEnumMoniker::Next returns S_FALSE when there are no (more) items, so
checking SUCCEEDED does not make sense.
-------------- next part --------------
From 27e60ac9ed3b10c7dfc4d6ec08aac8d8ff74e819 Mon Sep 17 00:00:00 2001
From: Thomas Faber <thomas.faber at reactos.org>
Date: Tue, 13 Jun 2017 11:50:01 +0200
Subject: devenum/tests: Fix return value check for IEnumMoniker_Next.
Signed-off-by: Thomas Faber <thomas.faber at reactos.org>
---
dlls/devenum/tests/devenum.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/dlls/devenum/tests/devenum.c b/dlls/devenum/tests/devenum.c
index c2173fedc10..5a41183655a 100644
--- a/dlls/devenum/tests/devenum.c
+++ b/dlls/devenum/tests/devenum.c
@@ -154,8 +154,8 @@ static void test_moniker_isequal(void)
ok(SUCCEEDED(res), "Cannot create enum moniker (res = %x)\n", res);
if (SUCCEEDED(res))
{
- if (SUCCEEDED(IEnumMoniker_Next(enum_moniker0, 1, &moniker0, NULL)) &&
- SUCCEEDED(IEnumMoniker_Next(enum_moniker0, 1, &moniker1, NULL)))
+ if (IEnumMoniker_Next(enum_moniker0, 1, &moniker0, NULL) == S_OK &&
+ IEnumMoniker_Next(enum_moniker0, 1, &moniker1, NULL) == S_OK)
{
res = IMoniker_IsEqual(moniker0, moniker1);
ok(res == S_FALSE, "IMoniker_IsEqual should fail (res = %x)\n", res);
@@ -177,8 +177,8 @@ static void test_moniker_isequal(void)
ok(SUCCEEDED(res), "Cannot create enum moniker (res = %x)\n", res);
if (SUCCEEDED(res))
{
- if (SUCCEEDED(IEnumMoniker_Next(enum_moniker0, 1, &moniker0, NULL)) &&
- SUCCEEDED(IEnumMoniker_Next(enum_moniker1, 1, &moniker1, NULL)))
+ if (IEnumMoniker_Next(enum_moniker0, 1, &moniker0, NULL) == S_OK &&
+ IEnumMoniker_Next(enum_moniker1, 1, &moniker1, NULL) == S_OK)
{
res = IMoniker_IsEqual(moniker0, moniker1);
ok(res == S_FALSE, "IMoniker_IsEqual should failed (res = %x)\n", res);
@@ -201,8 +201,8 @@ static void test_moniker_isequal(void)
ok(SUCCEEDED(res), "Cannot create enum moniker (res = %x)\n", res);
if (SUCCEEDED(res))
{
- if (SUCCEEDED(IEnumMoniker_Next(enum_moniker0, 1, &moniker0, NULL)) &&
- SUCCEEDED(IEnumMoniker_Next(enum_moniker1, 1, &moniker1, NULL)))
+ if (IEnumMoniker_Next(enum_moniker0, 1, &moniker0, NULL) == S_OK &&
+ IEnumMoniker_Next(enum_moniker1, 1, &moniker1, NULL) == S_OK)
{
res = IMoniker_IsEqual(moniker0, moniker1);
ok(res == S_OK, "IMoniker_IsEqual failed (res = %x)\n", res);
--
2.12.2.windows.2
More information about the wine-patches
mailing list