codecalphademux: Do not set a GstFlowReturn from a boolean

This was a small overlook, gst_pad_send_event() returns a boolean,
so setting it into ret could confuse the flow combiner. Though,
it didn't bug, since both 0 and 1 are success (though 1 being
undefined).

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/2247>
This commit is contained in:
Nicolas Dufresne 2021-05-14 14:05:59 -04:00
parent 35775f1aec
commit 0484d658a8

View file

@ -116,8 +116,8 @@ gst_codec_alpha_demux_chain (GstPad * pad, GstObject * object,
if (alpha_buffer) {
ret = gst_pad_push (self->alpha_pad, alpha_buffer);
} else {
ret = gst_pad_push_event (self->alpha_pad,
gst_event_new_gap (pts, duration));
gst_pad_push_event (self->alpha_pad, gst_event_new_gap (pts, duration));
ret = GST_PAD_LAST_FLOW_RETURN (self->alpha_pad);
}
ret = gst_flow_combiner_update_flow (self->flow_combiner, ret);