Leahnaya/dolphin
1
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2026-05-09 12:24:04 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

MsgHandler: Use log level matching MsgType for alerts

Browse Source 
When calling `ShowMessageAlert` with a given `MsgType`, log the alert
with a `LogLevel` matching the `MsgType` instead of always using
`LogLevel::LERROR`.
This commit is contained in:
Dentomologist 2026-05-06 12:44:43 -07:00
parent 6d5399246e
commit dfda04c4e4
1 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
Source/Core/Common/MsgHandler.cpp
View File

@ -73,6 +73,19 @@ const char* GetCaption(MsgType style)
return "Unhandled caption";
}
}
Log::LogLevel GetMessageAlertLogLevel(const MsgType style)
{
switch (style)
{
case MsgType::Critical:
return Log::LogLevel::LERROR;
case MsgType::Warning:
return Log::LogLevel::LWARNING;
default:
return Log::LogLevel::LINFO;
}
}
} // Anonymous namespace
// Select which of these functions that are used for message boxes. If
@ -108,10 +121,10 @@ static bool ShowMessageAlert(std::string_view text, bool yes_no, Common::Log::Lo
MsgType style, const char* file, int line)
{
const char* caption = GetCaption(style);
const Log::LogLevel level = GetMessageAlertLogLevel(style);
// Directly call GenericLogFmt rather than using the normal log macros so that we can use the
// caller's line file and line number
Common::Log::GenericLogFmt<2>(Common::Log::LogLevel::LERROR, log_type, file, line,
FMT_STRING("{}: {}"), caption, text);
Common::Log::GenericLogFmt<2>(level, log_type, file, line, FMT_STRING("{}: {}"), caption, text);
// Panic alerts.
if (style == MsgType::Warning && s_abort_on_panic_alert)