[PATCH 1/5] ivi-shell: remove struct link_screen

Pekka Paalanen ppaalanen at gmail.com
Thu Oct 1 03:23:05 PDT 2015


On Thu, 01 Oct 2015 17:13:47 +0900
Nobuhiko Tanibata <nobuhiko_tanibata at xddp.denso.co.jp> wrote:

> Hi,
> 
> Tested-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA at xddp.denso.co.jp>
> Reviewed-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20151001/4a59602d/attachment-0001.sig>


More information about the wayland-devel mailing list