[PATCH 1/2] Atomic query requests

Keith Packard keithp at keithp.com
Fri Dec 2 02:33:01 PST 2011


On Thu,  1 Dec 2011 15:53:40 -0500, Adam Jackson <ajax at redhat.com> wrote:

> +	If the pointer argument is None, information for the core pointer is
> +	returned.  Otherwise, if the server supports the X Input Extension
> +	version 2 or greater, the pointer argument may name any pointer
> +	device.

Should this allow for keyboard devices as well and just use the
associated pointer? There are lots of other places in the server that
allow any device and find the right associated keyboard/pointer.


> +typedef struct {
> +    Window  window B32;
> +    CARD16  x B16;
> +    CARD16  y B16;
> +} xXFixesPointerWindow;

I bet you'll need a #define sz_ here

> +typedef struct {
> +    BYTE    type;   /* X_Reply */
> +    BYTE    pad1;
> +    CARD16  sequenceNumber B16;
> +    CARD32  length B32;
> +    CARD16  mask B16;
> +    CARD16  pad2 B16;
> +    CARD32  numWindows B32;
> +    CARD32  pad3 B32;
> +    CARD32  pad4 B32;
> +    CARD32  pad5 B32;
> +    CARD32  pad6 B32;
> +} xXFixesQueryPointerReply;
> +
> +#define sz_xXFixesQueryPointerReply 32

numWindows is not needed here -- you can compute it from length.

> +typedef struct {
> +    CARD8   reqType;
> +    CARD8   xfixesReqType;
> +    CARD16  length B16;
> +    Window  window B32;
> +} xXFixesQueryAncestorsReq;
> +
> +#define sz_xXFixesQueryAncestorsReq sizeof(xXFixesQueryAncestorsReq)
> +
> +typedef struct {
> +    BYTE    type;   /* X_Reply */
> +    BYTE    pad1;
> +    CARD16  sequenceNumber B16;
> +    CARD32  length B32;
> +    CARD32  numWindows B32;
> +    CARD32  pad3 B32;
> +    CARD32  pad4 B32;
> +    CARD32  pad5 B32;
> +    CARD32  pad6 B32;
> +    CARD32  pad7 B32;
> +} xXFixesQueryAncestorsReply;
> +
> +#define sz_xXFixesQueryAncestorsReply 32

numWindows is not needed here, it can be computed from length.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20111202/6293d702/attachment.pgp>


More information about the xorg-devel mailing list