[cairo] cairo_get_clip_as_rectangles

[Kristian Høgsberg]

On 6/28/06, Carl Worth <cworth at cworth.org> wrote:
> On Wed, 28 Jun 2006 17:06:57 +1200, Robert O'Callahan wrote:
> >
> > Attached is a patch for my next attempt at getting the clip as a
> > rectangle list.
...
> > + *            If the output *is_tight is set to true, then the union is exactly
> > + * the clip area. (If *is_tight is false, the union may or may not be exactly
> > + * the clip area.)
>
> The wording here is a bit awkward as it might suggest that the user
> needs to set is_tight to TRUE or FALSE before calling the function,
> (perhaps as a hint for what they want). Maybe something like:
>
>         The @is_tight parameter is used as a return value to indicate
>         whether the returned rectangles are a tight bound on the clip
>         region or not. A tight bound does not include any pixels not
>         in the clip region. It is valid to pass a NULL pointer if the
>         caller is not interested in the tightness.

Shouldn't this be even stricter: "does not include any pixels not
completely covered by the clip region".  Or maybe drop the double
negative: "A thight bound only includes pixels completely covered by
the clip region".

cheers,
Kristian