On Thu, 01 Oct 2015 17:13:47 +0900 Nobuhiko Tanibata <nobuhiko_tanib...@xddp.denso.co.jp> wrote:
> Hi, > > Tested-by: Nobuhiko Tanibata <nobuhiko_tanib...@xddp.denso.co.jp> > Reviewed-by: Nobuhiko Tanibata <nobuhiko_tanib...@xddp.denso.co.jp> > > These constructors are reserved for a feature of 'a ivi-surface to > several layers' and 'a ivi-layer to several screens'. This feature is > required for > - for example, car navigation application prepares junction guide on a > surface, which shall be displayed in its own UI and cloned on > instrumental cluster in another screen as well. In this case, a > ivi-surface can be located in both. > > However, I have to modify weston core to realize it. So, for the time > being, we can remove them at this time to simplify codes. Hi, I'm not sure you need to modify Weston core. You need to create one weston_view for each ivi-screen/ivi-layer/ivi-surface combination. Emre, these are nice clean-ups. I have a couple of comments for the future: - mind the line length in the commit message, I re-wrapped all the patches - use a space after a keyword like 'if', I think there were a few without Pushed all 5 patches: dcba1a1..64635ee master -> master Thanks, pq
pgpOH45ZQlqp9.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel