Fix FFmpegExaminer termination condition when there is no video;
authorCarl Hetherington <cth@carlh.net>
Sat, 1 Oct 2022 10:05:33 +0000 (12:05 +0200)
committerCarl Hetherington <cth@carlh.net>
Sat, 16 Dec 2023 01:07:37 +0000 (02:07 +0100)
it's hard to see how this ever worked.

src/lib/ffmpeg_examiner.cc

index d2d1093a9500abac1982adba50b1621ffb941a3b..59c74b9cc82dce5565fdbd233bfb2d8ddbec0b9d 100644 (file)
@@ -153,7 +153,7 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c, shared_ptr<Jo
 
                av_packet_free (&packet);
 
-               if (_first_video && got_all_audio && temporal_reference.size() >= (PULLDOWN_CHECK_FRAMES * 2)) {
+               if (got_all_audio && (!_video_stream || (_first_video && temporal_reference.size() >= (PULLDOWN_CHECK_FRAMES * 2)))) {
                        /* All done */
                        break;
                }