From: Victor Toso <me@xxxxxxxxxxxxxx> Move declaration of variable to top and break big g_strdup_printf line. Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx> --- src/channel-display-gst.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c index 430bb95..0c8fc01 100644 --- a/src/channel-display-gst.c +++ b/src/channel-display-gst.c @@ -250,6 +250,10 @@ static void free_pipeline(SpiceGstDecoder *decoder) static gboolean create_pipeline(SpiceGstDecoder *decoder) { const gchar *src_caps, *gstdec_name; + gchar *desc; + GstAppSinkCallbacks appsink_cbs = { 0 }; + GError *err = NULL; + switch (decoder->base.codec_type) { case SPICE_VIDEO_CODEC_TYPE_MJPEG: src_caps = "caps=image/jpeg"; @@ -293,10 +297,12 @@ static gboolean create_pipeline(SpiceGstDecoder *decoder) * - Set max-bytes=0 on appsrc so it does not drop frames that may be * needed by those that follow. */ - gchar *desc = g_strdup_printf("appsrc name=src is-live=true format=time max-bytes=0 block=true %s ! %s ! videoconvert ! appsink name=sink caps=video/x-raw,format=BGRx sync=false drop=false", src_caps, gstdec_name); + desc = g_strdup_printf("appsrc name=src is-live=true format=time max-bytes=0 block=true " + "%s ! %s ! videoconvert ! appsink name=sink " + "caps=video/x-raw,format=BGRx sync=false drop=false", + src_caps, gstdec_name); SPICE_DEBUG("GStreamer pipeline: %s", desc); - GError *err = NULL; decoder->pipeline = gst_parse_launch_full(desc, NULL, GST_PARSE_FLAG_FATAL_ERRORS, &err); g_free(desc); if (!decoder->pipeline) { @@ -307,7 +313,8 @@ static gboolean create_pipeline(SpiceGstDecoder *decoder) decoder->appsrc = GST_APP_SRC(gst_bin_get_by_name(GST_BIN(decoder->pipeline), "src")); decoder->appsink = GST_APP_SINK(gst_bin_get_by_name(GST_BIN(decoder->pipeline), "sink")); - GstAppSinkCallbacks appsink_cbs = {NULL, NULL, &new_sample, {NULL}}; + + appsink_cbs.new_sample = new_sample; gst_app_sink_set_callbacks(decoder->appsink, &appsink_cbs, decoder, NULL); decoder->clock = gst_pipeline_get_clock(GST_PIPELINE(decoder->pipeline)); -- 2.9.3 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel