diff options
author | Nicolas Dufresne <nicolas.dufresne@collabora.com> | 2017-09-06 13:55:13 -0400 |
---|---|---|
committer | Nicolas Dufresne <nicolas.dufresne@collabora.com> | 2017-09-06 14:20:46 -0400 |
commit | 2bf665486e02cf380175baae43308de7dc03e733 (patch) | |
tree | d783617e139e31b52c0dd9b837ebbe0855581dcc /sys | |
parent | 5d6fbcb0af73b0d43bbead5d2c140c308b677fcc (diff) | |
download | gst-plugins-base-2bf665486e02cf380175baae43308de7dc03e733.tar.gz gst-plugins-base-2bf665486e02cf380175baae43308de7dc03e733.tar.xz |
Request minimum buffer even if need_pool is FALSE
When tee is used, it will not request a pool, but still it wants to
know how many buffers are required.
https://bugzilla.gnome.org/show_bug.cgi?id=730758
Diffstat (limited to 'sys')
-rw-r--r-- | sys/ximage/ximagesink.c | 20 | ||||
-rw-r--r-- | sys/xvimage/xvimagesink.c | 20 |
2 files changed, 18 insertions, 22 deletions
diff --git a/sys/ximage/ximagesink.c b/sys/ximage/ximagesink.c index cf5f19c33..9176ca69d 100644 --- a/sys/ximage/ximagesink.c +++ b/sys/ximage/ximagesink.c @@ -1463,6 +1463,7 @@ gst_x_image_sink_propose_allocation (GstBaseSink * bsink, GstQuery * query) GstXImageSink *ximagesink = GST_X_IMAGE_SINK (bsink); GstBufferPool *pool = NULL; GstCaps *caps; + GstVideoInfo info; guint size; gboolean need_pool; @@ -1471,26 +1472,23 @@ gst_x_image_sink_propose_allocation (GstBaseSink * bsink, GstQuery * query) if (caps == NULL) goto no_caps; - if (need_pool) { - GstVideoInfo info; + if (!gst_video_info_from_caps (&info, caps)) + goto invalid_caps; - if (!gst_video_info_from_caps (&info, caps)) - goto invalid_caps; + /* the normal size of a frame */ + size = info.size; + if (need_pool) { pool = gst_x_image_sink_create_pool (ximagesink, caps, info.size, 0); - /* the normal size of a frame */ - size = info.size; - if (pool == NULL) goto no_pool; } - if (pool) { - /* we need at least 2 buffer because we hold on to the last one */ - gst_query_add_allocation_pool (query, pool, size, 2, 0); + /* we need at least 2 buffer because we hold on to the last one */ + gst_query_add_allocation_pool (query, pool, size, 2, 0); + if (pool) gst_object_unref (pool); - } /* we also support various metadata */ gst_query_add_allocation_meta (query, GST_VIDEO_META_API_TYPE, NULL); diff --git a/sys/xvimage/xvimagesink.c b/sys/xvimage/xvimagesink.c index 9cd0da1a4..4ca0c1135 100644 --- a/sys/xvimage/xvimagesink.c +++ b/sys/xvimage/xvimagesink.c @@ -1051,6 +1051,7 @@ gst_xv_image_sink_propose_allocation (GstBaseSink * bsink, GstQuery * query) GstXvImageSink *xvimagesink = GST_XV_IMAGE_SINK (bsink); GstBufferPool *pool = NULL; GstCaps *caps; + GstVideoInfo info; guint size; gboolean need_pool; @@ -1059,27 +1060,24 @@ gst_xv_image_sink_propose_allocation (GstBaseSink * bsink, GstQuery * query) if (caps == NULL) goto no_caps; - if (need_pool) { - GstVideoInfo info; + if (!gst_video_info_from_caps (&info, caps)) + goto invalid_caps; - if (!gst_video_info_from_caps (&info, caps)) - goto invalid_caps; + /* the normal size of a frame */ + size = info.size; + if (need_pool) { GST_DEBUG_OBJECT (xvimagesink, "create new pool"); pool = gst_xv_image_sink_create_pool (xvimagesink, caps, info.size, 0); - /* the normal size of a frame */ - size = info.size; - if (pool == NULL) goto no_pool; } - if (pool) { - /* we need at least 2 buffer because we hold on to the last one */ - gst_query_add_allocation_pool (query, pool, size, 2, 0); + /* we need at least 2 buffer because we hold on to the last one */ + gst_query_add_allocation_pool (query, pool, size, 2, 0); + if (pool) gst_object_unref (pool); - } /* we also support various metadata */ gst_query_add_allocation_meta (query, GST_VIDEO_META_API_TYPE, NULL); |