[Xcb] [PATCH 1/6] Move the INT64 types in sync.xml to the sync: namespace

Peter Harris pharris at opentext.com
Wed Nov 6 19:30:13 CET 2013


On 2013-11-06 13:11, Keith Packard wrote:
> Peter Harris <pharris at opentext.com> writes:
> 
>>> diff --git a/src/sync.xml b/src/sync.xml index 516d149..ae3bbbc
>>> 100644 --- a/src/sync.xml +++ b/src/sync.xml @@ -41,14 +41,14
>>> @@ for licensing information. <item name="Events">
>>> <bit>5</bit></item> </enum>
>>> 
>>> -  <struct name="INT64"> +  <struct name="sync:INT64">
>> 
>> This change isn't necessary, and should probably be omitted.
> 
> I'm sure I don't understand the XML that well, but why wouldn't I
> need the namespaced name for this instance?

This line declares a struct. All declarations are in the enclosing
namespace by definition.

The others are references, which aren't pinned to the enclosing
namespace (or there would be far too many xproto:Foo everywhere).

The oldest example of namespaces in xcb is glx.xml, which declares
 <xidtype name="WINDOW" />
(note the lack of a namespace in the declaration) and then uses
glx:WINDOW to reference it for the remainder of the file, or
xproto:WINDOW to reference the core WINDOW type.

Peter Harris
-- 
               Open Text Connectivity Solutions Group
Peter Harris                    http://connectivity.opentext.com/
Research and Development        Phone: +1 905 762 6001
pharris at opentext.com            Toll Free: 1 877 359 4866


More information about the Xcb mailing list