[Spice-devel] [protocol v10 01/27] protocol: Add support for the VP8 and h264 video codecs

[Frediano Ziglio]

> Acked-by: Christophe Fergeau <cfergeau at redhat.com>
> 

Agree!

Should we bump the spice-protocol version too?

Frediano

> On Tue, Mar 01, 2016 at 04:50:16PM +0100, Francois Gouget wrote:
> > Clients that support multiple codecs must advertise the
> > SPICE_DISPLAY_CAP_MULTI_CODEC capability and one
> > SPICE_DISPLAY_CAP_CODEC_XXX per supported codec.
> > 
> > Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
> > ---
> >  spice.proto      | 2 ++
> >  spice/enums.h    | 2 ++
> >  spice/protocol.h | 4 ++++
> >  3 files changed, 8 insertions(+)
> > 
> > diff --git a/spice.proto b/spice.proto
> > index af970f2..4a0778d 100644
> > --- a/spice.proto
> > +++ b/spice.proto
> > @@ -329,6 +329,8 @@ flags8 path_flags { /* TODO: C enum names changes */
> >  
> >  enum8 video_codec_type {
> >      MJPEG = 1,
> > +    VP8,
> > +    H264,
> >  };
> >  
> >  flags8 stream_flags {
> > diff --git a/spice/enums.h b/spice/enums.h
> > index 613db52..c6e9840 100644
> > --- a/spice/enums.h
> > +++ b/spice/enums.h
> > @@ -139,6 +139,8 @@ typedef enum SpicePathFlags {
> >  
> >  typedef enum SpiceVideoCodecType {
> >      SPICE_VIDEO_CODEC_TYPE_MJPEG = 1,
> > +    SPICE_VIDEO_CODEC_TYPE_VP8,
> > +    SPICE_VIDEO_CODEC_TYPE_H264,
> >  
> >      SPICE_VIDEO_CODEC_TYPE_ENUM_END
> >  } SpiceVideoCodecType;
> > diff --git a/spice/protocol.h b/spice/protocol.h
> > index 3e6c624..f4a2259 100644
> > --- a/spice/protocol.h
> > +++ b/spice/protocol.h
> > @@ -137,6 +137,10 @@ enum {
> >      SPICE_DISPLAY_CAP_LZ4_COMPRESSION,
> >      SPICE_DISPLAY_CAP_PREF_COMPRESSION,
> >      SPICE_DISPLAY_CAP_GL_SCANOUT,
> > +    SPICE_DISPLAY_CAP_MULTI_CODEC,
> > +    SPICE_DISPLAY_CAP_CODEC_MJPEG,
> > +    SPICE_DISPLAY_CAP_CODEC_VP8,
> > +    SPICE_DISPLAY_CAP_CODEC_H264,
> >  };
> >  
> >  enum {
> > --
> > 2.7.0
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
>