From 00f8129b4d29d8bf94610c4fc46b3faf66d98236 Mon Sep 17 00:00:00 2001
From: RetroEdit <30182911+RetroEdit@users.noreply.github.com>
Date: Wed, 26 Aug 2020 13:18:24 +0000
Subject: [PATCH] TAStudio: Fix deletion crashing when deleting invalid indexes
 (#2339)

* TAStudio: Fix deletion crashing when deleting invalid indexes

The problem occurs when trying to delete a selection where some of the frames were valid, but the last displayed line (which is currently an extra non-input frame) was in the selection.

* LINQ alignment tweak
---
 src/BizHawk.Client.Common/movie/tasproj/TasMovie.Editing.cs | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/BizHawk.Client.Common/movie/tasproj/TasMovie.Editing.cs b/src/BizHawk.Client.Common/movie/tasproj/TasMovie.Editing.cs
index d74b71751b..bec6f1d655 100644
--- a/src/BizHawk.Client.Common/movie/tasproj/TasMovie.Editing.cs
+++ b/src/BizHawk.Client.Common/movie/tasproj/TasMovie.Editing.cs
@@ -114,7 +114,10 @@ namespace BizHawk.Client.Common
 			{
 				// Separate the given frames into contiguous blocks
 				// and process each block independently
-				List<int> framesToDelete = frames.OrderBy(fr => fr).ToList();
+				List<int> framesToDelete = frames
+					.Where(fr => fr >= 0 && fr < InputLogLength)
+					.OrderBy(fr => fr)
+					.ToList();
 				// f is the current index for framesToDelete
 				int f = 0;
 				int numDeleted = 0;