From 2d6bac8798850a6e7b84e4d402eaeb4a390f8ddc Mon Sep 17 00:00:00 2001
From: YoshiRulz <OSSYoshiRulz@gmail.com>
Date: Sat, 25 Jul 2020 08:44:50 +1000
Subject: [PATCH] Tiny ToolManager.Load cleanup

---
 .../tools/ToolManager.cs                      | 21 ++++---------------
 1 file changed, 4 insertions(+), 17 deletions(-)

diff --git a/src/BizHawk.Client.EmuHawk/tools/ToolManager.cs b/src/BizHawk.Client.EmuHawk/tools/ToolManager.cs
index 9e0529e3e6..6789a5f3d3 100644
--- a/src/BizHawk.Client.EmuHawk/tools/ToolManager.cs
+++ b/src/BizHawk.Client.EmuHawk/tools/ToolManager.cs
@@ -87,23 +87,11 @@ namespace BizHawk.Client.EmuHawk
 		/// <summary>
 		/// Loads the tool dialog T (T must implement <see cref="IToolForm"/>) , if it does not exist it will be created, if it is already open, it will be focused
 		/// </summary>
-		/// <typeparam name="T">Type of tool you want to load</typeparam>
 		/// <param name="focus">Define if the tool form has to get the focus or not (Default is true)</param>
-		/// <returns>An instantiated <see cref="IToolForm"/></returns>
-		public T Load<T>(bool focus = true)
-			where T : class, IToolForm
-		{
-			return Load<T>("", focus);
-		}
-
-		/// <summary>
-		/// Loads the tool dialog T (T must implement <see cref="IToolForm"/>) , if it does not exist it will be created, if it is already open, it will be focused
-		/// </summary>
-		/// <typeparam name="T">Type of tool you want to load</typeparam>
 		/// <param name="toolPath">Path to the .dll of the external tool</param>
-		/// <param name="focus">Define if the tool form has to get the focus or not (Default is true)</param>
+		/// <typeparam name="T">Type of tool you want to load</typeparam>
 		/// <returns>An instantiated <see cref="IToolForm"/></returns>
-		public T Load<T>(string toolPath, bool focus = true)
+		public T Load<T>(bool focus = true, string toolPath = "")
 			where T : class, IToolForm
 		{
 			if (!IsAvailable<T>()) return null;
@@ -123,8 +111,7 @@ namespace BizHawk.Client.EmuHawk
 				_tools.Remove(existingTool);
 			}
 
-			var newTool = CreateInstance<T>(toolPath);
-			if (newTool == null) return null;
+			if (!(CreateInstance<T>(toolPath) is T newTool)) return null;
 
 			if (newTool is Form form) form.Owner = _owner;
 			ServiceInjector.UpdateServices(_emulator.ServiceProvider, newTool);
@@ -153,7 +140,7 @@ namespace BizHawk.Client.EmuHawk
 
 			newTool.Restart();
 			newTool.Show();
-			return (T)newTool;
+			return newTool;
 		}
 
 		/// <summary>Loads the external tool's entry form.</summary>