[Spice-devel] [PATCH spice-gtk] use G_TYPE_POINTER instead of GST_TYPE_PIPELINE

Christophe Fergeau cfergeau at redhat.com
Thu Jan 3 15:00:43 UTC 2019


On Thu, Jan 03, 2019 at 04:46:29PM +0200, Snir Sheriber wrote:
> GST_TYPE_PIPELINE is undeclared when gstvideo is disabled

I'd rahter that we keep using the more specific type. Feel free to have
a 
#ifndef HAVE_GST_VIDEO
#define GST_TYPE_PIPELINE G_TYPE_POINTER
#endif
but would that code be used at all if gstreamer is not in use?

Christophe

> 
> Signed-off-by: Snir Sheriber <ssheribe at redhat.com>
> ---
>  src/channel-display.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/channel-display.c b/src/channel-display.c
> index 2a1d9d1..b0fe57e 100644
> --- a/src/channel-display.c
> +++ b/src/channel-display.c
> @@ -476,7 +476,7 @@ static void spice_display_channel_class_init(SpiceDisplayChannelClass *klass)
>                       g_cclosure_user_marshal_BOOLEAN__POINTER,
>                       G_TYPE_BOOLEAN,
>                       1,
> -                     GST_TYPE_PIPELINE);
> +                     G_TYPE_POINTER);
>  
>      channel_set_handlers(SPICE_CHANNEL_CLASS(klass));
>  }
> -- 
> 2.19.1
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20190103/fd32afb1/attachment.sig>


More information about the Spice-devel mailing list