<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> <html> <head> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body text="#000000" bgcolor="#ffffff"> We can avoid using stencil test for cases where drawing twice will produce same result. For example, opaque solid source with op OVER. Clearing entire stencil buffer for every stroke might be quite expensive for most of drivers and GPUs. It would be good to restrict clearing area within bounded rectangle. And have you measured the effect of glColorMask() in clipping code? AFAIK, it was extremely expensive. (But I didn't measure it with recent GPUs and drivers.) Here's my suggestion. if (! used_stencil_buffer_for_clip) {     glStencilFunc (GL_NEVER, ...); // Nothing will be drawn to the color buffer     glStencilOp (GL_ONE, GL_ONE, GL_ONE);     drawBoundedRectangle (); // Stroke extent } One last thing is that you consumes stencil buffer to prevent overlap thus it would be hard to cache stencil buffer for a clip. On 10/17/2011 11:56 PM, Henry (Yu) Song - SISA wrote: <blockquote cite="mid:3955FA337689574EB32F94B12A7E6E9E0B18C84B@ex10mail" type="cite"> <meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1"> <meta name="Generator" content="Microsoft Word 12 (filtered medium)"> <style></style> <div class="WordSection1"> When stroking we do not send our polygon to the tessellator, so<o:p></o:p> it may have overlapping stroke components. Use the stencil buffer<o:p></o:p> to prevent stroke components from overlapping.<o:p></o:p> ---<o:p></o:p> src/cairo-gl-msaa-compositor.c |   26 ++++++++++++++++++++++++++<o:p></o:p> 1 files changed, 26 insertions(+), 0 deletions(-)<o:p></o:p> <o:p> </o:p> diff --git a/src/cairo-gl-msaa-compositor.c b/src/cairo-gl-msaa-compositor.c<o:p></o:p> index 7a0e828..01ef40f 100644<o:p></o:p> --- a/src/cairo-gl-msaa-compositor.c<o:p></o:p> +++ b/src/cairo-gl-msaa-compositor.c<o:p></o:p> @@ -247,6 +247,27 @@ _stroke_shaper_add_quad (void                *closure,<o:p></o:p>                                          quad);<o:p></o:p> }<o:p></o:p> <o:p></o:p> +static void<o:p></o:p> +_prevent_overlapping_drawing (cairo_clip_t     *clip,<o:p></o:p> +                       cairo_bool_t      used_stencil_buffer_for_clip)<o:p></o:p> +{<o:p></o:p> +   if (used_stencil_buffer_for_clip == FALSE) {<o:p></o:p> +     /* Enable the stencil buffer, even if we have no clip so that<o:p></o:p> +        we can use it below to prevent overlapping shapes. We initialize<o:p></o:p> +        it all to one here which represents infinite clip. */<o:p></o:p> +     glDepthMask (GL_TRUE);<o:p></o:p> +     glEnable (GL_STENCIL_TEST);<o:p></o:p> +     glClearStencil(1);<o:p></o:p> +     glClear (GL_STENCIL_BUFFER_BIT);<o:p></o:p> +     glStencilFunc (GL_EQUAL, 1, 1);<o:p></o:p> +    }<o:p></o:p> +<o:p></o:p> +    /* This means that once we draw to a particular pixel nothing else can<o:p></o:p> +       be drawn there until the stencil buffer is reset or the stencil test<o:p></o:p> +       is disabled. */<o:p></o:p> +    glStencilOp (GL_ZERO, GL_ZERO, GL_ZERO);<o:p></o:p> +}<o:p></o:p> +<o:p></o:p> static cairo_int_status_t<o:p></o:p> _cairo_gl_msaa_compositor_stroke (const cairo_compositor_t *compositor,<o:p></o:p>                          cairo_composite_rectangles_t      *composite,<o:p></o:p> @@ -261,6 +282,8 @@ _cairo_gl_msaa_compositor_stroke (const cairo_compositor_t      *compositor,<o:p></o:p>      cairo_gl_surface_t *dst = (cairo_gl_surface_t *) composite->surface;<o:p></o:p>      struct _tristrip_composite_info info;<o:p></o:p> <o:p></o:p> +    cairo_bool_t used_stencil_buffer_for_clip = FALSE;<o:p></o:p> +<o:p></o:p>      if (antialias != CAIRO_ANTIALIAS_NONE)<o:p></o:p>      return CAIRO_INT_STATUS_UNSUPPORTED;<o:p></o:p> <o:p></o:p> @@ -296,11 +319,14 @@ _cairo_gl_msaa_compositor_stroke (const cairo_compositor_t      *compositor,<o:p></o:p>      if (! _cairo_composite_rectangles_can_reduce_clip (composite,<o:p></o:p>                                           composite->clip))<o:p></o:p>      {<o:p></o:p> +     used_stencil_buffer_for_clip = TRUE;<o:p></o:p>      status = _draw_clip_to_stencil_buffer (info.ctx, &info.setup, composite->clip);<o:p></o:p>      if (unlikely (status))<o:p></o:p>          goto finish;<o:p></o:p>      }<o:p></o:p> <o:p></o:p> +    _prevent_overlapping_drawing (composite->clip, used_stencil_buffer_for_clip);<o:p></o:p> +<o:p></o:p>      status = _cairo_path_fixed_stroke_to_shaper ((cairo_path_fixed_t *) path,<o:p></o:p>                                     style,<o:p></o:p>                                     ctm,<o:p></o:p> -- <o:p></o:p> 1.7.4.1<o:p></o:p> <o:p> </o:p> --<o:p></o:p> cairo mailing list<o:p></o:p> <a moz-do-not-send="true" href="mailto:cairo@cairographics.org">cairo@cairographics.org</a><o:p></o:p> <a moz-do-not-send="true" href="http://lists.cairographics.org/mailman/listinfo/cairo">http://lists.cairographics.org/mailman/listinfo/cairo</a><o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> </div> <pre wrap=""> <fieldset class="mimeAttachmentHeader"></fieldset> -- cairo mailing list <a class="moz-txt-link-abbreviated" href="mailto:cairo@cairographics.org">cairo@cairographics.org</a> <a class="moz-txt-link-freetext" href="http://lists.cairographics.org/mailman/listinfo/cairo">http://lists.cairographics.org/mailman/listinfo/cairo</a></pre> </blockquote> </body> </html>