omxvideoenc: revert draining on ALLOCATION and DRAIN query
authorGuillaume Desmottes <guillaume.desmottes@collabora.com>
Fri, 20 Sep 2019 09:32:24 +0000 (15:02 +0530)
committerGuillaume Desmottes <guillaume.desmottes@collabora.com>
Mon, 23 Sep 2019 08:20:54 +0000 (13:50 +0530)
My latest patch introduces some regressions which I have no time to
debug properly at the moment so just revert it for now.

omx/gstomxvideoenc.c

index a007dad..f266596 100644 (file)
@@ -231,8 +231,6 @@ static GstCaps *gst_omx_video_enc_getcaps (GstVideoEncoder * encoder,
     GstCaps * filter);
 static gboolean gst_omx_video_enc_decide_allocation (GstVideoEncoder * encoder,
     GstQuery * query);
-static gboolean gst_omx_video_enc_sink_query (GstVideoEncoder * encoder,
-    GstQuery * query);
 
 static GstFlowReturn gst_omx_video_enc_drain (GstOMXVideoEnc * self);
 
@@ -490,8 +488,6 @@ gst_omx_video_enc_class_init (GstOMXVideoEncClass * klass)
   video_encoder_class->getcaps = GST_DEBUG_FUNCPTR (gst_omx_video_enc_getcaps);
   video_encoder_class->decide_allocation =
       GST_DEBUG_FUNCPTR (gst_omx_video_enc_decide_allocation);
-  video_encoder_class->sink_query =
-      GST_DEBUG_FUNCPTR (gst_omx_video_enc_sink_query);
 
   klass->cdata.type = GST_OMX_COMPONENT_TYPE_FILTER;
   klass->cdata.default_sink_template_caps =
@@ -3299,26 +3295,3 @@ gst_omx_video_enc_decide_allocation (GstVideoEncoder * encoder,
 
   return TRUE;
 }
-
-static gboolean
-gst_omx_video_enc_sink_query (GstVideoEncoder * encoder, GstQuery * query)
-{
-  GstOMXVideoEnc *self = GST_OMX_VIDEO_ENC (encoder);
-
-  switch (GST_QUERY_TYPE (query)) {
-    case GST_QUERY_ALLOCATION:
-    case GST_QUERY_DRAIN:
-    {
-      GST_DEBUG_OBJECT (encoder, "%s query: drain encoder",
-          GST_QUERY_TYPE_NAME (query));
-
-      gst_omx_video_enc_drain (self);
-    }
-    default:
-      break;
-  }
-
-  return
-      GST_VIDEO_ENCODER_CLASS (gst_omx_video_enc_parent_class)->sink_query
-      (encoder, query);
-}