<div>agree with you.</div>
<div>I saw the patch you committed:</div>
<div>185a3518968d5ca0babb798fe10f247171a1e001</div>
<div>Just a slight modification is needed, I think:</div>
<div>The dst_x and dst_y should be considered when</div>
<div>trimming the mask.<br><br></div>
<div class="gmail_quote">2012/6/20 Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk" target="_blank">chris@chris-wilson.co.uk</a>></span><br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div class="im">On Wed, 20 Jun 2012 04:04:43 +0800, Chuanbo Weng <<a href="mailto:strgnm@gmail.com">strgnm@gmail.com</a>> wrote:<br>> If the operator is unbounded and mask is not large enough, the area<br>> affected by _cairo_surface_mask() in render_glyphs_via_mask() will<br>
> be too larger, then result in output error. Fixes overlapping-glyphs<br>> and unbounded-operator test cases in cairo-test-suite.<br><br></div>Whoops, the error here is that we have begun to pass in a NULL clip to<br>
_cairo_surface_mask(), which as you can see has the effecting of<br>clearing everything outside of the masked region.<br><span class="HOEnZb"><font color="#888888">-Chris<br><br>--<br>Chris Wilson, Intel Open Source Technology Centre<br>
</font></span></blockquote></div><br>