ShuriZma
/
BizHawk
mirror of https://github.com/TASEmulators/BizHawk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Tiny ToolManager.Load cleanup

This commit is contained in:
YoshiRulz 2020-07-25 08:44:50 +10:00
parent 0fc3ac2105
commit 2d6bac8798
No known key found for this signature in database
GPG Key ID: C4DE31C245353FB7
1 changed files with 4 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/BizHawk.Client.EmuHawk/tools/ToolManager.cs
View File

@ -87,23 +87,11 @@ namespace BizHawk.Client.EmuHawk
/// <summary> /// <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 /// 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> /// </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> /// <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="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> /// <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 where T : class, IToolForm
{ {
if (!IsAvailable<T>()) return null; if (!IsAvailable<T>()) return null;
@ -123,8 +111,7 @@ namespace BizHawk.Client.EmuHawk
_tools.Remove(existingTool); _tools.Remove(existingTool);
} }
var newTool = CreateInstance<T>(toolPath); if (!(CreateInstance<T>(toolPath) is T newTool)) return null;
if (newTool == null) return null;
if (newTool is Form form) form.Owner = _owner; if (newTool is Form form) form.Owner = _owner;
ServiceInjector.UpdateServices(_emulator.ServiceProvider, newTool); ServiceInjector.UpdateServices(_emulator.ServiceProvider, newTool);
@ -153,7 +140,7 @@ namespace BizHawk.Client.EmuHawk
newTool.Restart(); newTool.Restart();
newTool.Show(); newTool.Show();
return (T)newTool; return newTool;
} }
/// <summary>Loads the external tool's entry form.</summary> /// <summary>Loads the external tool's entry form.</summary>