Shift DTS & CTS by minimum CTS to mimick ffprobe's behavior (#17)

* shift DTS & CTS by minimum CTS

* improve doc of Sample's DTS/CTS

Author: Wumpf

src/reader.rs

@@ -137,6 +137,17 @@ impl Mp4 {
             let mut ctts_run_index = -1i64;
             let mut dts_shift = 0;
 
+            // The smallest presentation timestamp observed in this stream.
+            //
+            // This is typically 0, but in the presence of sample reordering (caused by AVC/HVC b-frames), it may be non-zero.
+            // In fact, many formats don't require this to be zero, but video players typically
+            // normalize the shown time to start at zero.
+            // This is roughly equivalent to FFmpeg's internal `min_corrected_pts`
+            // https://github.com/FFmpeg/FFmpeg/blob/4047b887fc44b110bccb1da09bcb79d6e454b88b/libavformat/isom.h#L202
+            // To learn more about this I recommend reading the patch that introduced this in FFmpeg:
+            // https://patchwork.ffmpeg.org/project/ffmpeg/patch/[email protected]/#12592
+            let mut min_composition_timestamp = i64::MAX;
+
             let mut samples = Vec::<Sample>::new();
 
             fn get_sample_chunk_offset(stbl: &StblBox, chunk_index: u64) -> u64 {
@@ -231,6 +242,7 @@ impl Mp4 {
                 } else {
                     decode_timestamp
                 };
+                min_composition_timestamp = min_composition_timestamp.min(composition_timestamp);
 
                 let is_sync = if let Some(stss) = &stbl.stss {
                     if last_stss_index < stss.entries.len()
@@ -271,6 +283,15 @@ impl Mp4 {
                 }
             }
 
+            // Shift both DTS & CTS by the smallest CTS.
+            // For details, see declaration of `min_composition_timestamp` above.
+            if min_composition_timestamp != 0 {
+                for sample in &mut samples {
+                    sample.decode_timestamp -= min_composition_timestamp;
+                    sample.composition_timestamp -= min_composition_timestamp;
+                }
+            }
+
             tracks.insert(
                 trak.tkhd.track_id,
                 Track {
@@ -524,10 +545,16 @@ pub struct Sample {
 
     /// Timestamp of the sample at which it should be decoded,
     /// in time units.
+    ///
+    /// This is offsetted:
+    /// * with decode timestamp shift determined from negative sample offsets
+    /// * such that the first [`Self::composition_timestamp`] is zero.
     pub decode_timestamp: i64,
 
     /// Timestamp of the sample at which the sample should be displayed,
     /// in time units.
+    ///
+    /// This is offsetted such that the first composition timestamp is zero.
     pub composition_timestamp: i64,
 
     /// Duration of the sample in time units.