From: Gwenole Beauchesne Date: Mon, 23 Jul 2012 16:01:26 +0000 (+0200) Subject: pluginutils: cosmetics (indentation fixes). X-Git-Tag: accepted/trunk/20120822.173359~36 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=aa64ce0bede54466bfab34cdcdd9f10de101784a;p=profile%2Fivi%2Fgstreamer-vaapi.git pluginutils: cosmetics (indentation fixes). --- diff --git a/gst/vaapi/gstvaapipluginutil.c b/gst/vaapi/gstvaapipluginutil.c index e500e5c..ce16a43 100644 --- a/gst/vaapi/gstvaapipluginutil.c +++ b/gst/vaapi/gstvaapipluginutil.c @@ -25,169 +25,171 @@ # include "config.h" #endif #include +#include #if USE_X11 # include #endif #if USE_GLX # include #endif - #include "gstvaapipluginutil.h" -#include - /* Preferred first */ static const char *display_types[] = { - "gst-vaapi-display", - "vaapi-display", - "x11-display", - "x11-display-name", - NULL + "gst-vaapi-display", + "vaapi-display", + "x11-display", + "x11-display-name", + NULL }; gboolean -gst_vaapi_ensure_display (gpointer element, GstVaapiDisplay **display) +gst_vaapi_ensure_display(gpointer element, GstVaapiDisplay **display) { - GstVideoContext *context; + GstVideoContext *context; - g_return_val_if_fail (GST_IS_VIDEO_CONTEXT (element), FALSE); - g_return_val_if_fail (display != NULL, FALSE); + g_return_val_if_fail(GST_IS_VIDEO_CONTEXT(element), FALSE); + g_return_val_if_fail(display != NULL, FALSE); - /* Already exist ? */ - if (*display) - return TRUE; + /* Already exist ? */ + if (*display) + return TRUE; - context = GST_VIDEO_CONTEXT (element); - gst_video_context_prepare (context, display_types); + context = GST_VIDEO_CONTEXT(element); + gst_video_context_prepare(context, display_types); - /* If no neighboor, or application not interested, use system default */ + /* If no neighboor, or application not interested, use system default */ #if USE_GLX - if (!*display) - *display = gst_vaapi_display_glx_new (NULL); + if (!*display) + *display = gst_vaapi_display_glx_new(NULL); #endif - if (!*display) - *display = gst_vaapi_display_x11_new (NULL); - - /* FIXME allocator should return NULL in case of failure */ - if (*display && !gst_vaapi_display_get_display(*display)) { - g_object_unref (*display); - *display = NULL; - } + if (!*display) + *display = gst_vaapi_display_x11_new(NULL); - return (*display != NULL); + /* FIXME allocator should return NULL in case of failure */ + if (*display && !gst_vaapi_display_get_display(*display)) { + g_object_unref(*display); + *display = NULL; + } + return (*display != NULL); } void -gst_vaapi_set_display (const gchar *type, - const GValue *value, - GstVaapiDisplay **display) +gst_vaapi_set_display( + const gchar *type, + const GValue *value, + GstVaapiDisplay **display +) { - GstVaapiDisplay *dpy = NULL; + GstVaapiDisplay *dpy = NULL; - if (!strcmp (type, "x11-display-name")) { - g_return_if_fail (G_VALUE_HOLDS_STRING (value)); + if (!strcmp(type, "x11-display-name")) { + g_return_if_fail(G_VALUE_HOLDS_STRING(value)); #if USE_GLX - dpy = gst_vaapi_display_glx_new (g_value_get_string (value)); + dpy = gst_vaapi_display_glx_new(g_value_get_string(value)); #endif - if (!dpy) - dpy = gst_vaapi_display_x11_new (g_value_get_string (value)); - } else if (!strcmp (type, "x11-display")) { - g_return_if_fail (G_VALUE_HOLDS_POINTER (value)); + if (!dpy) + dpy = gst_vaapi_display_x11_new(g_value_get_string(value)); + } + else if (!strcmp(type, "x11-display")) { + g_return_if_fail(G_VALUE_HOLDS_POINTER(value)); #if USE_GLX - dpy = gst_vaapi_display_glx_new_with_display (g_value_get_pointer (value)); + dpy = gst_vaapi_display_glx_new_with_display(g_value_get_pointer(value)); #endif - if (!dpy) - dpy = gst_vaapi_display_x11_new_with_display (g_value_get_pointer (value)); - } else if (!strcmp (type, "vaapi-display")) { - g_return_if_fail (G_VALUE_HOLDS_POINTER (value)); - dpy = gst_vaapi_display_new_with_display (g_value_get_pointer (value)); - } else if (!strcmp (type, "gst-vaapi-display")) { - g_return_if_fail (G_VALUE_HOLDS_OBJECT (value)); - dpy = g_value_dup_object (value); - } - - if (dpy) { - if (*display) - g_object_unref (*display); - *display = dpy; - } + if (!dpy) + dpy = gst_vaapi_display_x11_new_with_display(g_value_get_pointer(value)); + } + else if (!strcmp(type, "vaapi-display")) { + g_return_if_fail(G_VALUE_HOLDS_POINTER(value)); + dpy = gst_vaapi_display_new_with_display(g_value_get_pointer(value)); + } + else if (!strcmp(type, "gst-vaapi-display")) { + g_return_if_fail(G_VALUE_HOLDS_OBJECT(value)); + dpy = g_value_dup_object(value); + } + + if (dpy) { + if (*display) + g_object_unref(*display); + *display = dpy; + } } gboolean -gst_vaapi_reply_to_query (GstQuery *query, GstVaapiDisplay *display) +gst_vaapi_reply_to_query(GstQuery *query, GstVaapiDisplay *display) { - const gchar **types; - const gchar *type; - gint i; - gboolean res = FALSE; - - if (!display) - return FALSE; - - types = gst_video_context_query_get_supported_types (query); - - if (!types) - return FALSE; - - for (i = 0; types[i]; i++) { - type = types[i]; - - if (!strcmp (type, "gst-vaapi-display")) { - gst_video_context_query_set_object (query, type, G_OBJECT (display)); - - } else if (!strcmp (type, "vaapi-display")) { - VADisplay vadpy = gst_vaapi_display_get_display(display); - gst_video_context_query_set_pointer (query, type, vadpy); - - } else if (!strcmp (type, "x11-display") && - GST_VAAPI_IS_DISPLAY_X11(display)) { - GstVaapiDisplayX11 *xvadpy = GST_VAAPI_DISPLAY_X11 (display); - Display *x11dpy = gst_vaapi_display_x11_get_display (xvadpy); - gst_video_context_query_set_pointer (query, type, x11dpy); - - } else if (!strcmp (type, "x11-display-name") && - GST_VAAPI_IS_DISPLAY_X11(display)) { - GstVaapiDisplayX11 *xvadpy = GST_VAAPI_DISPLAY_X11 (display); - Display *x11dpy = gst_vaapi_display_x11_get_display (xvadpy); - gst_video_context_query_set_string (query, type, DisplayString(x11dpy)); - - } else { - continue; + const gchar **types; + const gchar *type; + gint i; + gboolean res = FALSE; + + if (!display) + return FALSE; + + types = gst_video_context_query_get_supported_types(query); + + if (!types) + return FALSE; + + for (i = 0; types[i]; i++) { + type = types[i]; + + if (!strcmp(type, "gst-vaapi-display")) { + gst_video_context_query_set_object(query, type, G_OBJECT(display)); + } + else if (!strcmp(type, "vaapi-display")) { + VADisplay vadpy = gst_vaapi_display_get_display(display); + gst_video_context_query_set_pointer(query, type, vadpy); + } + else if (!strcmp(type, "x11-display") && + GST_VAAPI_IS_DISPLAY_X11(display)) { + GstVaapiDisplayX11 *xvadpy = GST_VAAPI_DISPLAY_X11(display); + Display *x11dpy = gst_vaapi_display_x11_get_display(xvadpy); + gst_video_context_query_set_pointer(query, type, x11dpy); + + } + else if (!strcmp(type, "x11-display-name") && + GST_VAAPI_IS_DISPLAY_X11(display)) { + GstVaapiDisplayX11 *xvadpy = GST_VAAPI_DISPLAY_X11(display); + Display *x11dpy = gst_vaapi_display_x11_get_display(xvadpy); + gst_video_context_query_set_string(query, type, DisplayString(x11dpy)); + } + else { + continue; + } + + res = TRUE; + break; } - - res = TRUE; - break; - } - - return res; + return res; } gboolean -gst_vaapi_append_surface_caps (GstCaps *out_caps, GstCaps *in_caps) +gst_vaapi_append_surface_caps(GstCaps *out_caps, GstCaps *in_caps) { - GstStructure *structure; - const GValue *v_width, *v_height, *v_framerate, *v_par; - guint i, n_structures; - - structure = gst_caps_get_structure (in_caps, 0); - v_width = gst_structure_get_value (structure, "width"); - v_height = gst_structure_get_value (structure, "height"); - v_framerate = gst_structure_get_value (structure, "framerate"); - v_par = gst_structure_get_value (structure, "pixel-aspect-ratio"); - if (!v_width || !v_height) - return FALSE; - - n_structures = gst_caps_get_size (out_caps); - for (i = 0; i < n_structures; i++) { - structure = gst_caps_get_structure (out_caps, i); - gst_structure_set_value (structure, "width", v_width); - gst_structure_set_value (structure, "height", v_height); - if (v_framerate) - gst_structure_set_value (structure, "framerate", v_framerate); - if (v_par) - gst_structure_set_value (structure, "pixel-aspect-ratio", v_par); - } - return TRUE; + GstStructure *structure; + const GValue *v_width, *v_height, *v_framerate, *v_par; + guint i, n_structures; + + structure = gst_caps_get_structure(in_caps, 0); + v_width = gst_structure_get_value(structure, "width"); + v_height = gst_structure_get_value(structure, "height"); + v_framerate = gst_structure_get_value(structure, "framerate"); + v_par = gst_structure_get_value(structure, "pixel-aspect-ratio"); + if (!v_width || !v_height) + return FALSE; + + n_structures = gst_caps_get_size(out_caps); + for (i = 0; i < n_structures; i++) { + structure = gst_caps_get_structure(out_caps, i); + gst_structure_set_value(structure, "width", v_width); + gst_structure_set_value(structure, "height", v_height); + if (v_framerate) + gst_structure_set_value(structure, "framerate", v_framerate); + if (v_par) + gst_structure_set_value(structure, "pixel-aspect-ratio", v_par); + } + return TRUE; } GType diff --git a/gst/vaapi/gstvaapipluginutil.h b/gst/vaapi/gstvaapipluginutil.h index 8d684b0..9c3812f 100644 --- a/gst/vaapi/gstvaapipluginutil.h +++ b/gst/vaapi/gstvaapipluginutil.h @@ -24,8 +24,6 @@ #ifndef GST_VAAPI_PLUGIN_UTIL_H #define GST_VAAPI_PLUGIN_UTIL_H -#include -#include #include /** @@ -47,9 +45,18 @@ enum _GstVaapiDisplayType { GType gst_vaapi_display_type_get_type(void) G_GNUC_CONST; -gboolean gst_vaapi_ensure_display (gpointer element, GstVaapiDisplay **display); -void gst_vaapi_set_display (const gchar *type, const GValue *value, GstVaapiDisplay **display); -gboolean gst_vaapi_reply_to_query (GstQuery *query, GstVaapiDisplay *display); +gboolean +gst_vaapi_ensure_display(gpointer element, GstVaapiDisplay **display); + +void +gst_vaapi_set_display( + const gchar *type, + const GValue *value, + GstVaapiDisplay **display +); + +gboolean +gst_vaapi_reply_to_query(GstQuery *query, GstVaapiDisplay *display); gboolean gst_vaapi_append_surface_caps (GstCaps *out_caps, GstCaps *in_caps);