<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Dec 11, 2015 at 4:15 AM, Oded Gabbay <span dir="ltr"><<a href="mailto:oded.gabbay@gmail.com" target="_blank">oded.gabbay@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Dec 10, 2015 at 11:19 PM, Bill Spitzak <<a href="mailto:spitzak@gmail.com">spitzak@gmail.com</a>> wrote:<br>
> Can you include my patches to fix the filtering? They have been posted for a<br>
> long time now.<br>
><br>
> The last patch makes GOOD/BEST use filtering for scaling images down. This<br>
> matches the current Cairo behavior and would allow Cairo to use the pixman<br>
> backend rather than doing an image fallback for any image scaling smaller<br>
> than .75. It also contains a bunch of minor optimizaion and filter selection<br>
> tweaks that makes the output somewhat better than current Cairo.<br>
><br>
</span>Hi Bill,<br>
<br>
Unfortunately, I don't see anyone reviewed your patches, and from what<br>
I heard, those are quite significant changes.<br>
<br>
It's a shame you didn't bring this up when I did the first development<br>
release 4 months ago. Then we had enough time to check and test it.<br>
I'm quite hesitant of including such changes right before the final<br>
development version, even with a review.<br></blockquote><div><br></div><div>I did send email on May 22, 2015, in response to your comments. </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I suggest that you try to contact one of pixman's veterans (Soren,<br>
Siarhei, Matt, Pekka, Ben) offline and ask them nicely to at least<br>
skim over the patches and give a high-level opinion about the series.<br></blockquote><div><br></div><div>These were discussed with Soren before. He disagreed with my previous version because I changed to a single filter calculation rather than his pair of filters being convoluted. This version preserves the pair of filters, with some fixes of bugs that caused artifacts in the resulting filters. I'm sending email directly in case they are not reading the pixman list.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Also, check if you need to rebase the patches against current pixman<br>
and if so, maybe send the series again. It might stir up a discussion.<br></blockquote><div><br></div><div>The patches applied to the newest version without any conflicts and my test programs still work. I have resent them to the pixman mailing list. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'm willing to review them in terms of correctness and code style, but<br>
I'm not veteran enough in pixman to give an opinion on the underlying<br>
changes (which is the most important issue).<br></blockquote><div><br></div><div>Anything would be great.</div><div><br></div><div>I believe these work well and have been using them for a while. This would allow the removal of redundant code in Cairo, and would allow 2-pass filtering to be done at some point in the future, which would really improve pixman performance.</div><div><br></div></div></div></div>