diff --git a/src/BizHawk.Client.EmuHawk/movie/EditCommentsForm.cs b/src/BizHawk.Client.EmuHawk/movie/EditCommentsForm.cs index aa2dba3ca8..246034a471 100644 --- a/src/BizHawk.Client.EmuHawk/movie/EditCommentsForm.cs +++ b/src/BizHawk.Client.EmuHawk/movie/EditCommentsForm.cs @@ -27,14 +27,11 @@ namespace BizHawk.Client.EmuHawk private void EditCommentsForm_Load(object sender, EventArgs e) { - if (_movie.Comments.Any()) + for (int i = 0; i < _movie.Comments.Count; i++) { - for (int i = 0; i < _movie.Comments.Count; i++) - { - CommentGrid.Rows.Add(); - var c = CommentGrid.Rows[i].Cells[0]; - c.Value = _movie.Comments[i]; - } + CommentGrid.Rows.Add(); + var c = CommentGrid.Rows[i].Cells[0]; + c.Value = _movie.Comments[i]; } if (_readOnly) diff --git a/src/BizHawk.Client.EmuHawk/tools/Debugger/BreakpointControl.cs b/src/BizHawk.Client.EmuHawk/tools/Debugger/BreakpointControl.cs index 9cc967fa5b..5c095e4c04 100644 --- a/src/BizHawk.Client.EmuHawk/tools/Debugger/BreakpointControl.cs +++ b/src/BizHawk.Client.EmuHawk/tools/Debugger/BreakpointControl.cs @@ -182,7 +182,6 @@ namespace BizHawk.Client.EmuHawk private void RemoveBreakpointButton_Click(object sender, EventArgs e) { - if (!EditableItems.Any()) return; var items = EditableItems.ToList(); if (!items.Any()) return; foreach (var item in items) _breakpoints.Remove(item); @@ -205,7 +204,6 @@ namespace BizHawk.Client.EmuHawk private void BreakpointView_ItemActivate(object sender, EventArgs e) { - if (!EditableItems.Any()) return; var items = EditableItems.ToList(); if (!items.Any()) return; foreach (var item in items) item.Active = !item.Active; diff --git a/src/BizHawk.Client.EmuHawk/tools/TAStudio/TAStudio.ListView.cs b/src/BizHawk.Client.EmuHawk/tools/TAStudio/TAStudio.ListView.cs index 2276ca5d42..02fbd23fb9 100644 --- a/src/BizHawk.Client.EmuHawk/tools/TAStudio/TAStudio.ListView.cs +++ b/src/BizHawk.Client.EmuHawk/tools/TAStudio/TAStudio.ListView.cs @@ -1390,12 +1390,9 @@ namespace BizHawk.Client.EmuHawk } } - if (_extraAxisRows.Any()) + foreach (int row in _extraAxisRows) { - foreach (int row in _extraAxisRows) - { - CurrentTasMovie.SetAxisState(row, _axisEditColumn, value); - } + CurrentTasMovie.SetAxisState(row, _axisEditColumn, value); } if (value != prev) // Auto-restore diff --git a/src/BizHawk.Emulation.Cores/Arcades/MAME/MAME.ISettable.cs b/src/BizHawk.Emulation.Cores/Arcades/MAME/MAME.ISettable.cs index a2b5e39ce2..a310cb0cc7 100644 --- a/src/BizHawk.Emulation.Cores/Arcades/MAME/MAME.ISettable.cs +++ b/src/BizHawk.Emulation.Cores/Arcades/MAME/MAME.ISettable.cs @@ -94,17 +94,13 @@ namespace BizHawk.Emulation.Cores.Arcades.MAME var DIPSwitchOptions = MameGetString(MAMELuaCommand.GetDIPSwitchOptions(tag, fieldName)); var options = DIPSwitchOptions.Split(new[] { '@' }, StringSplitOptions.RemoveEmptyEntries); - + if (options.Length is 0) continue; foreach (var option in options) { var opt = option.Split(new[] { '~' }, StringSplitOptions.RemoveEmptyEntries); setting.Options.Add(opt[0], opt[1]); } - - if (options.Any()) - { - CurrentDriverSettings.Add(setting); - } + CurrentDriverSettings.Add(setting); } } }