<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 11, 2016 at 12:35 PM, Søren Sandmann <span dir="ltr"><<a href="mailto:soren.sandmann@gmail.com" target="_blank">soren.sandmann@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="">On Mon, Apr 11, 2016 at 2:43 PM, Bill Spitzak <span dir="ltr"><<a href="mailto:spitzak@gmail.com" target="_blank">spitzak@gmail.com</a>></span> wrote:<br> <br></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>I feel this can be fixed. It is already correct for subsample_bits==0. Since both the filter generator and filtering code would be changed in the same version, only programs that generate their own filters would actually be incompatible. And as you point out the error is very tiny for large numbers of subsamples, and Cairo is already using an excessively large number of subsamples (likely because I was trying to remove the difference between identity filters and nearest filtering, and did not realize this was the underlying problem).</div></div></blockquote><div><br></div></span><div class="gmail_extra">It is technically an ABI break, and cairo 1.14 did ship with a copied-and-pasted filter generator that assumes the current subpixel positioning.<br><br>But yeah, maybe we can just ignore that, if we make sure a there is a cairo 1.14.x update that uses pixman_filter_create_separate_convolution() instead of the copied-and-pasted filter generator.<br><br>Other than that, the fix should be straight-forward enough.</div></div></div></div></blockquote><div><br></div><div>As I wrote that Cairo patch I can state that I think it would be perfectly fine to make this change. The subsamples are set unnaturally high in that patch and hide the problem. Also it is possible my filter generator is producing centered samples so this could be an improvement, need to check. In addition, other than the (abandoned) idea of getting good/best into pixman, this patch series is also designed so that Cairo can use the pixman filter generator rather than it's own.</div><div><br></div></div><br></div></div>