[PATCH 2/4] Rename region macros to mixed case and remove screen argument

Daniel Stone daniel at fooishbar.org
Wed May 26 16:25:23 PDT 2010


On Wed, May 26, 2010 at 02:57:18PM -0700, Keith Packard wrote:
> On Wed, 26 May 2010 16:54:44 +0100, Daniel Stone <daniel at fooishbar.org> wrote:
> > Can we just man up and use NULL? (Or if we don't, define NullRegion to
> > be NULL instead of 0.)
> 
> NullRegion has the advantage of providing both visual and compiler type
> checking in functions containing numerous arguments. I've found that
> useful at times.

Ah, fair enough.  Changing it to NULL rather than 0 (being as it's a
pointer rather than an integer) is still a goer though.

Cheers,
Daniel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100527/5b51cffd/attachment-0001.pgp>


More information about the xorg-devel mailing list