[cairo] GSoC: Scan converting rasteriser update
Jeff Muizelaar
jeff at infidigm.net
Tue Nov 4 13:13:35 PST 2008
On Wed, Oct 29, 2008 at 06:43:04PM +0200, M Joonas Pihlaja wrote:
> I hope the new spans-pruning branch proves easier to review now
> that there's less irrelevant clutter in there.
Here's a review of the path-fixed changes:
[path-fixed] Avoid extra indirection when iterating already flat paths.
- looks good
[path-fixed] New _cairo_path_fixed_is_region().
- could test path has curve_to and fail immediately on that?
[path-fixed] New _cairo_path_fixed_bounds_approximate_box().
- perhaps give the reason for wanting bounds_approximate_box
- _cairo_path_fixed_bounds_approximate_box should have some
documentation about what it computes. i.e. what the approximation is
- it looks like _cairo_path_fixed_interpret can't fail, so we technically
don't need to deal with the failure case. Perhaps we should actually
assert on failure...
-Jeff
More information about the cairo
mailing list