codecs: mpeg2decoder: Use GstFlowReturn everywhere
authorSeungha Yang <seungha@centricular.com>
Tue, 21 Sep 2021 15:05:43 +0000 (00:05 +0900)
committerNicolas Dufresne <nicolas@ndufresne.ca>
Mon, 4 Oct 2021 20:56:46 +0000 (20:56 +0000)
commit7c557c2d6554695d3c862a96bf114f248a6dbfa9
tree1372972b676058f72f4c128d9154168c1b6f3fcc
parente3227457634732134ae0588d1ff9c7e010f011d9
codecs: mpeg2decoder: Use GstFlowReturn everywhere

boolean return value is not sufficient for representing the reason
of error in most cases. For instance, any errors around new_sequence()
would mean negotiation error, not just *ERROR*.
And some subclasses will allocate buffer/memory/surface on new_picture()
but it could be failed because of expected error, likely flushing

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1019>
subprojects/gst-plugins-bad/gst-libs/gst/codecs/gstmpeg2decoder.c
subprojects/gst-plugins-bad/gst-libs/gst/codecs/gstmpeg2decoder.h
subprojects/gst-plugins-bad/sys/d3d11/gstd3d11mpeg2dec.cpp
subprojects/gst-plugins-bad/sys/va/gstvampeg2dec.c