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
