gstaudiobasesink should it return ERROR for clips with audio corrupt track.
Submitted by Lyon
Link to original bug (#758215)
Description
When I'm playing to playing clips with audio corrupt track.
and it will just quit and report error in gst_audio_base_sink_wait_event()
that "sink not negotiated before EOS event"
while on GST1.4.5, these clips doesn't meet this issue.
I checked gst_audio_base_sink_wait_event()
there is a commit for this on GST1.6:
commit 1df69786
Author: Jan Schmidt jan@centricular.com
Date: Wed Jan 28 22:42:17 2015 +1100
audiobasesink: Make sure the ringbuffer is started before waiting
Don't call the basesink wait_event implementation until we're sure
the ringbuffer is running, because it might wait on a non-running
clock.
But for those clips with corrupt audio track, there might be no output from audio decoder, and thus the audio track have no chance to negotiate.
Do it really need return GST_ELEMENT_ERROR() and quit playing for those clips with audio corrupt track?
Similar as https://bugzilla.gnome.org/show_bug.cgi?id=741542 audio/video decoder error, we've tried not to return GST_ELEMENT_ERROR(), and keep other track to play normally.
could we just print warning(error) instead as what we do for audiodecoder.
Version: 1.6.0