From f3650381c356885b6150fd9b0c8a97363b880c13 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=ADctor=20Manuel=20J=C3=A1quez=20Leal?= Date: Thu, 18 Jan 2018 13:10:59 +0100 Subject: [PATCH] vaapisink: check for display's color-balance properties Check for display's color-balance properties, available by the VA-API driver, before setting them. Also logs an info message of those unavailable properties. https://bugzilla.gnome.org/show_bug.cgi?id=792638 --- gst/vaapi/gstvaapisink.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/gst/vaapi/gstvaapisink.c b/gst/vaapi/gstvaapisink.c index a4bea49425..8bd8303098 100644 --- a/gst/vaapi/gstvaapisink.c +++ b/gst/vaapi/gstvaapisink.c @@ -680,8 +680,10 @@ cb_sync_values_from_display (GstVaapiSink * sink, GstVaapiDisplay * display) for (i = 0; i < G_N_ELEMENTS (sink->cb_values); i++) { const guint cb_id = CB_HUE + i; - if (!gst_vaapi_display_has_property (display, cb_map[i].prop_name)) + if (!gst_vaapi_display_has_property (display, cb_map[i].prop_name)) { + GST_INFO_OBJECT (sink, "backend does not handle %s", cb_map[i].prop_name); continue; + } value = 0.0; g_object_get (display, cb_map[i].prop_name, &value, NULL); @@ -700,6 +702,10 @@ cb_sync_values_to_display (GstVaapiSink * sink, GstVaapiDisplay * display) const guint cb_id = CB_HUE + i; if (!(sink->cb_changed & (1U << cb_id))) continue; + if (!gst_vaapi_display_has_property (display, cb_map[i].prop_name)) { + GST_INFO_OBJECT (sink, "backend does not handle %s", cb_map[i].prop_name); + continue; + } g_object_set_property (G_OBJECT (display), cb_map[i].prop_name, cb_get_gvalue (sink, cb_id));