[Spice-devel] [PATCH spice-common v5 1/9] protocol: add preferred video codec message

[Victor Toso]

Hi,

On Fri, Jan 06, 2017 at 04:16:56AM -0500, Frediano Ziglio wrote:
> > 
> > From: Victor Toso <me at victortoso.com>
> > 
> > Client might want to choose a preferred video codec for streaming for
> > different reasons which having hardware decoder support being the most
> > interest one.
> > 
> > This message allows the client to send an array of video codecs in
> > order of preference.
> > 
> > Signed-off-by: Victor Toso <victortoso at redhat.com>
> > ---
> >  common/messages.h | 5 +++++
> >  spice.proto       | 5 +++++
> >  2 files changed, 10 insertions(+)
> > 
> > diff --git a/common/messages.h b/common/messages.h
> > index 516a345..5316cc0 100644
> > --- a/common/messages.h
> > +++ b/common/messages.h
> > @@ -644,6 +644,11 @@ typedef struct SpiceMsgcPortEvent {
> >      uint8_t event;
> >  } SpiceMsgcPortEvent;
> >  
> > +typedef struct SpiceMsgcDisplayPreferredVideoCodecType {
> > +    uint32_t num_of_codecs;
> > +    uint8_t codecs[0];
> > +} SpiceMsgcDisplayPreferredVideoCodecType;
> > +
> >  typedef struct SpiceMsgcDisplayPreferredCompression {
> >      uint8_t image_compression;
> >  } SpiceMsgcDisplayPreferredCompression;
> > diff --git a/spice.proto b/spice.proto
> > index 0bfc515..758f9bb 100644
> > --- a/spice.proto
> > +++ b/spice.proto
> > @@ -985,6 +985,11 @@ channel DisplayChannel : BaseChannel {
> >  
> >      message {
> >      } gl_draw_done;
> > +
> > +    message {
> > +        uint32 num_of_codecs;
> 
> I remember we discussed about uint8 instead

Yes, really sorry about that. I've just send a v6.

>
> > +        video_codec_type codecs[num_of_codecs] @end;
> > +    } preferred_video_codec_type;
> >  };
> >  
> >  flags16 keyboard_modifier_flags {
> 
> Frediano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170106/cf7e8ec2/attachment.sig>