va: basedec: No need to call base class' decide_allocation().
authorHe Junyan <junyan.he@intel.com>
Sun, 25 Oct 2020 17:22:12 +0000 (01:22 +0800)
committerGStreamer Merge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Tue, 27 Oct 2020 14:18:07 +0000 (14:18 +0000)
We have already done the jobs in gst_va_base_dec_decide_allocation()
and no need to call base class' decide_allocation() again. The base
class' decide_allocation() will set_format() again and let use do the
image/surface testing again, which is low performance and no needed.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/1698>

sys/va/gstvabasedec.c

index e544c3b0f43fba6e5a5f72aae52e81da68e4c580..87aa8f5bced145fb5b77d6eda9dbc06961c9872c 100644 (file)
@@ -334,8 +334,7 @@ gst_va_base_dec_decide_allocation (GstVideoDecoder * decoder, GstQuery * query)
   gst_object_unref (allocator);
   gst_object_unref (pool);
 
-  return GST_VIDEO_DECODER_CLASS (parent_class)->decide_allocation (decoder,
-      query);
+  return TRUE;
 
 wrong_caps:
   {