Re: [PATCH] Fix fixup_unbounded_boxes() to work even if the box is smaller than the extents

Chris Wilson <[email protected]>
Newsgroups gmane.comp.lib.cairo
Message-ID <[email protected]>
On Wed, 17 Aug 2011 17:14:24 -0400, Jeff Muizelaar <[email protected]> wrote:
> This patch fixes the problem we ran into here:
> https://bugzilla.mozilla.org/show_bug.cgi?id=668921
> 
> Currently fixup_unbounded_boxes assumes that the extents are tight when 
> we only have one box. This is not always true. e.g. when we tesselate boxes
> we can end up with only one box which is smaller than the extents and 
> the extents are not updated.
> 
> Does this look like the correct fix?

No, that only helps with the immediate bug, but the situation can still
arise if the tessellator reduces many boxes down to 2+, which due to
some fluke of winding or malice is smaller than the initial extents.

The approach taken on other paths is to correct the extents once we have
an accurate bounds.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
--
cairo mailing list
[email protected]
http://lists.cairographics.org/mailman/listinfo/cairo
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.