From c490d8baa2e19a5d270abae670611896b53baa33 Mon Sep 17 00:00:00 2001 From: YoshiRulz Date: Tue, 3 May 2022 18:15:45 +1000 Subject: [PATCH] Minor refactor to debug menu --- src/BizHawk.Client.EmuHawk/MainForm.Debug.cs | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/BizHawk.Client.EmuHawk/MainForm.Debug.cs b/src/BizHawk.Client.EmuHawk/MainForm.Debug.cs index cdcc04ebf7..3c9c158a1a 100644 --- a/src/BizHawk.Client.EmuHawk/MainForm.Debug.cs +++ b/src/BizHawk.Client.EmuHawk/MainForm.Debug.cs @@ -36,11 +36,11 @@ namespace BizHawk.Client.EmuHawk private sealed class DebugVSystemMenuItem : ToolStripMenuItemEx { - public readonly IReadOnlyCollection ExtraSysIDs; + public readonly IReadOnlyCollection SysIDs; - public DebugVSystemMenuItem(string labelText, IReadOnlyCollection? extraSysIDs = null) + public DebugVSystemMenuItem(string labelText, params string[] extraSysIDs) { - ExtraSysIDs = extraSysIDs ?? Array.Empty(); + SysIDs = new[] { labelText }.Concat(extraSysIDs).ToHashSet(); Text = labelText; } } @@ -187,7 +187,7 @@ namespace BizHawk.Client.EmuHawk Text = "Firmware", }, new ToolStripSeparatorEx(), - new DebugVSystemMenuItem("GB") + new DebugVSystemMenuItem(VSystemID.Raw.GB, VSystemID.Raw.GBC) { DropDownItems = { @@ -199,7 +199,7 @@ namespace BizHawk.Client.EmuHawk }, }, }, - new DebugVSystemMenuItem("GBA") + new DebugVSystemMenuItem(VSystemID.Raw.GBA) { DropDownItems = { @@ -211,7 +211,7 @@ namespace BizHawk.Client.EmuHawk }, }, }, - new DebugVSystemMenuItem("N64") + new DebugVSystemMenuItem(VSystemID.Raw.N64) { DropDownItems = { @@ -228,7 +228,7 @@ namespace BizHawk.Client.EmuHawk var coreName = Emulator.Attributes().CoreName; foreach (var item in ((ToolStripMenuItemEx) ddoSender).DropDownItems.OfType()) { - var groupEnabled = item.Text == sysID || item.ExtraSysIDs.Contains(sysID); + var groupEnabled = item.SysIDs.Contains(sysID); foreach (var child in item.DropDownItems.Cast().Where(static child => child.RequiresLoadedRom)) // RequiresLoadedRom == false => leave Enabled as default true { child.Enabled = groupEnabled && (child.RequiresCore is null || child.RequiresCore == coreName);