Core Features dialog - properly determine if a core is ISettable, don't add properties and fields if the interface isn't implemented anyway

BizHawk.Client.EmuHawk/CoreFeatureAnalysis.cs

@@ -50,25 +50,41 @@ namespace BizHawk.Client.EmuHawk
 			ReleasedCoresLabel.Text = cores.Count(c => c.CoreAttributes.Released).ToString();
 
 			CoreTree.Nodes.Clear();
+			CoreTree.BeginUpdate();
 
 			foreach (var core in cores)
 			{
 				var coreNode = new TreeNode
 				{
 					Text = core.CoreAttributes.CoreName + (core.CoreAttributes.Released ? string.Empty : " (UNRELEASED)"),
-					ForeColor = core.CoreAttributes.Released ? Color.Black : Color.DarkGray,
+					ForeColor = core.CoreAttributes.Released ? Color.Black : Color.DarkGray
-					// TODO
-					//ForeColor = services.All(s => s.IsAssignableFrom(core.CoreType)) ? Color.Black : Color.Red
 				};
 
 				foreach (var service in services)
 				{
+					bool isImplemented = false;
+					if (service.IsAssignableFrom(core.CoreType))
+					{
+						isImplemented = true;
+					}
+					else if (service.IsAssignableFrom(typeof(ISettable<,>)))
+					{
+						isImplemented = core.CoreType
+							.GetInterfaces()
+							.Where(t => t.IsGenericType &&
+										t.GetGenericTypeDefinition() == typeof(ISettable<,>))
+							.FirstOrDefault() != null;
+					}
+
+
 					var serviceNode = new TreeNode
 					{
 						Text = service.Name,
-						ForeColor = (service.IsAssignableFrom(core.CoreType)) ? Color.Black : Color.Red
+						ForeColor = isImplemented ? Color.Black : Color.Red
 					};
 
+					if (isImplemented)
+					{
 						foreach (var field in service.GetProperties())
 						{
 							serviceNode.Nodes.Add(new TreeNode
@@ -84,6 +100,7 @@ namespace BizHawk.Client.EmuHawk
 								Text = field.Name
 							});
 						}
+					}
 
 					coreNode.Nodes.Add(serviceNode);
 				}
@@ -91,6 +108,8 @@ namespace BizHawk.Client.EmuHawk
 
 				CoreTree.Nodes.Add(coreNode);
 			}
+
+			CoreTree.EndUpdate();
 		}
 	}
 }