isledecomp/isle-portable
1
0
Fork 0
mirror of https://github.com/isledecomp/isle-portable.git synced 2026-02-03 12:31:15 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

isleapp: Use boolean return in LoadConfig

Browse Source 
This to check if the config loaded correctly or not.
Seems to be a good solution to the file loading issue.

Maybe there's a better solution
SetupWindow failing because of a config erroring seems a bit odd.
This commit is contained in:
AllMeatball 2025-05-12 15:29:05 -05:00
parent 39f954adad
commit 43f58f04f3
1 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
ISLE/isleapp.cpp
View File

@ -447,7 +447,10 @@ MxU8 IsleApp::MapMouseButtonFlagsToModifier(SDL_MouseButtonFlags p_flags)
// FUNCTION: ISLE 0x4023e0
MxResult IsleApp::SetupWindow()
{
LoadConfig();
if (!LoadConfig()) {
return FAILURE;
}
SetupVideoFlags(
m_fullScreen,
m_flipSurfaces,
@ -522,7 +525,7 @@ MxResult IsleApp::SetupWindow()
}
// FUNCTION: ISLE 0x4028d0
void IsleApp::LoadConfig()
bool IsleApp::LoadConfig()
{
char* prefPath = SDL_GetPrefPath("isledecomp", "isle");
char* iniConfig;
@ -549,9 +552,11 @@ void IsleApp::LoadConfig()
SDL_LogInfo(SDL_LOG_CATEGORY_APPLICATION, "Loading sane defaults");
FILE* iniFP = fopen(iniConfig, "wb");
if (!iniFP) {
SDL_LogError(SDL_LOG_CATEGORY_APPLICATION, "Failed to write config at '%s': %s", iniConfig, strerror(errno));
if (iniFP) {
fclose(iniFP);
} else {
SDL_LogError(SDL_LOG_CATEGORY_APPLICATION, "Failed to write config at '%s': %s", iniConfig, strerror(errno));
return false;
}
dict = iniparser_load(iniConfig);
@ -639,6 +644,8 @@ void IsleApp::LoadConfig()
iniparser_freedict(dict);
delete[] iniConfig;
SDL_free(prefPath);
return true;
}
// FUNCTION: ISLE 0x402c20