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