h265parse: Don't enable passthrough by default
authorSeungha Yang <seungha@centricular.com>
Sat, 3 Oct 2020 17:02:16 +0000 (02:02 +0900)
committerSeungha Yang <seungha@centricular.com>
Wed, 14 Oct 2020 18:25:17 +0000 (03:25 +0900)
SEI messages contain various information which wouldn't be conveyed
by using upstream CAPS (HDR, timecode for example).

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/1639>

gst/videoparsers/gsth265parse.c

index 09706a1..e94f86d 100644 (file)
@@ -2978,7 +2978,13 @@ gst_h265_parse_set_caps (GstBaseParse * parse, GstCaps * caps)
   if (format == h265parse->format && align == h265parse->align) {
     /* do not set CAPS and passthrough mode if SPS/PPS have not been parsed */
     if (h265parse->have_sps && h265parse->have_pps) {
+      /* Don't enable passthrough here. This element will parse various
+       * SEI messages which would be very important/useful for downstream
+       * (HDR, timecode for example)
+       */
+#if 0
       gst_base_parse_set_passthrough (parse, TRUE);
+#endif
 
       /* we did parse codec-data and might supplement src caps */
       gst_h265_parse_update_src_caps (h265parse, caps);