cccombiner: implement scheduling
authorMathieu Duponchelle <mathieu@centricular.com>
Tue, 9 Mar 2021 12:22:10 +0000 (13:22 +0100)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Wed, 17 Mar 2021 22:00:25 +0000 (22:00 +0000)
commit08442cc79216956eff03a9b35040e6b1fff11ac1
tree173abc256ee2697ea34f5252e49db808657899a2
parent80792e12d4752a05ed80f56a5d1b69927a904c0a
cccombiner: implement scheduling

Prior to that, cccombiner's behaviour was essentially that of
a funnel: it strictly looked at input timestamps to associate
together video and caption buffers.

This patch instead exposes a "schedule" property, with a default
of TRUE, to control whether caption buffers should be smoothly
scheduled, in order to have exactly one per output video buffer.

This can involve rewriting input captions, for example when the
input is CDP sequence counters are rewritten, time codes are dropped
and potentially re-injected if the input video frame had a time code
meta.

Caption buffers may also get split up in order to assign captions to
the correct field when the input is interlaced.

This can also imply that the input will drift from synchronization,
when there isn't enough padding in the input stream to catch up. In
that case the element will start dropping old caption buffers once
the number of buffers in its internal queue reaches a certain limit
(configurable).

The property is exposed so that existing users of cccombiner can
revert back to the original behaviour, but should eventually be
removed, as that behaviour was simply inadequate.

This commit also disallows changing the input caption type, as
this would needlessly complicate implementation, and removes
the corresponding test.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/2076>
docs/plugins/gst_plugins_cache.json
ext/closedcaption/gstcccombiner.c
ext/closedcaption/gstcccombiner.h
tests/check/elements/cccombiner.c