[cairo] GSoC: Scan converting rasteriser update
Jeff Muizelaar
jeff at infidigm.net
Fri Nov 14 14:17:13 PST 2008
On Thu, Nov 13, 2008 at 11:28:17PM -0500, Jeff Muizelaar wrote:
> Some more review. I didn't get as much time to look at the new code today as I
> would've liked, so this is all I have for now.
>
> [cairo-spans] Implement a span renderer for cairo_image_surface_t.
> - it might be good to separate out the _mask_init() and _mask_fini()
> changes into a separate patch. i.e. separate out the cosmetic
> changes from the behavioural changes.
>
> - the function declaration style of _cairo_image_surface_span_renderer_render_row()
> and _cairo_image_surface_span_renderer_destroy() don't match the rest of the file.
- the following line should not have a (void*) cast. It should cast to
(struct _cairo_image_surface_span_renderer)
struct _cairo_image_surface_span_renderer *self = (void*)super;
-Jeff
More information about the cairo
mailing list