Re: [PATCH spice-common] ptypes: Improve some attribute documentation

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

 



A couple minor language suggestions:


On Wed, 2018-06-27 at 17:47 +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  python_modules/ptypes.py | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/python_modules/ptypes.py b/python_modules/ptypes.py
> index 056e984..22a6f7b 100644
> --- a/python_modules/ptypes.py
> +++ b/python_modules/ptypes.py
> @@ -67,6 +67,7 @@ valid_attributes=set([
>      'end',
>      # the C structure contains a pointer to data
>      # for instance we want to write an array to an allocated array
> +    # The demarshaller allocate space for data pointed by this
> pointer.

allocate -> allocates
data pointed by this pointer -> either:
  A) data pointed to by this pointer
  B) data that this pointer points to

>      'to_ptr',
>      # write output to this C structure
>      'ctype',
> @@ -75,8 +76,12 @@ valid_attributes=set([
>      # used in demarshaller to use directly data from message without
> a copy
>      'nocopy',
>      # store member array in a pointer
> -    # similar to to_ptr but has an additional argument which is the
> name of a C
> -    # field which will store the array length
> +    # Has an argument which is the name of a C field which will
> store the
> +    # array length
> +    # The demarshaller store a reference to the original message
> buffer so

store -> stores

> +    # you should keep a reference to the original message to avoid a
> dandling

dandling -> dangling

> +    # pointer
> +    # Is useful for large buffers to avoid extra memory allocation
> and copy

copy -> copying

>      'as_ptr',
>      # do not generate marshall code
>      # used for last members to be able to marshall them manually


Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>

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




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