Re: [PATCH] image: Add public API for the creation of a surface for a pixman_image_t

On Tue, 20 Mar 2012 17:34:10 +0100, [email protected] (=?utf-8?Q?S=C3=B8ren?= Sandmann) wrote:

Thanks Søren for your feedback.

> If the main reason for this API is to help people with weird hardware,
> maybe cairo_format_t could simply be extended to cover all known pixman
> formats?

The goal was to try and advertise only a few formats that are well
supported and well tested, mainly to prevent the reduce the complexity
of implementation. However, all that really means is that instead of
checking for CAIRO_FORMAT_INVALID and performing a conversion to a known
format, the backend checks against a list of formats it can handle. So
the primary point has to be, do we want people to feed all-and-sundry
formats into cairo? The goal there has always been to guide people to
choose formats that will work well anywhere.

> Another possibility might be to add a new "pixman" surface type that was
> explicitly documented to be less ABI stable than cairo itself.

That's an interesting suggestion and neatly works around a couple of
issues that the patch raised.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre

--
cairo mailing list
[email protected]
http://lists.cairographics.org/mailman/listinfo/cairo