<div dir="ltr">On Tue, Feb 16, 2016 at 4:45 PM, Bill Spitzak <span dir="ltr"><<a href="mailto:spitzak@gmail.com" target="_blank">spitzak@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I have a new version of this patch, which fixes a math error that<br>
caused it to produce too many samples at small sizes (large scales). I<br>
am not clear if I can submit a v14 of just this patch or I should<br>
instead submit a v14 of the entire patch series.<br></blockquote><div><br></div><div>I don't think this functionality belongs in pixman at all. Computing the number of subsamples is essentially a performance/quality tradeoff, which is something that cairo and other clients need to make for themselves. In general, pixman is not the place for heuristics; its API should be predictable and explicit.<br><br>So, NACK from me on this patch.<br></div><div><br><br></div><div>Søren<br></div><br></div></div></div>