winecfg: Use FAILED instead of !SUCCEEDED.
This commit is contained in:
parent
f67256fb2f
commit
2daee2e4a2
@ -591,11 +591,11 @@ BOOL browse_for_unix_folder(HWND dialog, WCHAR *pszPath)
|
|||||||
LoadStringW(GetModuleHandle(NULL), IDS_CHOOSE_PATH, pszChoosePath, FILENAME_MAX);
|
LoadStringW(GetModuleHandle(NULL), IDS_CHOOSE_PATH, pszChoosePath, FILENAME_MAX);
|
||||||
|
|
||||||
hr = SHGetDesktopFolder(&pDesktop);
|
hr = SHGetDesktopFolder(&pDesktop);
|
||||||
if (!SUCCEEDED(hr)) return FALSE;
|
if (FAILED(hr)) return FALSE;
|
||||||
|
|
||||||
hr = IShellFolder_ParseDisplayName(pDesktop, NULL, NULL, wszUnixRootDisplayName, NULL,
|
hr = IShellFolder_ParseDisplayName(pDesktop, NULL, NULL, wszUnixRootDisplayName, NULL,
|
||||||
&pidlUnixRoot, NULL);
|
&pidlUnixRoot, NULL);
|
||||||
if (!SUCCEEDED(hr)) {
|
if (FAILED(hr)) {
|
||||||
IShellFolder_Release(pDesktop);
|
IShellFolder_Release(pDesktop);
|
||||||
return FALSE;
|
return FALSE;
|
||||||
}
|
}
|
||||||
@ -612,14 +612,14 @@ BOOL browse_for_unix_folder(HWND dialog, WCHAR *pszPath)
|
|||||||
hr = IShellFolder_GetDisplayNameOf(pDesktop, pidlSelectedPath, SHGDN_FORPARSING,
|
hr = IShellFolder_GetDisplayNameOf(pDesktop, pidlSelectedPath, SHGDN_FORPARSING,
|
||||||
&strSelectedPath);
|
&strSelectedPath);
|
||||||
IShellFolder_Release(pDesktop);
|
IShellFolder_Release(pDesktop);
|
||||||
if (!SUCCEEDED(hr)) {
|
if (FAILED(hr)) {
|
||||||
SHFree(pidlSelectedPath);
|
SHFree(pidlSelectedPath);
|
||||||
return FALSE;
|
return FALSE;
|
||||||
}
|
}
|
||||||
|
|
||||||
hr = StrRetToStrW(&strSelectedPath, pidlSelectedPath, &pszSelectedPath);
|
hr = StrRetToStrW(&strSelectedPath, pidlSelectedPath, &pszSelectedPath);
|
||||||
SHFree(pidlSelectedPath);
|
SHFree(pidlSelectedPath);
|
||||||
if (!SUCCEEDED(hr)) return FALSE;
|
if (FAILED(hr)) return FALSE;
|
||||||
|
|
||||||
lstrcpyW(pszPath, pszSelectedPath);
|
lstrcpyW(pszPath, pszSelectedPath);
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user