Hello Bryce,

As far as I know, there are two controller plugins which are using ivi layout 
interface.
        1. HMI controller
        2. IVI controller in Genivi Wayland IVI Extension

I updated the hmi controller for these changes, and ivi controller does not use 
these APIs.

Furthermore, IVI Layout APIs are internal. It is quite often that weston 
plugins gets break after a major release because of either API changes or data 
struct changes.

For example, we have an input plugin in Wayland IVI Extension which replaces 
the default grab interfaces. The plugin does not compile with 1.10 weston 
because weston_pointer data struct is changed after 1.9.

Best regards

Emre Ucan
Software Group I (ADITG/SW1)

Tel. +49 5121 49 6937
> -----Original Message-----
> From: Bryce Harrington [mailto:[email protected]]
> Sent: Freitag, 26. Februar 2016 19:03
> To: Ucan, Emre (ADITG/SW1)
> Cc: [email protected]
> Subject: Re: [PATCH weston 00/14] IVI Layout API Cleanup
> 
> On Fri, Feb 26, 2016 at 03:57:56PM +0000, Ucan, Emre (ADITG/SW1) wrote:
> > I removed the get APIs, because the same information can be get from
> > ivi_layout_get_properties_of_surface/layer APIs. Therefore, these APIs
> > are redundant.
> 
> Looks like a good cleanup, but do we have any concerns about API stability in
> dropping these getter/setters?
> 
> Bryce
> 
> > Furthermore, I removed *_set_position/dimension APIs, because position
> > and dimension can be set by
> > ivi_layout_surface/layer_set_destination_rectangle APIs.
> >
> > I adjusted ivi-layout-transition.c, ivi shell test code and
> > hmi-controller.c for these changes.
> >
> > Emre Ucan (14):
> >   ivi-shell: remove ivi_layout_surface_get_visibility API
> >   ivi-shell: remove ivi_layout_layer_get_visibility API
> >   ivi-shell: remove ivi_layout_surface_get_opacity API
> >   ivi-shell: remove ivi_layout_layer_get_opacity API
> >   ivi-shell: remove ivi_layout_surface_get_position API
> >   ivi-shell: remove ivi_layout_layer_get_position API
> >   ivi-shell: remove ivi_layout_surface_get_dimension API
> >   ivi-shell: remove ivi_layout_layer_get_dimension API
> >   ivi-shell: remove ivi_layout_surface_get_orientation API
> >   ivi-shell: remove ivi_layout_layer_get_orientation API
> >   ivi-shell: remove ivi_layout_surface_set_position API
> >   ivi-shell: remove ivi_layout_layer_set_position API
> >   ivi-shell: remove ivi_layout_surface_set_dimension API
> >   ivi-shell: remove ivi_layout_layer_set_dimension API
> >
> >  ivi-shell/hmi-controller.c        |   17 ++-
> >  ivi-shell/ivi-layout-export.h     |  127 --------------------
> >  ivi-shell/ivi-layout-private.h    |   17 +--
> >  ivi-shell/ivi-layout-transition.c |   19 ++-
> >  ivi-shell/ivi-layout.c            |  237 
> > +------------------------------------
> >  ivi-shell/ivi-shell.c             |    7 +-
> >  tests/ivi_layout-internal-test.c  |  220 +++++-----------------------------
> >  tests/ivi_layout-test-plugin.c    |  126 +++++---------------
> >  8 files changed, 85 insertions(+), 685 deletions(-)
> >
> > --
> > 1.7.9.5
> > _______________________________________________
> > wayland-devel mailing list
> > [email protected]
> > https://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to