xfixes: cursor-name is after cursor-image in GetCursorImageAndName

Submitted by Peter Harris on Feb. 3, 2015, 11:31 p.m.


Message ID 1423006298-5427-1-git-send-email-pharris@opentext.com
State New
Headers show

Not browsing as part of any series.

Commit Message

Peter Harris Feb. 3, 2015, 11:31 p.m.
The current version of fixesproto.txt has cursor-name before
cursor-image, but all known implementations put cursor-name after
cursor-image on the wire.

Signed-off-by: Peter Harris <pharris@opentext.com>
 src/xfixes.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/src/xfixes.xml b/src/xfixes.xml
index c5cba35..0a3d5ff 100644
--- a/src/xfixes.xml
+++ b/src/xfixes.xml
@@ -291,13 +291,13 @@  authorization from the authors.
       <field type="ATOM"   name="cursor_atom" altenum="Atom" />
       <field type="CARD16" name="nbytes" />
       <pad bytes="2" />
-      <list  type="char"   name="name"><fieldref>nbytes</fieldref></list>
       <list  type="CARD32" name="cursor_image">
         <op op="*">
+      <list  type="char"   name="name"><fieldref>nbytes</fieldref></list>


On 2015-02-08 04:02, Christian Linhart wrote:
> Hi Peter,
> Thank you for finding and fixing that.
> I suggest that you post a patch for fixesproto.txt on xorg-devel,
> so that implementation and spec will be in sync.


> I have looked at the implementation in libXFixes and Xserver
> and this confirms what you have found out.
> Your patch looks good, therefore:
> Reviewed-by: Christian Linhart <chris@demorecorder.com>


Peter Harris