[PATCH weston v2 2/3] ivi-shell: introduce ivi_layout_view
Pekka Paalanen
ppaalanen at gmail.com
Wed Jun 29 08:38:39 UTC 2016
On Thu, 23 Jun 2016 02:06:24 +0000
"Natsume, Wataru (ADITJ/SWG)" <wnatsume at jp.adit-jv.com> wrote:
> Hello Emre-san,
>
> v2 patch looks fine with me.
> It passed review at my side except here. (So trivial.)
>
> +};
> + <--- It looks better to add blanc.
> struct ivi_layout_surface {
>
> Reviewed-by: Wataru Natsume <wnatsume at jp.adit-jv.com>
>
> Best Regards,
> Wataru Natsume
>
> ---
> Wataru Natsume / 夏目 亘
> Advanced Driver Information Technology
> Software Group (ADITJ/SWG)
> E-mail: wnatsume at jp.adit-jv.com
> Tel: +81-(0)566-61-4551 / Extension 551-43644
>
>
> -----Original Message-----
> From: wayland-devel [mailto:wayland-devel-bounces at lists.freedesktop.org] On Behalf Of Ucan, Emre (ADITG/SW1)
> Sent: Tuesday, June 14, 2016 11:44 PM
> To: wayland-devel at lists.freedesktop.org
> Subject: [PATCH weston v2 2/3] ivi-shell: introduce ivi_layout_view
>
> This patch introduces ivi_layout_view data struct, which is a wrapper of weston_view.
>
> There is always only one ivi_layout_view for an ivi_layout_surface and ivi_layout_layer pair.
> A surface could have many views with different geometry and transformations, so that a surface can be shown on:
> 1. On many screens
> 2. On the same screen with different positions
>
> The geometry of a view is modified, when properties of its layer are changed through ivi_layout_interface.
> Users of ivi_layout_interface does not have direct access to ivi_layout_view structure.
>
> v2 changes:
> 1. Use ivi_view_is_rendered function instead of
> active member
> 2. Add descriptions to introduced members of
> structs
>
> Signed-off-by: Emre Ucan <wbaeucan at de.adit-jv.com>
> Acked-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> ---
> ivi-shell/ivi-layout-private.h | 22 +++-
> ivi-shell/ivi-layout.c | 285 ++++++++++++++++++++++++----------------
> 2 files changed, 190 insertions(+), 117 deletions(-)
>
Hi,
I fixed the whitespace issue Natsume-san pointed out, and pushed:
28d240f..37d25bb master -> master
Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20160629/a97e0c7e/attachment.sig>
More information about the wayland-devel
mailing list