summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* xv: Add align. pad in struct AdaptorInfobaserock/james/xcb-protoDaniel Martin2014-06-101-0/+1
| | | | | | | | | | | The (list) 'name' in the struct AdaptorInfo is not padded. Therefore we've to add an alignment pad after it. Reference: http://cgit.freedesktop.org/xorg/lib/libXv/tree/src/Xv.c#n250 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79808 Signed-off-by: Daniel Martin <consume.noise@gmail.com> Signed-off-by: Peter Harris <pharris@opentext.com>
* dri2: Add notes on lists that should be linked to their length fields but ↵Robert Ancell2014-06-101-2/+8
| | | | | | are not. Reviewed-by: Daniel Martin <consume.noise@gmail.com>
* xkb: Fix use of nKTLevelsPeter Harris2014-04-291-2/+2
| | | | | | | | | | | | nLevelsPerType is nLevels long. This continues the work started in 76ca2c0b1527541be59c344118c538ba055ad9d8. nKTLevels could be used instead of <sumof nLevelsPerType>, but that change causes a change to libxcb's API. Reviewed-by: Ran Benita <ran234@gmail.com> Tested-by: Ran Benita <ran234@gmail.com> Signed-off-by: Peter Harris <pharris@opentext.com>
* xprint: Fix <field type="STRING8"> in PrintGetAttributes replyVincent W. Chen2014-04-111-1/+3
| | | | | | | | | The reply for PrintGetAttributes contains a field of type STRING8, which makes no sense as the STRING8 type is only used to denote a string. This has been changed to <list type="STRING8"> with field "stringLen" as its length. Reviewed-by: Peter Harris <pharris@opentext.com>
* Remove extraneous spaces between <value> tagsVincent W. Chen2014-04-103-3/+3
| | | | | | | | These are the only places where spaces are allowed between <value> tags. This is not only inconsistent but also complicates the parsing of values as integers. Signed-off-by: Uli Schlachter <psychon@znc.in>
* Also track directly imported modules in a separate listKeith Packard2014-02-192-0/+6
| | | | | | | | | This allows the generated header files to only include the directly referenced header files, with the indirectly referenced header files included by the directly referenced ones. Signed-off-by: Keith Packard <keithp@keithp.com> Signed-off-by: Uli Schlachter <psychon@znc.in>
* xkb: Use <pad align="4" /> in GetMapPeter Harris2014-01-211-56/+4
| | | | | | | | | | | | | There is a bug where lists of size other than one are automatically aligned by c_client.py. So alignment_pad3 and 4 were aligning twice and consuming too many bytes. Rather than change the type to CARD8 and wrap the <op> in yet another <op> to multiply by two, use the new <pad align> operation. Reviewed-By: Ran Benita <ran234@gmail.com> Tested-By: Ran Benita <ran234@gmail.com> Signed-off-by: Peter Harris <pharris@opentext.com>
* Support <pad align="n" />Peter Harris2014-01-212-3/+8
| | | | | Reviewed-By: Ran Benita <ran234@gmail.com> Signed-off-by: Peter Harris <pharris@opentext.com>
* Track pad count at the module levelPeter Harris2014-01-212-4/+5
| | | | | | | | If the pad count is reset for each bitcase, the names will collide in the encompasing switch struct. Reviewed-By: Ran Benita <ran234@gmail.com> Signed-off-by: Peter Harris <pharris@opentext.com>
* Release xcb-proto 1.101.10Uli Schlachter2013-12-222-2/+2
| | | | Signed-off-by: Uli Schlachter <psychon@znc.in>
* Add news for 1.10Julien Cristau2013-12-141-0/+9
| | | | Signed-off-by: Julien Cristau <jcristau@debian.org>
* xinput: Explicitly specify NotifyMode and NotifyDetailPeter Harris2013-12-141-4/+4
| | | | | | | | | There is a NotifyMode and a NotifyDetail in both xproto and xinput. Add a namespece to the enum references to be clear about which one is to be used. Signed-off-by: Peter Harris <pharris@opentext.com> Signed-off-by: Julien Cristau <jcristau@debian.org>
* Add NEWS entries for release 1.9Uli Schlachter2013-12-141-0/+52
| | | | | Signed-off-by: Uli Schlachter <psychon@znc.in> Signed-off-by: Julien Cristau <jcristau@debian.org>
* Rename ge events to GeGeneric eventsUli Schlachter2013-12-141-1/+1
| | | | | | | | | | | The new xcb_ge_event_t that libxcb generates from this definition has different fields than the old, hand-written xcb_ge_event_t. To undo an API break, libxcb will reintroduce the old hand-written version and mark it as deprecated, so that everyone can switch to this new version. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=71502 Signed-off-by: Uli Schlachter <psychon@znc.in> Signed-off-by: Julien Cristau <jcristau@debian.org>
* sync: VALUETYPE and TESTTYPE are enumPeter Harris2013-11-181-4/+4
| | | | | | | Typoed as mask in e6a246e50e62cbcba33d0e1d2371e69e6e089383 Signed-off-by: Peter Harris <pharris@opentext.com> Reviewed-By: Daniel Martin <consume.noise@gmail.com>
* sync: Add missing namespace for the INT64 structPeter Harris2013-11-181-4/+4
| | | | | Signed-off-by: Peter Harris <pharris@opentext.com> Reviewed-By: Daniel Martin <consume.noise@gmail.com>
* Present: Remove reference to nonexistent enumPeter Harris2013-11-181-1/+1
| | | | | Signed-off-by: Peter Harris <pharris@opentext.com> Reviewed-by: Uli Schlachter <psychon@znc.in>
* xkb: Add struct Property as per spec (commented out)Ran Benita2013-11-141-0/+8
| | | | | | | | | | The specification says that Property is a struct of two CountedString16 (name and value). This form can't be handled atm. correctly too. But, add it as a comment so we've it at hand when we know what to do with it. All credits go to Ran Benita, as he found this. Signed-off-by: Daniel Martin <consume.noise@gmail.com>
* xkb: Comment out intermixed fixed size fields and listsDaniel Martin2013-11-141-0/+4
| | | | | | | | | | | | | | c_client.py doesn't handle such intermixed fixed size fields and lists correctly. Therefor comment them out. With that change the size of the generated request structure for ListComponents has 8 Bytes which is the same as xkbListComponentsReq has and the server checks REQUEST_AT_LEAST_SIZE(xkbListComponentsReq). The same goes for GetKbdByName, which results in a structure of 12 Bytes. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-By: Ran Benita <ran234@gmail.com>
* xkb: Comment out the struct Property and {Get, Set}GeometryDaniel Martin2013-11-141-4/+8
| | | | | | | | | | | | | This struct has intermixed fixed size fields and lists, which c_client.py doesn't handle properly. Therefor comment out the struct and the fields type-referencing it. As doodads are commented out already and with Property gone the requests GetGeomtry and SetGeometry become pretty much useless. Therefor comment them out completly. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-By: Ran Benita <ran234@gmail.com>
* xkb: Comment out DoodadsDaniel Martin2013-11-141-0/+8
| | | | | | | | | | | | | | Doodads are known to be broken (*). Comment them out until they can be handled properly. (*) One problem is that TextDoodad and LogoDoodad have variadic lists at the end. But, they're within a union with other fixed size Doodads. A union doesn't have a hint which field is in use and therefor the code generator (at least c_client.py) just creates a generic iterator for the Doodad union, which fails for TextDoodad and LogoDoodad. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-By: Ran Benita <ran234@gmail.com>
* Add note on pretty patches and src/.gitattributesDaniel Martin2013-11-142-0/+21
| | | | | | | | Add a file HACKING describing howto to prettify patches (their hunk headers) and add src/.gitattributes, which is necessary for that. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-By: Ran Benita <ran234@gmail.com>
* xv: Remove obsolete comments in EncodingInfo and QueryEncodingsDaniel Martin2013-11-141-8/+0
| | | | | | | | | | | | | | | The padding in EncodingInfo (struct) and QueryEncodings (reply) has been fixed upstream: Xvproto: fix xvEncodingInfo to match actual wire protocol http://cgit.freedesktop.org/xorg/proto/videoproto/commit/?id=d970512 and Xvproto.h: fix xvQueryEncodingsReply to match actual wire protocol http://cgit.freedesktop.org/xorg/proto/videoproto/commit/?id=1a96222 So, we can remove the comments. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Arnaud Fontaine <arnau@debian.org>
* xinput: Add XI2 event enum. for event selectionDaniel Martin2013-11-141-1/+33
| | | | | | | | This adds an enumeration 'XIEventMask', which is necessary to select for XI2 events via the SelectEvent request. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-By: Ran Benita <ran234@gmail.com>
* xcbgen: Remove tab from xtypes.py.Matt Turner2013-11-141-1/+1
| | | | Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=71418
* Update to version 1.9xcb-proto-1.9Keith Packard2013-11-071-1/+1
| | | | Signed-off-by: Keith Packard <keithp@keithp.com>
* Add 'fd' element to fields group, fix present Generic event to validateKeith Packard2013-11-072-1/+5
| | | | | | This fixes 'make check' Signed-off-by: Keith Packard <keithp@keithp.com>
* Add Present protocol specificationKeith Packard2013-11-072-0/+206
| | | | | Signed-off-by: Keith Packard <keithp@keithp.com> Reviewed-by: Peter Harris <pharris@opentext.com>
* Add DRI3Keith Packard2013-11-072-0/+98
| | | | | Signed-off-by: Keith Packard <keithp@keithp.com> Reviewed-by: Peter Harris <pharris@opentext.com>
* Add MIT-SHM AttachFd and CreateSegment requestsKeith Packard2013-11-071-1/+20
| | | | | Signed-off-by: Keith Packard <keithp@keithp.com> Reviewed-by: Peter Harris <pharris@opentext.com>
* Add support for file descriptor request fieldsKeith Packard2013-11-072-2/+21
| | | | | | | These are present in the API, but not present on the wire. Signed-off-by: Keith Packard <keithp@keithp.com> Reviewed-by: Adam Jackson <ajax@redhat.com>
* Define X generic event structureKeith Packard2013-11-061-1/+15
| | | | | | | | This is used by libxcb, and was defined explicitly there but it's cleaner to use the XML specification. Signed-off-by: Keith Packard <keithp@keithp.com> Reviewed-by: Peter Harris <pharris@opentext.com>
* Add CARD64/INT64 protocol typesKeith Packard2013-11-062-0/+4
| | | | | | | | No reason to make people use two 32-bit values when every modern compiler has native 64-bit objects. Signed-off-by: Keith Packard <keithp@keithp.com> Reviewed-by: Adam Jackson <ajax@redhat.com>
* Move the INT64 types in sync.xml to the sync: namespaceKeith Packard2013-11-061-12/+12
| | | | | | | | | This makes sure we use the crazy sync-specific 64-bit datatype instead of the global INT64 type Signed-off-by: Keith Packard <keithp@keithp.com> Reviewed-by: Peter Harris <pharris@opentext.com> Reviewed-by: Adam Jackson <ajax@redhat.com>
* XKB: Rewrite AXOptionPeter Harris2013-11-051-17/+9
| | | | | | | | AXOption can be an <enum> instead of a <union>, which is a much simpler construct. Signed-off-by: Peter Harris <pharris@opentext.com> Reviewed-By: Ran Benita <ran234@gmail.com>
* XKB: Fix values of AXFBOpt enumPeter Harris2013-11-051-4/+4
| | | | | Signed-off-by: Peter Harris <pharris@opentext.com> Reviewed-By: Ran Benita <ran234@gmail.com>
* xkb: Fix struct ExtensionDeviceNotifyDaniel Martin2013-10-111-1/+1
| | | | | | | The field ledId is of type CARD16, not CARD8, see XKBproto.h. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-By: Ran Benita <ran234@gmail.com>
* xkb: Pad structs CommonDoodad and SALockDeviceBtnDaniel Martin2013-10-111-0/+2
| | | | | | | | | | | | | | | | * CommonDoodad is xkbAnyDoodadWireDesc in XKBproto.h. There it has a size of 20 like all Doodads: #define sz_xkbAnyDoodadWireDesc 20 Add the necessary 8 bytes of padding. * SALockDeviceBtn is an Action. All Actions are handled by the server and libX11 with sizeof(xkbActionWireDesc) or SIZEOF(xkbActionWireDesc). Which results in a size of 8: #define sz_xkbActionWireDesc 8 /* in XKBproto.h */ Add the necessary 3 bytes of padding. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com>
* randr: Fix GetProviderProperty replyDaniel Martin2013-10-111-2/+1
| | | | | | | | The first field has to be 'format' not a pad and 'num_items' is of type CARD32. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Arnaud Fontaine <arnau@debian.org>
* Enforce a bit or value in enum itemsDaniel Martin2013-08-251-1/+4
| | | | | | | | | | Do not allow enum items without an explicit bit or value. The value of an enum item is known when adding the enum to the xml specification. There's no reason why this information shouldn't be where it belongs to. Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* xv: Use explicit enum valueDaniel Martin2013-08-251-17/+17
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* xproto: Use explicit enum valueDaniel Martin2013-08-251-68/+68
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* xinput: Use explicit enum valueDaniel Martin2013-08-251-56/+56
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* xfixes: Use explicit enum valueDaniel Martin2013-08-251-11/+11
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* xevie: Use explicit enum valueDaniel Martin2013-08-251-2/+2
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* sync: Use explicit enum valueDaniel Martin2013-08-251-9/+9
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* shape: Use explicit enum valueDaniel Martin2013-08-251-8/+8
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* screensaver: Use explicit enum valueDaniel Martin2013-08-251-7/+7
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* render: Use explicit enum valueDaniel Martin2013-08-251-70/+70
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>
* dri2: Use explicit enum valueDaniel Martin2013-08-251-13/+13
| | | | | | Signed-off-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Ran Benita <ran234@gmail.com> (and tested) Reviewed-by: Josh Triplett <josh@joshtriplett.org>