[Xcb] [PATCH] sync: add missing state in AlarmNotify event

Peter Harris peter.harris at hummingbird.com
Thu Sep 4 08:21:14 PDT 2008


Julien Danjou wrote:
> 
> diff --git a/src/sync.xml b/src/sync.xml
> index d90c714..2a80395 100644
> --- a/src/sync.xml
> +++ b/src/sync.xml
> @@ -190,5 +190,7 @@ for licensing information.
>      <field type="INT64" name="counter_value" />
>      <field type="INT64" name="alarm_value" />
>      <field type="TIMESTAMP" name="timestamp" />
> +    <field type="ALARMSTATE" name="state" />
> +    <pad bytes="3" />
>    </event>
>  </xcb>

ALARMSTATE is an enum, and enums don't have a size associated with them.

I'd love to change that (or at least associate enums with the fields
they will be used with), but using an enum in place of a type isn't
currently permitted, AFAIK.

Should probably be CARD8 for now (possibly with a comment indicating it
is an ALARMSTATE).

Peter Harris
-- 
     Hummingbird Connectivity - A Division of Open Text
Peter Harris                    http://connectivity.hummingbird.com
Research and Development        Phone: +1 905 762 6001
peter.harris at hummingbird.com    Toll Free: 1 877 359 4866


More information about the Xcb mailing list