[Bug 748439] vtdec: strange error handling

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=748439

Ilya Konstantinov <ilya.konstantinov at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|gstreamer-bugs at lists.freede |ilya.konstantinov at gmail.com
                   |sktop.org                   |

--- Comment #3 from Ilya Konstantinov <ilya.konstantinov at gmail.com> ---
Created attachment 302442
  --> https://bugzilla.gnome.org/attachment.cgi?id=302442&action=edit
vtdec: use GST_VIDEO_DECODER_ERROR

This patch works. For normal decoding, everything is fine.

For vtenc_h264 ! breakmydata probability=0.1 ! vtdec, we get a series of errors
and bail out after GST_VIDEO_DECODER_MAX_ERRORS (= 10).

What troubles me is that 'gst_video_decoder_set_max_errors' is not called
anywhere (and in particular, not configurable via props). e.g. for an RTC
use-case, do we really want anything other than -1 ?

Also, to keep compatibility, I'd default vtdec's max_errors to -1.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.