Skip to content

Commit 36ce76d

Browse files
committed
If these two weird things were needed before, they don't seem to be needed now and just break things the way you'd expect them to break things.
1 parent 40b8ef7 commit 36ce76d

File tree

1 file changed

+2
-11
lines changed

1 file changed

+2
-11
lines changed

src/BizHawk.Client.Common/movie/tasproj/TasMovie.Editing.cs

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,7 @@ internal partial class TasMovie
1111

1212
public override void RecordFrame(int frame, IController source)
1313
{
14-
// RetroEdit: This check is questionable; recording at frame 0 is valid and should be reversible.
15-
// Also, frame - 1, why?
16-
// Is the precondition compensating for frame - 1 reindexing?
17-
if (frame != 0)
18-
{
19-
ChangeLog.AddGeneralUndo(frame - 1, frame - 1, $"Record Frame: {frame}");
20-
}
14+
ChangeLog.AddGeneralUndo(frame, frame, $"Record Frame: {frame}");
2115

2216
SetFrameAt(frame, Bk2LogEntryGenerator.GenerateLogEntry(source));
2317

@@ -31,10 +25,7 @@ public override void RecordFrame(int frame, IController source)
3125
InvalidateAfter(frame);
3226
}
3327

34-
if (frame != 0)
35-
{
36-
ChangeLog.SetGeneralRedo();
37-
}
28+
ChangeLog.SetGeneralRedo();
3829
}
3930

4031
public override void Truncate(int frame)

0 commit comments

Comments
 (0)