[PATCH wayland-protocols, weston v3 0/17] Stabilize wl_scaler as wp_viewporter

[Yong Bakos]

On Apr 26, 2016, at 7:50 AM, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> 
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> Hi,
> 
> here is the v3 of the stabilization series, which I hope to the final to be
> merged.
> 
> The wayland-protocols patches are essentially the same as v2, except few
> cosmetic changes as suggested by review comments. There are mentioned in the
> review log in each patch, if there were any.
> 
> The weston patches are v3 too, though v2 was never posted. The changes to v1
> are:
> - patches 6-7 are new (simple preparation)
> - patches 8-12 are the same as v1
> - patches 13-17 are new, accounting for the spec changes from
>  wayland-protocols v1
> 
> Note, that there are now tests for the wp_viewporter API. \o/
> 
> 
> Wayland-protocols:
> 
> Pekka Paalanen (5):
>  stable: add viewporter draft
>  stable/viewporter: remove wp_viewport.set request
>  stable/viewporter: rename and build
>  stable/viewporter: rephrase a wp_viewport paragraph
>  stable/viewporter: add more error cases
> 
> Makefile.am                      |   1 +
> stable/viewporter/README         |   7 ++
> stable/viewporter/viewporter.xml | 189 +++++++++++++++++++++++++++++++++++++++
> 3 files changed, 197 insertions(+)
> create mode 100644 stable/viewporter/README
> create mode 100644 stable/viewporter/viewporter.xml

1-5 look good and v3 is Reviewed-by Yong Bakos <ybakos at humanoriented.com>.

yong



> 
> Weston:
> 
> Pekka Paalanen (12):
>  compositor: constify weston_surface_build_buffer_matrix()
>  compositor: refactor into convert_size_by_transform_scale()
>  compositor: migrate to stable viewporter.xml
>  compositor: rename scaler to viewport(er)
>  clients/scaler: migrate to wp_viewporter
>  clients/simple-damage: migrate to wp_viewporter
>  protocol: remove scaler.xml
>  compositor: fix wp_viewport use after free
>  compositor: check viewport source rect validity
>  compositor: check viewport dst size validity
>  compositor: fix wp_viewport.set_source errors
>  tests: add wp_viewporter tests
> 
> Makefile.am             |  27 ++-
> clients/scaler.c        |  43 ++--
> clients/simple-damage.c |  32 +--
> configure.ac            |   2 +
> protocol/scaler.xml     | 208 ------------------
> src/compositor.c        | 343 +++++++++++++++++++-----------
> src/compositor.h        |   5 +-
> tests/viewporter-test.c | 553 ++++++++++++++++++++++++++++++++++++++++++++++++
> 8 files changed, 823 insertions(+), 390 deletions(-)
> delete mode 100644 protocol/scaler.xml
> create mode 100644 tests/viewporter-test.c
> 
> 
> Thanks,
> pq
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel