shell32: Use FAILED instead of !SUCCEEDED.
Michael Stefaniuc
mstefani at redhat.de
Tue Jul 19 08:06:09 CDT 2011
---
dlls/shell32/shellpath.c | 4 ++--
dlls/shell32/tests/brsfolder.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/dlls/shell32/shellpath.c b/dlls/shell32/shellpath.c
index 4b14739..4407056 100644
--- a/dlls/shell32/shellpath.c
+++ b/dlls/shell32/shellpath.c
@@ -3423,10 +3423,10 @@ static HRESULT get_known_folder_path(
{
/* get parent's known folder path (recursive) */
hr = get_known_folder_registry_path(NULL, parentGuid, &parentRegistryPath);
- if(!SUCCEEDED(hr)) return hr;
+ if(FAILED(hr)) return hr;
hr = get_known_folder_path(parentGuid, parentRegistryPath, &parentPath);
- if(!SUCCEEDED(hr)) {
+ if(FAILED(hr)) {
HeapFree(GetProcessHeap(), 0, parentRegistryPath);
return hr;
}
diff --git a/dlls/shell32/tests/brsfolder.c b/dlls/shell32/tests/brsfolder.c
index 4dea03e..7b87967 100644
--- a/dlls/shell32/tests/brsfolder.c
+++ b/dlls/shell32/tests/brsfolder.c
@@ -204,7 +204,7 @@ static void test_click_make_new_folder_button(void)
test_folder_pathW, MAX_PATH);
hr = SHGetDesktopFolder(&test_folder_object);
ok (SUCCEEDED(hr), "SHGetDesktopFolder failed with hr 0x%08x\n", hr);
- if (!SUCCEEDED(hr)) {
+ if (FAILED(hr)) {
skip("SHGetDesktopFolder failed - skipping\n");
return;
}
@@ -331,7 +331,7 @@ static void test_selection(void)
hr = SHGetDesktopFolder(&desktop_object);
ok (SUCCEEDED(hr), "SHGetDesktopFolder failed with hr 0x%08x\n", hr);
- if (!SUCCEEDED(hr)) {
+ if (FAILED(hr)) {
skip("SHGetDesktopFolder failed - skipping\n");
return;
}
--
1.7.6
More information about the wine-patches
mailing list