diff --git a/Source/Android/jni/MainAndroid.cpp b/Source/Android/jni/MainAndroid.cpp index fa92372f6a..07846711e6 100644 --- a/Source/Android/jni/MainAndroid.cpp +++ b/Source/Android/jni/MainAndroid.cpp @@ -152,7 +152,7 @@ void Host_UpdateProgressDialog(const char* caption, int position, int total) { } -static bool MsgAlert(const char* caption, const char* text, bool yes_no, int /*Style*/) +static bool MsgAlert(const char* caption, const char* text, bool yes_no, MsgType /*style*/) { __android_log_print(ANDROID_LOG_ERROR, DOLPHIN_TAG, "%s:%s", caption, text); diff --git a/Source/Core/Common/MsgHandler.cpp b/Source/Core/Common/MsgHandler.cpp index 77c5547996..670023b491 100644 --- a/Source/Core/Common/MsgHandler.cpp +++ b/Source/Core/Common/MsgHandler.cpp @@ -16,7 +16,7 @@ #include #endif -bool DefaultMsgHandler(const char* caption, const char* text, bool yes_no, int Style); +bool DefaultMsgHandler(const char* caption, const char* text, bool yes_no, MsgType style); static MsgAlertHandler msg_handler = DefaultMsgHandler; static bool AlertEnabled = true; @@ -42,14 +42,14 @@ void SetEnableAlert(bool enable) AlertEnabled = enable; } -std::string GetTranslation(const char* string) +std::string GetStringT(const char* string) { return str_translator(string); } // This is the first stop for gui alerts where the log is updated and the // correct window is shown -bool MsgAlert(bool yes_no, int Style, const char* format, ...) +bool MsgAlert(bool yes_no, MsgType style, const char* format, ...) { // Read message and write it to the log std::string caption; @@ -68,18 +68,18 @@ bool MsgAlert(bool yes_no, int Style, const char* format, ...) crit_caption = str_translator(_trans("Critical")); } - switch (Style) + switch (style) { - case INFORMATION: + case MsgType::Information: caption = info_caption; break; - case QUESTION: + case MsgType::Question: caption = ques_caption; break; - case WARNING: + case MsgType::Warning: caption = warn_caption; break; - case CRITICAL: + case MsgType::Critical: caption = crit_caption; break; } @@ -92,24 +92,24 @@ bool MsgAlert(bool yes_no, int Style, const char* format, ...) ERROR_LOG(MASTER_LOG, "%s: %s", caption.c_str(), buffer); // Don't ignore questions, especially AskYesNo, PanicYesNo could be ignored - if (msg_handler && (AlertEnabled || Style == QUESTION || Style == CRITICAL)) - return msg_handler(caption.c_str(), buffer, yes_no, Style); + if (msg_handler && (AlertEnabled || style == MsgType::Question || style == MsgType::Critical)) + return msg_handler(caption.c_str(), buffer, yes_no, style); return true; } // Default non library dependent panic alert -bool DefaultMsgHandler(const char* caption, const char* text, bool yes_no, int Style) +bool DefaultMsgHandler(const char* caption, const char* text, bool yes_no, MsgType style) { #ifdef _WIN32 - int STYLE = MB_ICONINFORMATION; - if (Style == QUESTION) - STYLE = MB_ICONQUESTION; - if (Style == WARNING) - STYLE = MB_ICONWARNING; + int window_style = MB_ICONINFORMATION; + if (style == MsgType::Question) + window_style = MB_ICONQUESTION; + if (style == MsgType::Warning) + window_style = MB_ICONWARNING; return IDYES == MessageBox(0, UTF8ToTStr(text).c_str(), UTF8ToTStr(caption).c_str(), - STYLE | (yes_no ? MB_YESNO : MB_OK)); + window_style | (yes_no ? MB_YESNO : MB_OK)); #else fprintf(stderr, "%s\n", text); diff --git a/Source/Core/Common/MsgHandler.h b/Source/Core/Common/MsgHandler.h index b6fc9c32b3..dcb79f4196 100644 --- a/Source/Core/Common/MsgHandler.h +++ b/Source/Core/Common/MsgHandler.h @@ -7,22 +7,22 @@ #include // Message alerts -enum MSG_TYPE +enum class MsgType { - INFORMATION, - QUESTION, - WARNING, - CRITICAL + Information, + Question, + Warning, + Critical }; -typedef bool (*MsgAlertHandler)(const char* caption, const char* text, bool yes_no, int Style); +typedef bool (*MsgAlertHandler)(const char* caption, const char* text, bool yes_no, MsgType style); typedef std::string (*StringTranslator)(const char* text); void RegisterMsgAlertHandler(MsgAlertHandler handler); void RegisterStringTranslator(StringTranslator translator); -std::string GetTranslation(const char* string); -bool MsgAlert(bool yes_no, int Style, const char* format, ...) +std::string GetStringT(const char* string); +bool MsgAlert(bool yes_no, MsgType style, const char* format, ...) #ifdef __GNUC__ __attribute__((format(printf, 3, 4))) #endif @@ -30,31 +30,28 @@ bool MsgAlert(bool yes_no, int Style, const char* format, ...) void SetEnableAlert(bool enable); #ifdef _WIN32 -#define SuccessAlert(format, ...) MsgAlert(false, INFORMATION, format, __VA_ARGS__) -#define PanicAlert(format, ...) MsgAlert(false, WARNING, format, __VA_ARGS__) -#define PanicYesNo(format, ...) MsgAlert(true, WARNING, format, __VA_ARGS__) -#define AskYesNo(format, ...) MsgAlert(true, QUESTION, format, __VA_ARGS__) -#define CriticalAlert(format, ...) MsgAlert(false, CRITICAL, format, __VA_ARGS__) +#define SuccessAlert(format, ...) MsgAlert(false, MsgType::Information, format, __VA_ARGS__) +#define PanicAlert(format, ...) MsgAlert(false, MsgType::Warning, format, __VA_ARGS__) +#define PanicYesNo(format, ...) MsgAlert(true, MsgType::Warning, format, __VA_ARGS__) +#define AskYesNo(format, ...) MsgAlert(true, MsgType::Question, format, __VA_ARGS__) +#define CriticalAlert(format, ...) MsgAlert(false, MsgType::Critical, format, __VA_ARGS__) // Use these macros (that do the same thing) if the message should be translated. -#define SuccessAlertT(format, ...) MsgAlert(false, INFORMATION, format, __VA_ARGS__) -#define PanicAlertT(format, ...) MsgAlert(false, WARNING, format, __VA_ARGS__) -#define PanicYesNoT(format, ...) MsgAlert(true, WARNING, format, __VA_ARGS__) -#define AskYesNoT(format, ...) MsgAlert(true, QUESTION, format, __VA_ARGS__) -#define CriticalAlertT(format, ...) MsgAlert(false, CRITICAL, format, __VA_ARGS__) +#define SuccessAlertT(format, ...) MsgAlert(false, MsgType::Information, format, __VA_ARGS__) +#define PanicAlertT(format, ...) MsgAlert(false, MsgType::Warning, format, __VA_ARGS__) +#define PanicYesNoT(format, ...) MsgAlert(true, MsgType::Warning, format, __VA_ARGS__) +#define AskYesNoT(format, ...) MsgAlert(true, MsgType::Question, format, __VA_ARGS__) +#define CriticalAlertT(format, ...) MsgAlert(false, MsgType::Critical, format, __VA_ARGS__) -#define GetStringT(string) GetTranslation(string) #else -#define SuccessAlert(format, ...) MsgAlert(false, INFORMATION, format, ##__VA_ARGS__) -#define PanicAlert(format, ...) MsgAlert(false, WARNING, format, ##__VA_ARGS__) -#define PanicYesNo(format, ...) MsgAlert(true, WARNING, format, ##__VA_ARGS__) -#define AskYesNo(format, ...) MsgAlert(true, QUESTION, format, ##__VA_ARGS__) -#define CriticalAlert(format, ...) MsgAlert(false, CRITICAL, format, ##__VA_ARGS__) +#define SuccessAlert(format, ...) MsgAlert(false, MsgType::Information, format, ##__VA_ARGS__) +#define PanicAlert(format, ...) MsgAlert(false, MsgType::Warning, format, ##__VA_ARGS__) +#define PanicYesNo(format, ...) MsgAlert(true, MsgType::Warning, format, ##__VA_ARGS__) +#define AskYesNo(format, ...) MsgAlert(true, MsgType::Question, format, ##__VA_ARGS__) +#define CriticalAlert(format, ...) MsgAlert(false, MsgType::Critical, format, ##__VA_ARGS__) // Use these macros (that do the same thing) if the message should be translated. -#define SuccessAlertT(format, ...) MsgAlert(false, INFORMATION, format, ##__VA_ARGS__) -#define PanicAlertT(format, ...) MsgAlert(false, WARNING, format, ##__VA_ARGS__) -#define PanicYesNoT(format, ...) MsgAlert(true, WARNING, format, ##__VA_ARGS__) -#define AskYesNoT(format, ...) MsgAlert(true, QUESTION, format, ##__VA_ARGS__) -#define CriticalAlertT(format, ...) MsgAlert(false, CRITICAL, format, ##__VA_ARGS__) - -#define GetStringT(string) GetTranslation(string) +#define SuccessAlertT(format, ...) MsgAlert(false, MsgType::Information, format, ##__VA_ARGS__) +#define PanicAlertT(format, ...) MsgAlert(false, MsgType::Warning, format, ##__VA_ARGS__) +#define PanicYesNoT(format, ...) MsgAlert(true, MsgType::Warning, format, ##__VA_ARGS__) +#define AskYesNoT(format, ...) MsgAlert(true, MsgType::Question, format, ##__VA_ARGS__) +#define CriticalAlertT(format, ...) MsgAlert(false, MsgType::Critical, format, ##__VA_ARGS__) #endif diff --git a/Source/Core/DolphinWX/Main.cpp b/Source/Core/DolphinWX/Main.cpp index 1a5fbad188..a462dcfeb1 100644 --- a/Source/Core/DolphinWX/Main.cpp +++ b/Source/Core/DolphinWX/Main.cpp @@ -64,7 +64,7 @@ IMPLEMENT_APP(DolphinApp) -bool wxMsgAlert(const char*, const char*, bool, int); +bool wxMsgAlert(const char*, const char*, bool, MsgType); std::string wxStringTranslator(const char*); CFrame* main_frame = nullptr; @@ -351,7 +351,7 @@ void DolphinApp::OnIdle(wxIdleEvent& ev) // ------------ // Talk to GUI -bool wxMsgAlert(const char* caption, const char* text, bool yes_no, int /*Style*/) +bool wxMsgAlert(const char* caption, const char* text, bool yes_no, MsgType /*style*/) { if (wxIsMainThread()) {