put a virtual AskSaveChanges method in ToolFormBase, and only override it when needed, reduces a lot of boilerplate

This commit is contained in:
adelikat 2020-05-03 09:52:36 -05:00
parent 664fb5cb31
commit da5ddf3217
30 changed files with 7 additions and 45 deletions

View File

@ -42,7 +42,6 @@ namespace BizHawk.Client.EmuHawk
public void Restart() { }
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
private void Attach()

View File

@ -296,8 +296,6 @@ namespace BizHawk.Client.EmuHawk
}
}
public bool AskSaveChanges() => true;
#endregion
#region Control Events

View File

@ -194,7 +194,7 @@ namespace BizHawk.Client.EmuHawk
lvCDL.RowCount = _cdl.Count;
}
public bool AskSaveChanges()
public override bool AskSaveChanges()
{
// nothing to fear:
if (_cdl == null)

View File

@ -110,11 +110,6 @@ namespace BizHawk.Client.EmuHawk
: Path.GetFileName(Global.CheatList.CurrentFileName);
}
public bool AskSaveChanges()
{
return true;
}
private void LoadFile(FileSystemInfo file, bool append)
{
if (file != null)

View File

@ -156,8 +156,6 @@ namespace BizHawk.Client.EmuHawk
EngageDebugger();
}
public bool AskSaveChanges() => true;
public bool UpdateBefore => false;
}
}

View File

@ -54,7 +54,6 @@ namespace BizHawk.Client.EmuHawk
}
}
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
public GbGpuView()

View File

@ -54,8 +54,6 @@ namespace BizHawk.Client.EmuHawk
public bool UpdateBefore => false;
public bool AskSaveChanges() => true;
public void FastUpdate()
{
}

View File

@ -29,7 +29,6 @@ namespace BizHawk.Client.EmuHawk
// MobileDetailView memory;
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
public GbaGpuView()

View File

@ -30,7 +30,6 @@ namespace BizHawk.Client.EmuHawk
#region IToolForm
public bool UpdateBefore => true;
public bool AskSaveChanges() => true;
public void FastUpdate()
{

View File

@ -150,8 +150,6 @@ namespace BizHawk.Client.EmuHawk
UpdateValues();
}
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
private void bmpViewPal_MouseClick(object sender, MouseEventArgs e)

View File

@ -156,8 +156,6 @@ namespace BizHawk.Client.EmuHawk
public bool UpdateBefore => false;
public bool AskSaveChanges() => true;
public void NewUpdate(ToolFormUpdateType type) { }
public void UpdateValues()

View File

@ -644,7 +644,7 @@ namespace BizHawk.Client.EmuHawk
AddFileWatches();
}
public bool AskSaveChanges()
public override bool AskSaveChanges()
{
if (LuaImp.ScriptList.Changes && !string.IsNullOrEmpty(LuaImp.ScriptList.Filename))
{

View File

@ -102,7 +102,7 @@ namespace BizHawk.Client.EmuHawk
public bool UpdateBefore => true;
public bool AskSaveChanges()
public override bool AskSaveChanges()
{
if (_unsavedZones.Count == 0 || IsDisposed)
{

View File

@ -77,8 +77,6 @@ namespace BizHawk.Client.EmuHawk
{
}
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
#endregion

View File

@ -26,7 +26,6 @@ namespace BizHawk.Client.EmuHawk
SyncContents();
}
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
public void Restart()

View File

@ -39,7 +39,6 @@ namespace BizHawk.Client.EmuHawk
#region Public API
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
public void Restart()

View File

@ -59,7 +59,6 @@ namespace BizHawk.Client.EmuHawk
#region Public API
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
public void NewUpdate(ToolFormUpdateType type) { }

View File

@ -32,7 +32,6 @@ namespace BizHawk.Client.EmuHawk
#region Public API
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
public unsafe void Generate()

View File

@ -170,11 +170,8 @@ namespace BizHawk.Client.EmuHawk
{
}
public bool AskSaveChanges() => true;
public bool UpdateBefore => false;
// 32*16 samples, 16bit, mono, 8khz (but we'll change the sample rate)
private static readonly byte[] EmptyWav = {
0x52, 0x49, 0x46, 0x46, 0x24, 0x04, 0x00, 0x00, 0x57, 0x41, 0x56, 0x45, 0x66, 0x6D, 0x74, 0x20,

View File

@ -164,8 +164,6 @@ namespace BizHawk.Client.EmuHawk
CheckBoxVDC2_CheckedChanged(null, null);
}
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
#endregion

View File

@ -160,8 +160,6 @@ namespace BizHawk.Client.EmuHawk
UpdateValues();
}
public bool AskSaveChanges() => true;
public bool UpdateBefore => true;
private void bmpViewPalette_MouseClick(object sender, MouseEventArgs e)

View File

@ -41,7 +41,6 @@ namespace BizHawk.Client.EmuHawk
readonly List<DisplayTypeItem> displayTypeItems = new List<DisplayTypeItem>();
public bool UpdateBefore => false;
public bool AskSaveChanges() => true;
[RequiredService]
private LibsnesCore Emulator { get; set; }

View File

@ -96,7 +96,7 @@ namespace BizHawk.Client.EmuHawk
}
}
public bool AskSaveChanges()
public override bool AskSaveChanges()
{
if (_suppressAskSave)
{

View File

@ -35,7 +35,6 @@ namespace BizHawk.Client.EmuHawk
#region Public API
public bool AskSaveChanges() => true;
public bool UpdateBefore => false;
public void NewUpdate(ToolFormUpdateType type) { }

View File

@ -30,7 +30,6 @@ namespace BizHawk.Client.EmuHawk
public void NewUpdate(ToolFormUpdateType type) { }
public bool AskSaveChanges() => true;
public bool UpdateBefore => false;
public void UpdateValues() { }

View File

@ -16,6 +16,8 @@ namespace BizHawk.Client.EmuHawk
public IMovieSession MovieSession { get; set; }
public virtual bool AskSaveChanges() => true;
public static FileInfo OpenFileDialog(string currentFile, string path, string fileType, string fileExt)
{
if (!Directory.Exists(path))

View File

@ -95,8 +95,6 @@ namespace BizHawk.Client.EmuHawk
public bool UpdateBefore => false;
public bool AskSaveChanges() => true;
private void SaveConfigSettings()
{
//Tracer.Enabled = LoggingEnabled.Checked;

View File

@ -129,7 +129,6 @@ namespace BizHawk.Client.EmuHawk
#region IToolForm Implementation
public bool AskSaveChanges() => true;
public bool UpdateBefore => false;
public void Restart()

View File

@ -71,8 +71,6 @@ namespace BizHawk.Client.EmuHawk
[ConfigPersist]
public RamSearchSettings Settings { get; set; }
public bool AskSaveChanges() => true;
public bool UpdateBefore => false;
private void HardSetDisplayTypeDropDown(Common.DisplayType type)

View File

@ -112,7 +112,7 @@ namespace BizHawk.Client.EmuHawk
Changes();
}
public bool AskSaveChanges()
public override bool AskSaveChanges()
{
if (_watches.Changes)
{