From e72ce6d9cb4f7b179a99e5a954df1dc4d62061f5 Mon Sep 17 00:00:00 2001
From: adelikat <adelikat@tasvideos.org>
Date: Fri, 11 Nov 2016 15:33:29 -0600
Subject: [PATCH] Atari 2600 - show the Needs Reboot icon when changing sync
 settings

---
 .../Consoles/Atari/2600/Atari2600.ISettable.cs            | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/BizHawk.Emulation.Cores/Consoles/Atari/2600/Atari2600.ISettable.cs b/BizHawk.Emulation.Cores/Consoles/Atari/2600/Atari2600.ISettable.cs
index 2df755e350..5e0860fdb6 100644
--- a/BizHawk.Emulation.Cores/Consoles/Atari/2600/Atari2600.ISettable.cs
+++ b/BizHawk.Emulation.Cores/Consoles/Atari/2600/Atari2600.ISettable.cs
@@ -37,8 +37,9 @@ namespace BizHawk.Emulation.Cores.Atari.Atari2600
 
 		public bool PutSyncSettings(A2600SyncSettings o)
 		{
+			bool ret = A2600SyncSettings.NeedsReboot(SyncSettings, o);
 			SyncSettings = o;
-			return false;
+			return ret;
 		}
 
 		internal A2600Settings Settings { get; private set; }
@@ -180,6 +181,11 @@ namespace BizHawk.Emulation.Cores.Atari.Atari2600
 			{
 				SettingsUtil.SetDefaultValues(this);
 			}
+
+			public static bool NeedsReboot(A2600SyncSettings x, A2600SyncSettings y)
+			{
+				return !DeepEquality.DeepEquals(x, y);
+			}
 		}
 	}
 }