From 01eb878a3e28d9cb63029cc3e22fa91245e7f45a Mon Sep 17 00:00:00 2001
From: Nico de Poel <ndepoel@gmail.com>
Date: Sat, 27 Jul 2024 13:23:45 +0200
Subject: [PATCH] Removed pointless dispatch count, when it's only used to
 check for first execution

---
 Runtime/FrameInterpolation/FrameInterpolationContext.cs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/Runtime/FrameInterpolation/FrameInterpolationContext.cs b/Runtime/FrameInterpolation/FrameInterpolationContext.cs
index 876d108..4c46235 100644
--- a/Runtime/FrameInterpolation/FrameInterpolationContext.cs
+++ b/Runtime/FrameInterpolation/FrameInterpolationContext.cs
@@ -39,7 +39,6 @@ namespace FidelityFX.FrameGen
         private bool _firstExecution;
         private bool _asyncSupported;
         private ulong _previousFrameID;
-        private ulong _dispatchCount;
         
         public void Create(in FrameInterpolation.ContextDescription contextDescription)
         {
@@ -49,7 +48,6 @@ namespace FidelityFX.FrameGen
             _spdConstantsBuffer = CreateConstantBuffer<FrameInterpolation.InpaintingPyramidConstants>();
 
             _firstExecution = true;
-            _dispatchCount = 0;
             _previousFrameID = 0;
             _asyncSupported = (_contextDescription.flags & FrameInterpolation.InitializationFlags.EnableAsyncSupport) == FrameInterpolation.InitializationFlags.EnableAsyncSupport;
 
@@ -134,7 +132,8 @@ namespace FidelityFX.FrameGen
         {
             commandBuffer.BeginSample(_sampler);
 
-            bool reset = _dispatchCount == 0 || dispatchDescription.reset;
+            bool reset = _firstExecution || dispatchDescription.reset;
+            _firstExecution = false;
             Assert.IsTrue(!_asyncSupported || reset || dispatchDescription.frameID > _previousFrameID,
                 "When async support is enabled, and the reset flag is not set, frame ID must increment in each dispatch.");
 
@@ -142,7 +141,6 @@ namespace FidelityFX.FrameGen
             bool frameIdSkipped = (dispatchDescription.frameID - _previousFrameID) > 1;
             bool disjointFrameId = frameIdDecreased || frameIdSkipped;
             _previousFrameID = dispatchDescription.frameID;
-            _dispatchCount++;   // TODO: this is pointless, it does the same as _firstExecution, no need to do any counting
 
             Constants.renderSize = dispatchDescription.renderSize;
             Constants.displaySize = dispatchDescription.displaySize;