[PATCH 5/7] damage: Micro-tune the ReportNonEmpty case

Chris Wilson chris at chris-wilson.co.uk
Tue Sep 3 10:54:26 PDT 2013


On Mon, Sep 02, 2013 at 10:19:00AM -0400, Adam Jackson wrote:
> gcc doesn't seem to be smart enough to short-circuit the RegionUnion
> call when the region was previously empty.  Possibly this was partly
> that the second argument to RegionUnion() isn't const.
> 
> Signed-off-by: Adam Jackson <ajax at redhat.com>

After this patch, we only record the first damage event, and not the
cumulative pDamage->damage, which would seem to break the API.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the xorg-devel mailing list