[Bf-blender-cvs] [1949643ee58] master: Fix: Wrong logic for checking if we can reuse decoded frame

[Sebastian Parborg]

Commit: 1949643ee58c533036395ab2ad4299489460e44b
Author: Sebastian Parborg
Date:   Mon Jun 7 18:12:57 2021 +0200
Branches: master
https://developer.blender.org/rB1949643ee58c533036395ab2ad4299489460e44b

Fix: Wrong logic for checking if we can reuse decoded frame

We should only check if the new pts value lies inside the duration of
the current frame.

===================================================================

M	source/blender/imbuf/intern/anim_movie.c

===================================================================

diff --git a/source/blender/imbuf/intern/anim_movie.c b/source/blender/imbuf/intern/anim_movie.c
index 622b6cbfc16..b65c3e364db 100644
--- a/source/blender/imbuf/intern/anim_movie.c
+++ b/source/blender/imbuf/intern/anim_movie.c
@@ -1076,7 +1076,8 @@ static int64_t ffmpeg_get_pts_to_search(struct anim *anim,
 static bool ffmpeg_pts_matches_last_frame(struct anim *anim, int64_t pts_to_search)
 {
   if (anim->pFrame && anim->cur_frame_final) {
-    return labs(anim->cur_pts - pts_to_search) < anim->pFrame->pkt_duration;
+    int64_t diff = pts_to_search - anim->cur_pts;
+    return diff >= 0 && diff < anim->pFrame->pkt_duration;
   }
 
   return false;