Skip to content

Commit f8fea1b

Browse files
committed
INISettingsInterface: Return bool on saving settings
1 parent b5311e5 commit f8fea1b

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

src/frontend-common/ini_settings_interface.cpp

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,17 @@ INISettingsInterface::~INISettingsInterface()
1616
Save();
1717
}
1818

19-
void INISettingsInterface::Save()
19+
bool INISettingsInterface::Save()
2020
{
2121
SI_Error err = m_ini.SaveFile(m_filename.c_str(), false);
2222
if (err != SI_OK)
23+
{
2324
Log_WarningPrintf("Failed to save settings to '%s'.", m_filename.c_str());
24-
else
25-
m_dirty = false;
25+
return false;
26+
}
27+
28+
m_dirty = false;
29+
return true;
2630
}
2731

2832
void INISettingsInterface::Clear()

src/frontend-common/ini_settings_interface.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ class INISettingsInterface : public SettingsInterface
99
INISettingsInterface(std::string filename);
1010
~INISettingsInterface();
1111

12-
void Save();
12+
bool Save();
1313

1414
void Clear() override;
1515

0 commit comments

Comments
 (0)