Common/MsgHandler: Brace if statement in MsgAlert

The condition travels more than one line, so the body should be braced.
While we're at it, make the comparison against nullptr explicit.

1 files changed, 4 insertions, 1 deletions

diff --git a/Source/Core/Common/MsgHandler.cpp b/Source/Core/Common/MsgHandler.cpp
index 4744a5ff44..41979c1ef0 100644
--- a/Source/Core/Common/MsgHandler.cpp
+++ b/Source/Core/Common/MsgHandler.cpp
@@ -121,8 +121,11 @@ bool MsgAlert(bool yes_no, MsgType 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 (s_msg_handler && (s_alert_enabled || style == MsgType::Question || style == MsgType::Critical))
+  if (s_msg_handler != nullptr &&
+      (s_alert_enabled || style == MsgType::Question || style == MsgType::Critical))
+  {
     return s_msg_handler(caption.c_str(), buffer, yes_no, style);
+  }
 
   return true;
 }