Split FirmwaresMenuItem_Click

Author: YoshiRulz

src/BizHawk.Client.EmuHawk/MainForm.Events.cs

@@ -880,25 +880,22 @@ private void HotkeysMenuItem_Click(object sender, EventArgs e)
 			InputManager.SyncControls(Emulator, MovieSession, Config);
 		}
 
+		private void OpenFWConfigRomLoadFailed(RomLoader.RomErrorArgs args)
+		{
+			using FirmwaresConfig configForm = new(
+				this,
+				FirmwareManager,
+				Config.FirmwareUserSpecifications,
+				Config.PathEntries,
+				retryLoadRom: true,
+				reloadRomPath: args.RomPath);
+			args.Retry = this.ShowDialogWithTempMute(configForm) is DialogResult.Retry;
+		}
+
 		private void FirmwaresMenuItem_Click(object sender, EventArgs e)
 		{
-			if (e is RomLoader.RomErrorArgs args)
-			{
-				using var configForm = new FirmwaresConfig(
-					this,
-					FirmwareManager,
-					Config.FirmwareUserSpecifications,
-					Config.PathEntries,
-					retryLoadRom: true,
-					reloadRomPath: args.RomPath);
-				var result = this.ShowDialogWithTempMute(configForm);
-				args.Retry = result == DialogResult.Retry;
-			}
-			else
-			{
-				using var configForm = new FirmwaresConfig(this, FirmwareManager, Config.FirmwareUserSpecifications, Config.PathEntries);
-				this.ShowDialogWithTempMute(configForm);
-			}
+			using var configForm = new FirmwaresConfig(this, FirmwareManager, Config.FirmwareUserSpecifications, Config.PathEntries);
+			this.ShowDialogWithTempMute(configForm);
 		}
 
 		private void MessagesMenuItem_Click(object sender, EventArgs e)

src/BizHawk.Client.EmuHawk/MainForm.cs

@@ -3694,7 +3694,7 @@ private void ShowLoadError(object sender, RomLoader.RomErrorArgs e)
 					text: "The core needs certain firmware to load this rom.\n\nOpen the firmware manager now?",
 					useOKCancel: true))
 				{
-					FirmwaresMenuItem_Click(null, e);
+					OpenFWConfigRomLoadFailed(e);
 					if (e.Retry)
 					{
 						// Retry loading the ROM here. This leads to recursion, as the original call to LoadRom has not exited yet,