movenc: Add some comments explaining subtle details in writing the edit lists
authorMartin Storsjö <martin@martin.st>
Sat, 11 Oct 2014 20:56:24 +0000 (23:56 +0300)
committerMartin Storsjö <martin@martin.st>
Sun, 12 Oct 2014 14:46:01 +0000 (17:46 +0300)
Signed-off-by: Martin Storsjö <martin@martin.st>
libavformat/movenc.c

index 4017a56..218aa50 100644 (file)
@@ -1620,6 +1620,11 @@ static int mov_write_edts_tag(AVIOContext *pb, MOVTrack *track)
 
     avio_wb32(pb, entry_count);
     if (delay > 0) { /* add an empty edit to delay presentation */
+        /* In the positive delay case, the delay includes the cts
+         * offset, and the second edit list entry below trims out
+         * the same amount from the actual content. This makes sure
+         * that the offsetted last sample is included in the edit
+         * list duration as well. */
         if (version == 1) {
             avio_wb64(pb, delay);
             avio_wb64(pb, -1);
@@ -1634,6 +1639,9 @@ static int mov_write_edts_tag(AVIOContext *pb, MOVTrack *track)
          * here, but use FFMIN in case dts is a a small positive integer
          * rounded to 0 when represented in MOV_TIMESCALE units. */
         start_ct  = -FFMIN(track->cluster[0].dts, 0);
+        /* Note, this delay is calculated from the pts of the first sample,
+         * ensuring that we don't reduce the duration for cases with
+         * dts<0 pts=0. */
         duration += delay;
     }