codecs: vp8decoder: Use GstFlowReturn everywhere
authorSeungha Yang <seungha@centricular.com>
Fri, 17 Sep 2021 14:23:06 +0000 (23:23 +0900)
committerNicolas Dufresne <nicolas@ndufresne.ca>
Mon, 20 Sep 2021 13:03:44 +0000 (13:03 +0000)
commitaaeb76f09c5e3c83439fbfebfd15f36767b34f0d
treebe28d882ee7e96185e67e7b9c66cd7eddf0921b7
parentfcad4cc646a23e4e621ec5e8485958ab78d98090
codecs: vp8decoder: 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/gst-plugins-bad/-/merge_requests/2528>
gst-libs/gst/codecs/gstvp8decoder.c
gst-libs/gst/codecs/gstvp8decoder.h
sys/d3d11/gstd3d11vp8dec.cpp
sys/nvcodec/gstnvvp8dec.c
sys/v4l2codecs/gstv4l2codecvp8dec.c
sys/va/gstvavp8dec.c