[WLANWIZ] Compact FAILED() usages

Author: SigmaTel71

dll/shellext/wlanwiz/advsettings.cpp

@@ -2,9 +2,8 @@
  * PROJECT:     ReactOS Shell
  * LICENSE:     LGPL-2.1-or-later (https://spdx.org/licenses/LGPL-2.1-or-later)
  * PURPOSE:     ReactOS Wizard for Wireless Network Connections (WLAN Advanced Settings)
- * COPYRIGHT:   Copyright 2024 Vitaly Orekhov <[email protected]>
+ * COPYRIGHT:   Copyright 2024-2025 Vitaly Orekhov <[email protected]>
  */
-
 #include "main.h"
 
 LRESULT CWlanWizard::OnAdvancedSettings(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL& bHandled)
@@ -17,35 +16,22 @@ LRESULT CWlanWizard::OnAdvancedSettings(WORD wNotifyCode, WORD wID, HWND hWndCtl
 	LPCITEMIDLIST pidlChild = NULL;
 	SHDESCRIPTIONID did = { 0 };
 
-	HRESULT hr = SHGetSpecialFolderLocation(NULL, CSIDL_CONNECTIONS, &pidl);
-
-	if (FAILED(hr))
+	if (FAILED(SHGetSpecialFolderLocation(NULL, CSIDL_CONNECTIONS, &pidl)))
 		goto Exit;
 
-	hr = SHBindToParent(pidl, IID_IShellFolder, reinterpret_cast<LPVOID*>(&sfConn), &pidlChild);
-
-	if (FAILED(hr))
+	if (FAILED(SHBindToParent(pidl, IID_IShellFolder, reinterpret_cast<LPVOID*>(&sfConn), &pidlChild)))
 		goto Exit;
 
-	hr = SHGetDataFromIDListW(sfConn, pidlChild, SHGDFIL_DESCRIPTIONID, &did, sizeof(did));
-
-	if (FAILED(hr))
+	if (FAILED(SHGetDataFromIDListW(sfConn, pidlChild, SHGDFIL_DESCRIPTIONID, &did, sizeof(did))))
 		goto Exit;
 
-	hr = StringFromIID(did.clsid, &lpwszNetConCLSID);
-
-	if (FAILED(hr))
+	if (FAILED(StringFromIID(did.clsid, &lpwszNetConCLSID)))
 		goto Exit;
 
 	ILFree(pidl);
 	sfConn.Release();
 #else
-#if _WIN32_WINNT < _WIN32_WINNT_VISTA
-	HRESULT hr = StringFromIID(CLSID_NetworkConnections, &lpwszNetConCLSID);
-#else
-	HRESULT hr = StringFromIID(CLSID_ConnectionFolder, &lpwszNetConCLSID);
-#endif
-	if (FAILED(hr))
+	if (FAILED(StringFromIID(CLSID_NetworkConnections, &lpwszNetConCLSID)))
 		goto Exit;
 #endif
 	cswNetworkAdapterPath.Format(L"::%s\\::%s", lpwszNetConCLSID, this->m_sGUID);

dll/shellext/wlanwiz/wlanwiz.cpp

@@ -81,12 +81,19 @@ BOOL CWlanWizard::FindWlanDevice(ATL::CString sGUID)
 {
     /* Create WLAN client handle */
     DWORD dwResult = WlanOpenHandle(WLAN_API_VERSION_1_0, NULL, &this->dwNegotiatedVersion, &this->hWlanClient);
-
     if (FAILED(dwResult))
-        return 0; // TODO: handle it
+    {
+        DPRINT1("WlanOpenHandle failed: %lx\n", dwResult);
+        return FALSE;
+    }
 
     /* Get list of all WLAN devices */
     dwResult = WlanEnumInterfaces(this->hWlanClient, NULL, &this->lstWlanInterfaces);
+    if (FAILED(dwResult))
+    {
+        DPRINT1("WlanEnumInterfaces failed: %lx\n", dwResult);
+        return FALSE;
+    }
 
     /* If GUID was supplied on start, we will try to find a device that has it. */
     if (!sGUID.IsEmpty())