>
> From: Victor Toso <me@xxxxxxxxxxxxxx>
>
> This way we have a map of the necessary GstCaps to a given
> SPICE_VIDEO_CODEC_TYPE.
>
> This patch is also a preparatory patch to:
>
> - Identify which GstElements in GstRegistry can handle this GstCaps
>
> - Use Playbin as wrapper to all elements beside GstAppSrc and
> GstAppSink. In this case, we should rely on GstCaps to reduce
> typefind errors as we should know what kind of data is expected
>
> Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>
> Signed-off-by: Victor Toso <me@xxxxxxxxxxxxxx>
> ---
> src/channel-display-gst.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c
> index c4190b2..8c34b77 100644
> --- a/src/channel-display-gst.c
> +++ b/src/channel-display-gst.c
> @@ -62,14 +62,14 @@ static struct {
> { "decodebin", "" },
>
> /* SPICE_VIDEO_CODEC_TYPE_MJPEG */
> - { "jpegdec", "caps=image/jpeg" },
> + { "jpegdec", "image/jpeg" },
>
> /* SPICE_VIDEO_CODEC_TYPE_VP8
> *
> * typefind is unable to identify VP8 streams by design.
> * See: https://bugzilla.gnome.org/show_bug.cgi?id=756457
> */
> - { "vp8dec", "caps=video/x-vp8" },
> + { "vp8dec", "video/x-vp8" },
>
> /* SPICE_VIDEO_CODEC_TYPE_H264
> * h264 streams detection works fine and setting an incomplete cap
> @@ -78,7 +78,7 @@ static struct {
> { "h264parse ! avdec_h264", "" },
>
> /* SPICE_VIDEO_CODEC_TYPE_VP9 */
> - { "vp9dec", "caps=video/x-vp9" },
> + { "vp9dec", "video/x-vp9" },
> };
>
> G_STATIC_ASSERT(G_N_ELEMENTS(gst_opts) <= SPICE_VIDEO_CODEC_TYPE_ENUM_END);
> @@ -330,7 +330,7 @@ static gboolean create_pipeline(SpiceGstDecoder *decoder)
> * needed by those that follow.
> */
> desc = g_strdup_printf("appsrc name=src is-live=true format=time
> max-bytes=0 block=true "
> - "%s ! %s ! videoconvert ! appsink name=sink "
> + "caps=%s ! %s ! videoconvert ! appsink name=sink
> "
> "caps=video/x-raw,format=BGRx sync=false
> drop=false",
> gst_opts[opt].dec_caps, gst_opts[opt].dec_name);
> SPICE_DEBUG("GStreamer pipeline: %s", desc);
This for h264 is "caps=" without caps specified.
Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel