From 419ed1b46a2a86e665f5885fd8fbd4609c39cf4a Mon Sep 17 00:00:00 2001
From: Lioncash <mathew1800@gmail.com>
Date: Mon, 9 Apr 2018 03:18:51 -0400
Subject: [PATCH 1/2] DVDInterface: Deduplicate code in UpdateInterrupts()

---
 Source/Core/Core/HW/DVD/DVDInterface.cpp | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/Source/Core/Core/HW/DVD/DVDInterface.cpp b/Source/Core/Core/HW/DVD/DVDInterface.cpp
index b7d897fdbc..838148174b 100644
--- a/Source/Core/Core/HW/DVD/DVDInterface.cpp
+++ b/Source/Core/Core/HW/DVD/DVDInterface.cpp
@@ -586,15 +586,11 @@ void RegisterMMIO(MMIO::Mapping* mmio, u32 base)
 
 void UpdateInterrupts()
 {
-  if ((s_DISR.DEINT & s_DISR.DEINITMASK) || (s_DISR.TCINT & s_DISR.TCINTMASK) ||
-      (s_DISR.BRKINT & s_DISR.BRKINTMASK) || (s_DICVR.CVRINT & s_DICVR.CVRINTMASK))
-  {
-    ProcessorInterface::SetInterrupt(ProcessorInterface::INT_CAUSE_DI, true);
-  }
-  else
-  {
-    ProcessorInterface::SetInterrupt(ProcessorInterface::INT_CAUSE_DI, false);
-  }
+  const bool set_mask = (s_DISR.DEINT & s_DISR.DEINITMASK) || (s_DISR.TCINT & s_DISR.TCINTMASK) ||
+                        (s_DISR.BRKINT & s_DISR.BRKINTMASK) ||
+                        (s_DICVR.CVRINT & s_DICVR.CVRINTMASK);
+
+  ProcessorInterface::SetInterrupt(ProcessorInterface::INT_CAUSE_DI, set_mask);
 
   // Required for Summoner: A Goddess Reborn
   CoreTiming::ForceExceptionCheck(50);

From a8088b7365c082c1f1701ec9dd8ef45656c36a32 Mon Sep 17 00:00:00 2001
From: Lioncash <mathew1800@gmail.com>
Date: Mon, 9 Apr 2018 03:21:57 -0400
Subject: [PATCH 2/2] ProcessorInterface: Remove prefixed underscores from
 parameters

These are reserved by the implementation for any use.
---
 Source/Core/Core/HW/ProcessorInterface.cpp | 24 +++++++++++-----------
 Source/Core/Core/HW/ProcessorInterface.h   |  2 +-
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/Source/Core/Core/HW/ProcessorInterface.cpp b/Source/Core/Core/HW/ProcessorInterface.cpp
index 42e2924157..041ff50ee8 100644
--- a/Source/Core/Core/HW/ProcessorInterface.cpp
+++ b/Source/Core/Core/HW/ProcessorInterface.cpp
@@ -135,9 +135,9 @@ void UpdateException()
     PowerPC::ppcState.Exceptions &= ~EXCEPTION_EXTERNAL_INT;
 }
 
-static const char* Debug_GetInterruptName(u32 _causemask)
+static const char* Debug_GetInterruptName(u32 cause_mask)
 {
-  switch (_causemask)
+  switch (cause_mask)
   {
   case INT_CAUSE_PI:
     return "INT_CAUSE_PI";
@@ -176,33 +176,33 @@ static const char* Debug_GetInterruptName(u32 _causemask)
   }
 }
 
-void SetInterrupt(u32 _causemask, bool _bSet)
+void SetInterrupt(u32 cause_mask, bool set)
 {
   DEBUG_ASSERT_MSG(POWERPC, Core::IsCPUThread(), "SetInterrupt from wrong thread");
 
-  if (_bSet && !(m_InterruptCause & _causemask))
+  if (set && !(m_InterruptCause & cause_mask))
   {
-    DEBUG_LOG(PROCESSORINTERFACE, "Setting Interrupt %s (set)", Debug_GetInterruptName(_causemask));
+    DEBUG_LOG(PROCESSORINTERFACE, "Setting Interrupt %s (set)", Debug_GetInterruptName(cause_mask));
   }
 
-  if (!_bSet && (m_InterruptCause & _causemask))
+  if (!set && (m_InterruptCause & cause_mask))
   {
     DEBUG_LOG(PROCESSORINTERFACE, "Setting Interrupt %s (clear)",
-              Debug_GetInterruptName(_causemask));
+              Debug_GetInterruptName(cause_mask));
   }
 
-  if (_bSet)
-    m_InterruptCause |= _causemask;
+  if (set)
+    m_InterruptCause |= cause_mask;
   else
-    m_InterruptCause &= ~_causemask;  // is there any reason to have this possibility?
+    m_InterruptCause &= ~cause_mask;  // is there any reason to have this possibility?
   // F|RES: i think the hw devices reset the interrupt in the PI to 0
   // if the interrupt cause is eliminated. that isn't done by software (afaik)
   UpdateException();
 }
 
-static void SetResetButton(bool _bSet)
+static void SetResetButton(bool set)
 {
-  SetInterrupt(INT_CAUSE_RST_BUTTON, !_bSet);
+  SetInterrupt(INT_CAUSE_RST_BUTTON, !set);
 }
 
 static void ToggleResetButtonCallback(u64 userdata, s64 cyclesLate)
diff --git a/Source/Core/Core/HW/ProcessorInterface.h b/Source/Core/Core/HW/ProcessorInterface.h
index d487504719..d62c0ea1ee 100644
--- a/Source/Core/Core/HW/ProcessorInterface.h
+++ b/Source/Core/Core/HW/ProcessorInterface.h
@@ -73,7 +73,7 @@ inline u32 GetCause()
   return m_InterruptCause;
 }
 
-void SetInterrupt(u32 _causemask, bool _bSet = true);
+void SetInterrupt(u32 cause_mask, bool set = true);
 
 // Thread-safe func which sets and clears reset button state automagically
 void ResetButton_Tap();