[cairo] Re: new clipping change patch

[Owen Taylor]

On Wed, 2005-05-25 at 13:57 +0100, Gustavo J. A. M. Carneiro wrote:

> > Various comments:
> > 
> >  * I'm not sure the name change cairo_surface_set_clip_region() =>
> >    cairo_surface_clip_region() makes sense. It's still a setter...
> >    there are no plans to make clip_region() intersect with the
> >    current clip region.
> 
>   Do you mean that if I clip to a certain rectangle, then save state and
> call a function to draw some object, the object is allowed to change the
> clip region and draw outside the initial rectangle?
> 
>   If so, in my opinion this is a terrible idea.  It's analogous to
> letting a user gain root privileges.  Please, can we not allow this?
> Setting a new clip region should always intersect with the previous one.
> Most of the time the new clip region is a rectangle completely inside
> the old clip region, so this should not be a performance problem...

We're talking about backend here. Not about changing the frontend API,
where subsequent clips do intersect.

Note that the frontend API does, like PS, have cairo_reset_clip().

Regards,
							Owen

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.freedesktop.org/archives/cairo/attachments/20050525/ebc8fa48/attachment.pgp