[cairo] [PATCH] Support image filtering in xlib and xcb backends

Chris Wilson chris at chris-wilson.co.uk
Fri Aug 15 00:12:24 PDT 2014


On Thu, Aug 14, 2014 at 04:30:01PM -0700, Bryce Harrington wrote:
> On Tue, Aug 12, 2014 at 08:06:35PM -0700, Bill Spitzak wrote:
> > _cairo_pattern_analyze_filter() will change GOOD to BILINEAR if
> > it thinks bilinear is acceptable. Removed the equivalent test
> > from _pixman_image_set_properties.
> > 
> > The xlib and xcb backends will use the image fallback for GOOD
> > and BEST so that the convolution can be done.
> > 
> > Notes:
> > 
> > - I did not test the xcb changes.
> > 
> > - I am assuming other backends treat bilinear and good identically,
> > at least for scales larger than 1/1.35.
> > 
> > - A CAIRO_RENDER option maybe should be added in case pixman ever
> > supports GOOD/BEST.
> 
> Should the above notes be taken care of before this is merged?
> 
> Functionally this looks good, and I gather this is outcome from recent
> discussions with Chris and others.  I'd appreciate a third set of eyes
> on this just for sanity checking, but otherwise:
> 
> Reviewed-by: Bryce Harrington <b.harrington at samsung.com>

No. We have to make the fallback happen inside the drivers otherwise it
will never be fixed.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the cairo mailing list