[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-xcb
Subject:    Re: [Xcb] [PATCH proto 2/2 RESEND] xml-xcb.txt: remove description of the "localfield" element
From:       Christian Linhart <chris () DemoRecorder ! com>
Date:       2015-05-30 10:25:39
Message-ID: 55699023.3080100 () DemoRecorder ! com
[Download RAW message or body]

Hi Ran,

Thank you for the updated patch and sorry for my delayed review.

Your patch removes an obsolete part from the documentation.
Thank  you for noticing that.

Therefore:
Reviewed-by: Christian Linhart <chris@demorecorder.com>

I will push this patch right now because there was more than enough time for other reviews.

Chris

On 03/18/15 11:27, Ran Benita wrote:
> It was removed in:
> http://cgit.freedesktop.org/xcb/proto/commit/?id=854d08c8242e8fbe7b5a000b75ec6845419806c4
>
> Signed-off-by: Ran Benita <ran234@gmail.com>
> Reviewed-by: Peter Harris <pharris@opentext.com>
> ---
>  doc/xml-xcb.txt | 7 -------
>  1 file changed, 7 deletions(-)
>
> diff --git a/doc/xml-xcb.txt b/doc/xml-xcb.txt
> index 8d914a1..12c0881 100644
> --- a/doc/xml-xcb.txt
> +++ b/doc/xml-xcb.txt
> @@ -218,13 +218,6 @@ enum; the value is restricted to one of the constants named in the enum.
>    of the list in terms of other fields in the structure.  See the section
>    "Expressions" for details on the expression representation.
>  
> -<localfield type="identifier" name="identifier" />
> -
> -  This element represents a parameter in a request that is not sent over the
> -  wire.  The field can be referenced in the length expressions of lists or in
> -  an exprfield.  The type attribute declares the data type of the field, and
> -  the name attribute gives the name of the field.
> -
>  <exprfield type="identifier" name="identifier">expression</exprfield>
>  
>    This element represents a field in a request that is calculated rather than

_______________________________________________
Xcb mailing list
Xcb@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/xcb
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic