From 4ed7847fa97eff2626105859d4e68404a4d70894 Mon Sep 17 00:00:00 2001
From: Jonathan Thomas <jonathan@openshot.org>
Date: Fri, 14 Dec 2018 14:38:33 -0600
Subject: [PATCH] Improving cache performance by preventing the cache from
 getting behind the currently displaying frame #

---
 src/Qt/PlayerPrivate.cpp    | 2 +-
 src/Qt/VideoCacheThread.cpp | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/Qt/PlayerPrivate.cpp b/src/Qt/PlayerPrivate.cpp
index 63d20e99..1839f1eb 100644
--- a/src/Qt/PlayerPrivate.cpp
+++ b/src/Qt/PlayerPrivate.cpp
@@ -149,7 +149,7 @@ namespace openshot
 		else
 		{
 			// Update cache on which frame was retrieved
-			videoCache->current_display_frame = video_position;
+			videoCache->setCurrentFramePosition(video_position);
 
 			// return frame from reader
 			return reader->GetFrame(video_position);
diff --git a/src/Qt/VideoCacheThread.cpp b/src/Qt/VideoCacheThread.cpp
index ed224de5..208fcaab 100644
--- a/src/Qt/VideoCacheThread.cpp
+++ b/src/Qt/VideoCacheThread.cpp
@@ -102,6 +102,12 @@ namespace openshot
 				// Ignore out of bounds frame exceptions
 			}
 
+			// Is cache position behind current display frame?
+			if (position < current_display_frame) {
+				// Jump ahead
+				position = current_display_frame;
+			}
+
 	    	// Increment frame number
 	    	position++;
 	    }