Hi, I already reviewed the patches before Michael sent: Reviewed-by: Emre Ucan <[email protected]>
Best regards Emre Ucan Engineering Software Base (ADITG/ESB) Tel. +49 5121 49 6937 > -----Original Message----- > From: wayland-devel [mailto:wayland-devel- > [email protected]] On Behalf Of Michael Teyfel > Sent: Dienstag, 17. Oktober 2017 12:02 > To: [email protected] > Subject: [PATCH weston 00/14] Desktop Protocol Support for IVI-Shell > > Hello all, > > since some time I’m working on ivi-shell to add xdg-protocol support by > means of libweston-desktop. Due to my changes both xdg-protocol > applications and ivi-shell / ivi-application-protocol applications are > supported > within ivi-shell now. The known functionality is preserved and just extended > by a further protocol. The advantage is that client applications do not need > to > be edited to generate an id and are also not limited to use the custom ivi- > application protocol anymore, since the ids are handled by an id agent inside > of weston now. > > As a preparation for the changes the goto labels in ivi-shell have been > reworked to avoid memory leaks. In ivi-layout I added an interface > (ivi_layout_surface_set_id) to set the surface-id of an ivi-layout-surface. It > can be done once after being created by an xdg-protocol application to assign > a numeric id by means of an id agent for example. Additionally I introduced a > new event to notify about a desktop surface being configured > (desktop_surface_configured). An id agent can register to this event and > react to this accordingly by assigning an id by means of > ivi_layout_surface_set_id. As a result I also changed the test client > applications in the Weston repository and removed the ivi-application > protocol support since that has only been used, if xdg protocol is not > supported. Finally hmi-controller has been edited to accept desktop > surfaces. > > There are some things that can be done in the future: At first it would be > diligent, if hmi-controller would also use xdg protocol for the GUI itself. > Then > also the surface_configure event could be removed from hmi-controller. > Secondly the weston unit tests should also test the interface changes for > surface_set_id and also should stop using the ivi-application protocol. > > Thanks for reading and questions are very welcome. > > > Best regards > > Michael Teyfel > Advanced Driver Information Technology GmbH > Engineering Software Base (ADITG/ESB) > Robert-Bosch-Str. 200 > 31139 Hildesheim > Germany > Tel. +49 5121 49 6932 > Fax +49 5121 49 6999 > [email protected] > ADIT is a joint venture company of Robert Bosch GmbH/Robert Bosch Car > Multimedia GmbH and DENSO Corporation > Sitz: Hildesheim, Registergericht: Amtsgericht Hildesheim HRB 3438 > Geschäftsführung: Wilhelm Grabow, Ken Yaguchi > > > Michael Teyfel (14): > ivi-shell: rework goto labels to avoid memory leaks > ivi-shell: removed assert > ivi-shell: introduction of IVI_INVALID_ID > layout-interface: added interface to change surface id > ivi-layout: introduced configure_desktop_changed > ivi-layout: introduced surface create and configure > ivi-shell: linked libweston-desktop and added structs > ivi-shell: added libweston-desktop-api implementation > ivi-shell: remove surface_destroy_listener > ivi-shell: create weston_desktop in wet_shell_init > hmi-controller: register for desktop_surface_configured > simple-egl: remove ivi-application support > simple-shm: remove ivi-application support > window client: remove ivi-application support > > Makefile.am | 11 +-- > clients/simple-egl.c | 86 +++------------- > clients/simple-shm.c | 40 -------- > clients/window.c | 44 +-------- > ivi-shell/hmi-controller.c | 70 +++++++------ > ivi-shell/ivi-layout-export.h | 18 ++++ > ivi-shell/ivi-layout-private.h | 3 + > ivi-shell/ivi-layout-shell.h | 8 ++ > ivi-shell/ivi-layout.c | 142 ++++++++++++++++++++++----- > ivi-shell/ivi-shell.c | 218 > ++++++++++++++++++++++++++++++++++++++--- > ivi-shell/ivi-shell.h | 2 + > 11 files changed, 407 insertions(+), 235 deletions(-) > > -- > 2.7.4 > > _______________________________________________ > 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
