From: gb <gb@5584edef-b1fe-4b99-b61b-dd2bab72e969>
Date: Thu, 18 Mar 2010 08:16:59 +0000 (+0000)
Subject: Factor out buffers negotiation and optimization checks.
X-Git-Tag: accepted/trunk/20120822.173359~696
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=79c168751a1fc86f39ed5e04469890fd8849b3f8;p=profile%2Fivi%2Fgstreamer-vaapi.git

Factor out buffers negotiation and optimization checks.
---

diff --git a/sys/vaapiconvert/gstvaapiconvert.c b/sys/vaapiconvert/gstvaapiconvert.c
index c93dee9..a30aceb 100644
--- a/sys/vaapiconvert/gstvaapiconvert.c
+++ b/sys/vaapiconvert/gstvaapiconvert.c
@@ -186,14 +186,15 @@ gst_vaapiconvert_init(GstVaapiConvert *convert, GstVaapiConvertClass *klass)
 {
     GstPad *sinkpad;
 
-    convert->display            = NULL;
-    convert->images             = NULL;
-    convert->image_width        = 0;
-    convert->image_height       = 0;
-    convert->surfaces           = NULL;
-    convert->surface_width      = 0;
-    convert->surface_height     = 0;
-    convert->use_inout_buffers  = 0;
+    convert->display                    = NULL;
+    convert->images                     = NULL;
+    convert->image_width                = 0;
+    convert->image_height               = 0;
+    convert->surfaces                   = NULL;
+    convert->surface_width              = 0;
+    convert->surface_height             = 0;
+    convert->can_use_inout_buffers      = FALSE;
+    convert->use_inout_buffers          = FALSE;
 
     /* Override buffer allocator on sink pad */
     sinkpad = gst_element_get_static_pad(GST_ELEMENT(convert), "sink");
@@ -358,7 +359,7 @@ gst_vaapiconvert_ensure_image_pool(GstVaapiConvert *convert, GstCaps *caps)
         if (gst_video_format_parse_caps(caps, &vformat, NULL, NULL)) {
             image = gst_vaapi_video_pool_get_object(convert->images);
             if (image) {
-                convert->use_inout_buffers =
+                convert->can_use_inout_buffers =
                     (gst_vaapi_image_is_linear(image) &&
                      (gst_vaapi_image_get_data_size(image) ==
                       gst_video_format_get_size(vformat, width, height)));
@@ -391,20 +392,37 @@ gst_vaapiconvert_ensure_surface_pool(GstVaapiConvert *convert, GstCaps *caps)
 }
 
 static gboolean
-gst_vaapiconvert_set_caps(
-    GstBaseTransform *trans,
+gst_vaapiconvert_negotiate_buffers(
+    GstVaapiConvert  *convert,
     GstCaps          *incaps,
     GstCaps          *outcaps
 )
 {
-    GstVaapiConvert * const convert = GST_VAAPICONVERT(trans);
-
     if (!gst_vaapiconvert_ensure_image_pool(convert, incaps))
         return FALSE;
 
     if (!gst_vaapiconvert_ensure_surface_pool(convert, outcaps))
         return FALSE;
 
+    convert->use_inout_buffers = convert->can_use_inout_buffers;
+    GST_DEBUG("use-inout-buffers: %spossible, %s",
+              convert->can_use_inout_buffers ? "" : "not ",
+              convert->use_inout_buffers ? "enabled" : "disabled");
+    return TRUE;
+}
+
+static gboolean
+gst_vaapiconvert_set_caps(
+    GstBaseTransform *trans,
+    GstCaps          *incaps,
+    GstCaps          *outcaps
+)
+{
+    GstVaapiConvert * const convert = GST_VAAPICONVERT(trans);
+
+    if (!gst_vaapiconvert_negotiate_buffers(convert, incaps, outcaps))
+        return FALSE;
+
     return TRUE;
 }
 
@@ -442,9 +460,7 @@ gst_vaapiconvert_buffer_alloc(
     GstVaapiImage *image;
 
     /* Check if we can use the inout-buffers optimization */
-    if (!gst_vaapiconvert_ensure_surface_pool(convert, caps))
-        goto error;
-    if (!gst_vaapiconvert_ensure_image_pool(convert, caps))
+    if (!gst_vaapiconvert_negotiate_buffers(convert, caps, caps))
         goto error;
     if (!convert->use_inout_buffers)
         return GST_FLOW_OK;
diff --git a/sys/vaapiconvert/gstvaapiconvert.h b/sys/vaapiconvert/gstvaapiconvert.h
index b37ff2d..c41877b 100644
--- a/sys/vaapiconvert/gstvaapiconvert.h
+++ b/sys/vaapiconvert/gstvaapiconvert.h
@@ -71,7 +71,8 @@ struct _GstVaapiConvert {
     GstVaapiVideoPool  *surfaces;
     guint               surface_width;
     guint               surface_height;
-    guint               use_inout_buffers : 1;
+    guint               can_use_inout_buffers   : 1;
+    guint               use_inout_buffers       : 1;
 };
 
 struct _GstVaapiConvertClass {