Adam Jackson <[email protected]> writes:

> When dix hands us a new cursor we proxy it through to the host server;
> since we keep the host XID on the cursor bits private we can switch
> among them with just ChangeWindowAttributes.

Sorry for replying about your previous version; somehow I missed this in
my queue. Only one small comment below...

> +    cursor_pxm = xcb_generate_id(HostX.conn);
> +    xcb_create_pixmap(HostX.conn, 1, cursor_pxm, HostX.winroot, 1, 1);
> +    HostX.empty_cursor = xcb_generate_id(HostX.conn);
> +    xcb_create_cursor(HostX.conn,
> +                      HostX.empty_cursor,
> +                      cursor_pxm, cursor_pxm,
> +                      0,0,0,
> +                      0,0,0,
> +                      1,1);
> +    xcb_free_pixmap(HostX.conn, cursor_pxm);

Sadly, pixmap contents are not defined on creation. So, you'll have to
actually clear this thing. Create a GC, set foreground to zero and fill
rectangle (or draw a point; it's all the same :-)

Otherwise, this is all

Reviewed-by: Keith Packard <[email protected]>

-- 
[email protected]

Attachment: pgpSww4_Wf4o1.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to