[Spice-devel] [spice-gtk] streaming: Allows protocol to have more codecs

Pavel Grunt pgrunt at redhat.com
Mon Jan 9 09:57:04 UTC 2017


Ack

Pavel

On Mon, 2017-01-09 at 09:51 +0000, Frediano Ziglio wrote:
> The source file contains a list of fixed codecs for streaming.
> This list is tested to include all codecs from spice-protocol.
> If spice-protocol is extended to add more codecs this file starts
> failing to compile due to missing codecs.
> This patch allow to compile supporting less codecs than the
> ones listed in spice-protocol.
> 
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> ---
>  src/channel-display-gst.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c
> index f52299f..1f8fce5 100644
> --- a/src/channel-display-gst.c
> +++ b/src/channel-display-gst.c
> @@ -79,10 +79,10 @@ static struct {
>  
>  };
>  
> -G_STATIC_ASSERT(G_N_ELEMENTS(gst_opts) ==
> SPICE_VIDEO_CODEC_TYPE_ENUM_END);
> +G_STATIC_ASSERT(G_N_ELEMENTS(gst_opts) <=
> SPICE_VIDEO_CODEC_TYPE_ENUM_END);
>  
>  #define VALID_VIDEO_CODEC_TYPE(codec) \
> -    (codec > 0 && codec < SPICE_VIDEO_CODEC_TYPE_ENUM_END)
> +    (codec > 0 && codec < G_N_ELEMENTS(gst_opts))
>  
>  /* ---------- SpiceFrame ---------- */
>  


More information about the Spice-devel mailing list