[cairo] [PATCH] Fix _cairo_region_get_boxes usage

Vladimir Vukicevic vladimir at pobox.com
Mon Feb 23 17:22:09 PST 2009


On 2/15/09 4:17 PM, Chris Wilson wrote:
> On Mon, 2009-02-16 at 01:03 +0100, Soeren Sandmann wrote:
>> Here is a patch to do that. It's a net reduction of 56 lines, and most
>> callers are much simpler.
>
> Thank you Soren, that looks a vast improvement over the current
> get_boxes().

I'm a little late to this party, but this replaces a quick loop with a 
function call per iteration; it probably won't matter assuming that the 
number of rectangles is low, but internally I'd rather see us optimize 
for speed even at the cost of added complexity.  If we don't, we risk 
death-by-a-thousand-cuts style performance degradation...


    - Vlad


More information about the cairo mailing list