BreakpointWidget: Move variables closer to usage sites in OnLoad()

In the early-out case, we now don't end up constructing two vectors
that aren't even used. It also keeps relevant code together.

1 files changed, 6 insertions, 7 deletions

diff --git a/Source/Core/DolphinQt2/Debugger/BreakpointWidget.cpp b/Source/Core/DolphinQt2/Debugger/BreakpointWidget.cpp
index 7f0e81bfc0..77420adacf 100644
--- a/Source/Core/DolphinQt2/Debugger/BreakpointWidget.cpp
+++ b/Source/Core/DolphinQt2/Debugger/BreakpointWidget.cpp
@@ -253,26 +253,25 @@ void BreakpointWidget::OnNewBreakpoint()
 void BreakpointWidget::OnLoad()
 {
   IniFile ini;
-  BreakPoints::TBreakPointsStr newbps;
-  MemChecks::TMemChecksStr newmcs;
-
   if (!ini.Load(File::GetUserPath(D_GAMESETTINGS_IDX) + SConfig::GetInstance().GetGameID() + ".ini",
                 false))
   {
     return;
   }
 
-  if (ini.GetLines("BreakPoints", &newbps, false))
+  BreakPoints::TBreakPointsStr new_bps;
+  if (ini.GetLines("BreakPoints", &new_bps, false))
   {
     PowerPC::breakpoints.Clear();
-    PowerPC::breakpoints.AddFromStrings(newbps);
+    PowerPC::breakpoints.AddFromStrings(new_bps);
   }
 
-  if (ini.GetLines("MemoryBreakPoints", &newmcs, false))
+  MemChecks::TMemChecksStr new_mcs;
+  if (ini.GetLines("MemoryBreakPoints", &new_mcs, false))
   {
     PowerPC::memchecks.Clear();
     Settings::Instance().blockSignals(true);
-    PowerPC::memchecks.AddFromStrings(newmcs);
+    PowerPC::memchecks.AddFromStrings(new_mcs);
     Settings::Instance().blockSignals(false);
   }