<div dir="ltr"><div><div>Hey pq,<br></div>Glad to see this stuff finally look like landing ~1 year later. :-)  My reviewing ability on this code has decreased substantially since I wrote it so I'm kind of trusting you to have tested it thuroughly and trusting myself to have known what I was doing back then.  That said,<br><br></div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 10, 2015 at 7:48 AM, Derek Foreman <span dir="ltr"><<a href="mailto:derekf@osg.samsung.com" target="_blank">derekf@osg.samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 10/03/15 08:01 AM, Pekka Paalanen wrote:<br>
> From: Pekka Paalanen <<a href="mailto:pekka.paalanen@collabora.co.uk">pekka.paalanen@collabora.co.uk</a>><br>
><br>
> Hi,<br>
><br>
> as requested by Derek, here is a revised series from<br>
> <a href="http://lists.freedesktop.org/archives/wayland-devel/2015-March/020440.html" target="_blank">http://lists.freedesktop.org/archives/wayland-devel/2015-March/020440.html</a><br>
><br>
> The major addition to the series is that I pulled selected patches from<br>
> <a href="http://lists.freedesktop.org/archives/wayland-devel/2014-October/017801.html" target="_blank">http://lists.freedesktop.org/archives/wayland-devel/2014-October/017801.html</a><br>
> recently rebased by Derek in<br>
> <a href="https://github.com/ManMower/weston/commits/transforms" target="_blank">https://github.com/ManMower/weston/commits/transforms</a><br>
> to the beginning of my series, so that pixman-renderer would have less<br>
> code to move around.<br>
><br>
> While doing that, I added the zoom cleanups to the very beginning, and fixed<br>
> some fallout from the output matrix changes.<br>
><br>
> Jason, for all patches where you are the author, I'd like to get your S-o-b and/or<br>
> R-b. I did do some minor modifications, those are noted in the commit messages.<br>
><br>
> Derek, I'd appreciate R-b's, and S-o-b for the patch you authored in this<br>
> series. I have addressed your review comments from the last time, so some<br>
> patches already have your R-b, but wouldn't hurt to have a quick look again.<br>
><br>
> The series is also available in<br>
> <a href="http://cgit.collabora.com/git/user/pq/weston.git/log/?h=pixmancomplex-3" target="_blank">http://cgit.collabora.com/git/user/pq/weston.git/log/?h=pixmancomplex-3</a><br>
><br>
><br>
> Thanks,<br>
> pq<br>
><br>
> Derek Foreman (1):<br>
>   compositor: use weston_matrix_transform for<br>
>     weston_output_transform_coordinate<br>
<br>
</div></div>S-O-B me.<br>
<span class=""><br>
><br>
> Jason Ekstrand (4):<br>
>   Use pixel coordinates for weston_output.matrix<br>
>   zoom: Use pixels instead of GL coordinates<br>
>   compositor: Add surface-to-buffer and buffer-to-surface matrices<br>
>   pixman-renderer: simplify the output-to-buffer matrix computation<br>
<br>
</span>Since we're not allowing zoom at all, I guess we could continue to<br>
disallow zoom across this patch instead of rendering funny stuff.<br>
<br>
Personally, I'm not too concerned about cosmetic brokenness here since<br>
it's about to be fixed and didn't work previously. :)<br>
<span class=""><br>
><br>
> Pekka Paalanen (11):<br>
>   zoom: remove animation_xy as unused<br>
>   zoom: remove unused args from weston_zoom_transition<br>
>   rpi-renderer: minimal fix to zoom coordinates<br>
<br>
</span>Patch looks good, but I have no rpi to test.<br>
<br>
All patches Reviewed-By: Derek Foreman <<a href="mailto:derekf@osg.samsung.com">derekf@osg.samsung.com</a>><br>
<br>
No further comments. :)<br>
<div class="HOEnZb"><div class="h5"><br>
>   compositor: add weston_surface_to_buffer_region()<br>
>   pixman-renderer: refactor transformation computation<br>
>   pixman-renderer: refactor into region_intersect_only_translation()<br>
>   pixman-renderer: add view_transformation_is_translation()<br>
>   pixman-renderer: change repaint_region() arguments<br>
>   pixman-renderer: move code to draw_view_translated()<br>
>   pixman-renderer: implement source clipping<br>
>   pixman-renderer: implement view scissor<br>
><br>
>  src/compositor.c      | 292 ++++++++++++++++++-------------<br>
>  src/compositor.h      |  18 +-<br>
>  src/gl-renderer.c     |  14 +-<br>
>  src/pixman-renderer.c | 475 +++++++++++++++++++++++++++++---------------------<br>
>  src/rpi-renderer.c    |  10 +-<br>
>  src/zoom.c            |  98 +++--------<br>
>  6 files changed, 493 insertions(+), 414 deletions(-)<br>
><br>
<br>
</div></div></blockquote></div><br></div></div></div></div></div>