[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