Michael Stefaniuc : shell32: Use FAILED instead of !SUCCEEDED.

[Alexandre Julliard]

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

Author: Michael Stefaniuc <mstefani(a)redhat.de>
Date:   Thu Sep 10 00:28:56 2009 +0200

shell32: Use FAILED instead of !SUCCEEDED.

---

 dlls/shell32/cpanelfolder.c |    3 +--
 dlls/shell32/shellitem.c    |    4 ++--
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/dlls/shell32/cpanelfolder.c b/dlls/shell32/cpanelfolder.c
index af3e0ca..cd2f3b8 100644
--- a/dlls/shell32/cpanelfolder.c
+++ b/dlls/shell32/cpanelfolder.c
@@ -688,8 +688,7 @@ static HRESULT WINAPI ISF_ControlPanel_fnGetDisplayNameOf(IShellFolder2 * iface,
 	    PathAddBackslashW(wszPath);
 	    len = lstrlenW(wszPath);
 
-	    if (!SUCCEEDED
-	      (SHELL32_GetDisplayNameOfChild(iface, pidl, dwFlags | SHGDN_INFOLDER, wszPath + len, MAX_PATH + 1 - len)))
+            if (FAILED(SHELL32_GetDisplayNameOfChild(iface, pidl, dwFlags | SHGDN_INFOLDER, wszPath + len, MAX_PATH + 1 - len)))
 		return E_OUTOFMEMORY;
 	    if (!WideCharToMultiByte(CP_ACP, 0, wszPath, -1, szPath, MAX_PATH, NULL, NULL))
 		wszPath[0] = '\0';
diff --git a/dlls/shell32/shellitem.c b/dlls/shell32/shellitem.c
index 13cb1f6..29f072f 100644
--- a/dlls/shell32/shellitem.c
+++ b/dlls/shell32/shellitem.c
@@ -350,13 +350,13 @@ HRESULT WINAPI SHCreateShellItem(LPCITEMIDLIST pidlParent,
         {
             IPersistFolder2* ppf2Parent;
 
-            if (!SUCCEEDED(IPersistFolder2_QueryInterface(psfParent, &IID_IPersistFolder2, (void**)&ppf2Parent)))
+            if (FAILED(IPersistFolder2_QueryInterface(psfParent, &IID_IPersistFolder2, (void**)&ppf2Parent)))
             {
                 FIXME("couldn't get IPersistFolder2 interface of parent\n");
                 return E_NOINTERFACE;
             }
 
-            if (!SUCCEEDED(IPersistFolder2_GetCurFolder(ppf2Parent, &temp_parent)))
+            if (FAILED(IPersistFolder2_GetCurFolder(ppf2Parent, &temp_parent)))
             {
                 FIXME("couldn't get parent PIDL\n");
                 IPersistFolder2_Release(ppf2Parent);