multiqueue: Improve incoming SEGMENT handling
authorEdward Hervey <edward@centricular.com>
Thu, 15 Oct 2015 14:32:42 +0000 (16:32 +0200)
committerEdward Hervey <bilboed@bilboed.com>
Mon, 19 Oct 2015 12:31:05 +0000 (14:31 +0200)
commit7c8f76a39f248bd7528b72e31460304f7c1a0b48
treee7dfb326bbd9c05b76206fca6057657e3d9c7961
parent55fae236d0a8695f19ba8b82b2dc7325245384d2
multiqueue: Improve incoming SEGMENT handling

Previously this code was just blindly setting the cached flow return
of downstream to GST_FLOW_OK when we get a SEGMENT.

The problem is that this can not be done blindly. If downstream was
not linked, the corresponding sinqlequeue source pad thread might be
waiting for the next ID to be woken up upon.

By blindly setting the cached return value to GST_FLOW_OK, and if that
stream was the only one that was NOT_LINKED, then the next time we
check (from any other thread) to see if we need to wake up a source pad
thread ... we won't even try, because none of the cached flow return
are equal to GST_FLOW_NOT_LINKED.

This would result in that thread never being woken up

https://bugzilla.gnome.org/show_bug.cgi?id=756645
plugins/elements/gstmultiqueue.c