Fill in guessed PTS values for decoders that don't output them at all (avi and probably other obscure CFR containers)

Author: myrsloik

meson.build

@@ -2,7 +2,7 @@ project('BestSource', 'cpp',
     default_options: ['buildtype=release', 'b_ndebug=if-release', 'cpp_std=c++17'],
     license: 'MIT',
     meson_version: '>=0.53.0',
-    version: '11.0',
+    version: '12.0',
 )
 
 api_sources = files(

src/version.h

@@ -21,7 +21,7 @@
 #ifndef VERSION_H
 #define VERSION_H
 
-#define BEST_SOURCE_VERSION_MAJOR 11
+#define BEST_SOURCE_VERSION_MAJOR 12
 #define BEST_SOURCE_VERSION_MINOR 0
 
 #endif

src/videosource.cpp

@@ -1052,12 +1052,14 @@ bool BestVideoSource::IndexTrack(const ProgressFunction &Progress) {
     TrackIndex.LastFrameDuration = 0;
     bool HasKeyFrames = false;
     bool HasEarlyKeyFrames = false;
+    bool HasValidPTS = false;
 
     while (true) {
         AVFrame *F = Decoder->GetNextFrame();
         if (!F)
             break;
 
+        HasValidPTS = HasValidPTS || (F->pts != AV_NOPTS_VALUE);
         HasKeyFrames = HasKeyFrames || !!(F->flags & AV_FRAME_FLAG_KEY);
         if (TrackIndex.Frames.size() < 100)
             HasEarlyKeyFrames = HasKeyFrames;
@@ -1082,6 +1084,22 @@ bool BestVideoSource::IndexTrack(const ProgressFunction &Progress) {
         } else if (!HasEarlyKeyFrames) {
             BSDebugPrint("No keyframes found in the first 100 frames when indexing, this may or may not cause performance problems when seeking");
         }
+
+        // Fill out default incrementing PTS for files with no valid values
+        if (!HasValidPTS) {
+            size_t Step = VP.Duration / TrackIndex.Frames.size();
+            if (Step < 1 || VP.Duration % TrackIndex.Frames.size() != 0) {
+                // Assertion here because it probably doesn't exist
+                assert(false);
+                Step = 1;
+            }
+
+            int64_t PTS = 0;
+            for (auto &Iter : TrackIndex.Frames) {
+                Iter.PTS = PTS;
+                PTS += Step;
+            }
+        }
     }
 
     return !TrackIndex.Frames.empty();