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

Peter Harris pharris at opentext.com
Wed May 26 07:10:49 PDT 2010


On 2010-05-25 23:58, Keith Packard wrote:
> On Tue, 25 May 2010 16:57:05 -0400, Peter Harris <pharris at opentext.com> wrote:
> 
>> Can I talk you into renaming "miRectIn" to something else?
> 
> Do you have a suggestion? RegionRectIn seems a bit awkward to me.

Oooo. An invitation to bikeshed.

s/mi/dix/g instead of s/mi//g? Probably overkill for a single collision.

Alternatively, Soeren suggested RegionContainsRect upthread. (I can also
see RegionOverlapsRect, as the function returns partial overlap results
too.)

Dark Green.

I like the C language, but it could sure use a better namespace mechanism.

Peter Harris
-- 
               Open Text Connectivity Solutions Group
Peter Harris                    http://connectivity.opentext.com/
Research and Development        Phone: +1 905 762 6001
pharris at opentext.com            Toll Free: 1 877 359 4866


More information about the xorg-devel mailing list