Re: [PATCH spice-common] proto: Add some documentation to stream_report message

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 29, 2017 at 06:34:52AM -0500, Frediano Ziglio wrote:
> > 
> > On Tue, Nov 28, 2017 at 10:43:33AM +0000, Frediano Ziglio wrote:
> > > Most of the documentation is extracted from notes in spice-server
> > > code and comments.
> > > 
> > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> > > ---
> > >  spice.proto | 12 ++++++++++++
> > >  1 file changed, 12 insertions(+)
> > > 
> > > diff --git a/spice.proto b/spice.proto
> > > index ea37225..c1e7b95 100644
> > > --- a/spice.proto
> > > +++ b/spice.proto
> > > @@ -991,11 +991,23 @@ channel DisplayChannel : BaseChannel {
> > >      message {
> > >          uint32 stream_id;
> > >          uint32 unique_id;
> > > +        // the mm_time of the first frame included in the report
> > >          uint32 start_frame_mm_time;
> > > +        // the mm_time of the last_frame included in the report
> > 
> > "last frame"? (no underscore)
> > 
> > >          uint32 end_frame_mm_time;
> > > +        // the number of frames that reached the client during the time
> > > the
> > > +        // report is referring to
> > > +        // A special case is when num_frames == 0 and num_drops ==
> > > +        // 0xffffffffu, this is used in the client to communicate that
> > > +        // specific stream is not supported
> > 
> > "a specific stream"?
> > 
> 
> maye be "this is used in the client to communicate that the stream is not supported
> or client was not able to decode it"

"or that the client was not ..."
Looks good to me.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]