Nirbheek Chauhan authored 4 years ago and GStreamer Marge Bot committed 4 years ago

We already do this for the plugin.

gst-plugins-base#780 (comment 548179)

Part-of: <!642>

Part-of: <gstreamer/gst-plugins-good!644>

When generating the cache we inspect the base class through
an instance of one of its subclasses. We don't want potential
assignments in subclasses initialization to leak into the
base class documentation

Part-of: <gstreamer/gst-plugins-good!641>

When generating the cache we inspect the base class through
an instance of one of its subclasses. We don't want potential
assignments in subclasses initialization to leak into the
base class documentation

Part-of: <gstreamer/gst-plugins-good!641>

ORC_RESTRICT may not be defined in yadif.c and cause build error.

Part-of: <!637>

Håvard Graff authored 5 years ago and GStreamer Marge Bot committed 4 years ago

Part-of: <gstreamer/gst-plugins-good!410>

Some default values include our version string, like
user agent strings.

Add host cpu type check as we would enable asm only for x86_64

Part-of: <!636>

ARCH_X86_64 define will enable GCC specific code path in dv_types.h
while building dv plugin.

Part-of: <gstreamer/gst-plugins-good!636>

shout2send caps depend on what the libshout2
version in question supports, but the
documentation caps should always be the same.

Part-of: <gstreamer/gst-plugins-good!635>

Jan Schmidt authored 5 years ago and GStreamer Marge Bot committed 4 years ago

Move some code out of the enormous qtdemux.c into a separate
qtdemux_tags helper, and make some structs available via qtdemux.h
to accommodate that.

Part-of: <gstreamer/gst-plugins-good!634>

Jan Schmidt authored 5 years ago and GStreamer Marge Bot committed 4 years ago

Reduce a tiny bit of the bulk of qtdemux.c by moving some
agnostic helper functions out.

Part-of: <gstreamer/gst-plugins-good!634>

Jan Schmidt authored 5 years ago and GStreamer Marge Bot committed 4 years ago

Move the SVMI stereoscopic atom parsing out to a helper
function to shrink qtdemux_parse_trak a bit.

Add a bounds check that the received atom is large enough
before parsing it.

Add a note to the atom parser that svmi comes from the
MPEG-A spec 23000-11.

Part-of: <gstreamer/gst-plugins-good!634>

Guillaume Desmottes authored 4 years ago and GStreamer Marge Bot committed 4 years ago

Fix regression introduced in 7bc5e28d
preventing the device provider to send the device-added message for new
devices.
By early returning the patch was discarding add/remove events.

Fix #735

Part-of: <!626>

Would get "Tried to create target "qt5-qmlsink_qrc", but a
target of that name already exists." with older meson versions.

Work around that by renaming the qrc file.

Part-of: <gstreamer/gst-plugins-good!633>

Fixes gstreamer/gst-plugins-good#751

Part-of: <gstreamer/gst-plugins-good!631>

Expected return value for unhandled query is FALSE

Part-of: <!629>

Measured to be about 3.4x faster than C

Part-of: <gst-plugins-good!621>

Part-of: <gstreamer/gst-plugins-good!621>

Matthew Waters authored 4 years ago and GStreamer Marge Bot committed 4 years ago

Reset to the original OpenGL state as required by the GStreamer OpenGL
API contract.  Fixes output with a glimagesink element downstream.

Part-of: <gstreamer/gst-plugins-good!623>

Matthew Waters authored 4 years ago and GStreamer Marge Bot committed 4 years ago

Qt may replace the drawable with its own which breaks output if Qt is
not displaying the resulting video as used with e.g. glimagesink.

Part-of: <gstreamer/gst-plugins-good!623>

It was accidentally changed in gstreamer/gst-plugins-good!436

Part-of: <gstreamer/gst-plugins-good!624>

gst_element_class_set_metadata is meant to only be used with
static or inlined strings, which isn't the case for the 2 elements
here resulting in use-after-free later on.

https://gstreamer.freedesktop.org/documentation/gstreamer/gstelement.html?gi-language=c#gst_element_class_set_static_metadata

Part-of: <gstreamer/gst-plugins-good!622>

Sebastian Dröge authored 4 years ago and GStreamer Marge Bot committed 4 years ago

This reverts commit 54810bf4

Part-of: <gstreamer/gst-plugins-good!620>

Define g_queue_clear_full if glib < 2.60.

Fixes #747

Part-of: <gstreamer/gst-plugins-good!619>

Set up our plugin include list for tests in such a way that
we don't pull in *all* plugins from -bad but only the one
used in the splitmuxsink unit test, i.e. the timecode plugin,
so we don't accidentally use other encoders/decoders such as
nvenc/dec for example.

Part-of: <gstreamer/gst-plugins-good!617>

While the caller should make sure this does not happen, make sure timer
collision are not silently ignored and leaked.

Fixes #726

Part-of: <gstreamer/gst-plugins-good!616>

Until now, do_expected_timeout() was shortly dropping the JBUF_LOCK in order
to push RTX event event without causing deadlock. As a side effect, some
CPU hung would happen as the timerqueue would get filled while looping over
the due timers. To mitigate this, we were processing the lost timer first and
placing into a queue the remainign to be processed later.

In the gap caused by an unlock, we could endup receiving one of the seqnum
present in the pending timers. In that case, the timer would not be found and
a new one was created. When we then update the expected timer, the seqnum
would already exist and the updated timer would be lost.

In this patch we remove the unlock from do_expected_timeout() and place all
pending RTX event into a queue (instead of pending timer). Then, as soon as
we have selected a timer to wait (or if there is no timer to wait for) we send
all the upstream RTX events. As we no longer unlock, we no longer need to pop
more then one timer from the queue, and we do so with the lock held, which
blocks any new colliding timers from being created.

Part-of: <gstreamer/gst-plugins-good!616>