[PATCH] configure: set X render animated cursors optional
Keith Packard
keithp at keithp.com
Tue Sep 1 09:11:34 PDT 2009
On Tue, 2009-09-01 at 13:04 +0300, Vignatti Tiago (Nokia-D/Helsinki)
wrote:
> > > My only doubt is regarding the return type on ProcRenderCreateAnimCursor when
> > > the server does not support animated cursor. I put as BadImplementation.
> >
> > Doing so means that the server intentionally has an incomplete/wrong
> > implementation of render. Not sure if that's a good idea.
>
> This seems not to be a big deal. We already have something like this on the
> server. See for instance ProcXFixesCreateRegionFromPicture().
Render isn't really optional at this point.
> But my concern is another one now - we can save 16KB just removing this
> animated cursor code. Yeah, I've been non-verbose about this and you thought
> we're treating another issue. Sorry.
16KB of text space? Is that really an issue?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.x.org/archives/xorg-devel/attachments/20090901/2d9cf0a3/attachment.pgp
More information about the xorg-devel
mailing list