Don't fail ModernSHBrowseForFolderW when DefaultFolder cannot be set. (…

…#529)
M2Team · Dec 8, 2024 · e54855e · e54855e
1 parent 8df266d
commit e54855e
Showing 1 changed file with 13 additions and 15 deletions.
diff --git a/NanaZip.Shared/ModernWin32FileBrowser.cpp b/NanaZip.Shared/ModernWin32FileBrowser.cpp
@@ -51,24 +51,22 @@ EXTERN_C PIDLIST_ABSOLUTE WINAPI ModernSHBrowseForFolderW(
                         nullptr,
                         IID_PPV_ARGS(&DefaultFolder))))
                     {
-                        if (SUCCEEDED(FileDialog->SetDefaultFolder(
-                            DefaultFolder)))
+                        FileDialog->SetDefaultFolder(DefaultFolder);
+
+                        DefaultFolder->Release();
+                    }
+
+                    if (SUCCEEDED(FileDialog->Show(
+                        lpbi->hwndOwner)))
+                    {
+                        IShellItem* Result = nullptr;
+                        if (SUCCEEDED(FileDialog->GetResult(
+                            &Result)))
                         {
-                            if (SUCCEEDED(FileDialog->Show(
-                                lpbi->hwndOwner)))
-                            {
-                                IShellItem* Result = nullptr;
-                                if (SUCCEEDED(FileDialog->GetResult(
-                                    &Result)))
-                                {
-                                    ::SHGetIDListFromObject(Result, &IDList);
+                            ::SHGetIDListFromObject(Result, &IDList);
 
-                                    Result->Release();
-                                }
-                            }
+                            Result->Release();
                         }
-
-                        DefaultFolder->Release();
                     }
                 }
             }