From a6f3391c8118ad5e7050eba367149fcae926c52e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= <sebastian@centricular.com>
Date: Fri, 12 Nov 2021 11:36:31 +0200
Subject: [PATCH] qtdemux: Add a comment why only positive cslg shifts are
 considered

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1294>
---
 subprojects/gst-plugins-good/gst/isomp4/qtdemux.h | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/subprojects/gst-plugins-good/gst/isomp4/qtdemux.h b/subprojects/gst-plugins-good/gst/isomp4/qtdemux.h
index e61deb6c21..074cd671c9 100644
--- a/subprojects/gst-plugins-good/gst/isomp4/qtdemux.h
+++ b/subprojects/gst-plugins-good/gst/isomp4/qtdemux.h
@@ -477,7 +477,15 @@ struct _QtDemuxStream
   guint32 ctts_count;
   gint32 ctts_soffset;
 
-  /* cslg */
+  /* cslg composition_to_dts_shift or based on the smallest negative
+   * composition time offset.
+   *
+   * This is unsigned because only negative composition time offsets /
+   * positive composition_to_dts_shift matter here. In all other cases,
+   * DTS/PTS can be inferred directly without ending up with PTS>DTS.
+   *
+   * See 14496-12 6.4
+   */
   guint64 cslg_shift;
 
   /* fragmented */