[PATCH] Make RegionInit() and RegionCreate() take just a box and no size

Daniel Stone daniel at fooishbar.org
Tue May 3 04:44:47 PDT 2011


Hi,

On Mon, May 02, 2011 at 07:44:15AM -0400, Søren Sandmann wrote:
> The interface to these function was was very confusing since it gave
> the impression that they initialized the region from a list of boxes,
> which they didn't.
> 
> This patch changes the interface to take just one box and fixes all
> the callers accordingly.

Why not just change all RegionInit(region, box, 1) calls to use
RegionInitBoxes(region, box, 1), and add a RegionCreateBoxes() or
similar?

Cheers,
Daniel


More information about the xorg-devel mailing list