codecs: vp9decoder: Make duplicate_picture() vfunc optional
authorSeungha Yang <seungha@centricular.com>
Fri, 9 Apr 2021 03:45:46 +0000 (12:45 +0900)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Fri, 9 Apr 2021 12:13:24 +0000 (12:13 +0000)
commitf7a341a1f0f7ebe603af573f33b02d5c6445ae02
treef90cc0e61f54006825ae2f607a7cb725555cfe65
parent933ebba43535d75a3274f519cbcaf70863a9464c
codecs: vp9decoder: Make duplicate_picture() vfunc optional

The default implementation was required when superframe parsing
was handled by vp9decoder. For instance, if a superframe consists
of multiple frames with show_existing_frame header, it was vague
that which GstVp9Picture should consume GstVideoCodecFrame.

After 1.18 release, we introduced vp9parse element and
superframe should be handled by upstream vp9parse elemenet now.
So, we don't need to care about the superframe at vp9decoder class
level anymore. Simply, a frame corresponding to show_existing_frame
can be dropped if subclass doesn't implement duplicate_picture().

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/2151>
gst-libs/gst/codecs/gstvp9decoder.c