Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-06 Thread Bryce Harrington
On Fri, Nov 06, 2015 at 10:24:21AM +0200, Pekka Paalanen wrote: > On Fri, 6 Nov 2015 10:51:09 +0800 > Jonas Ådahl wrote: > > > On Thu, Nov 05, 2015 at 11:46:46AM -0800, Bryce Harrington wrote: > > > On Wed, Nov 04, 2015 at 04:49:50PM +0800, Jonas Ådahl wrote: > > > > Use the fullscreen-shell prot

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-06 Thread Jonas Ådahl
On Fri, Nov 06, 2015 at 10:24:21AM +0200, Pekka Paalanen wrote: > On Fri, 6 Nov 2015 10:51:09 +0800 > Jonas Ådahl wrote: > > > On Thu, Nov 05, 2015 at 11:46:46AM -0800, Bryce Harrington wrote: > > > On Wed, Nov 04, 2015 at 04:49:50PM +0800, Jonas Ådahl wrote: > > > > Use the fullscreen-shell prot

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-06 Thread Pekka Paalanen
On Fri, 6 Nov 2015 10:51:09 +0800 Jonas Ådahl wrote: > On Thu, Nov 05, 2015 at 11:46:46AM -0800, Bryce Harrington wrote: > > On Wed, Nov 04, 2015 at 04:49:50PM +0800, Jonas Ådahl wrote: > > > Use the fullscreen-shell protocol XML from the wayland-protocols > > > installation, and remove the one w

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-05 Thread Jonas Ådahl
On Thu, Nov 05, 2015 at 09:00:21PM -0800, Bryce Harrington wrote: > On Fri, Nov 06, 2015 at 10:51:09AM +0800, Jonas Ådahl wrote: > > On Thu, Nov 05, 2015 at 11:46:46AM -0800, Bryce Harrington wrote: > > > On Wed, Nov 04, 2015 at 04:49:50PM +0800, Jonas Ådahl wrote: > > > > Use the fullscreen-shell

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-05 Thread Bryce Harrington
On Fri, Nov 06, 2015 at 10:51:09AM +0800, Jonas Ådahl wrote: > On Thu, Nov 05, 2015 at 11:46:46AM -0800, Bryce Harrington wrote: > > On Wed, Nov 04, 2015 at 04:49:50PM +0800, Jonas Ådahl wrote: > > > Use the fullscreen-shell protocol XML from the wayland-protocols > > > installation, and remove the

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-05 Thread Jonas Ådahl
On Thu, Nov 05, 2015 at 11:46:46AM -0800, Bryce Harrington wrote: > On Wed, Nov 04, 2015 at 04:49:50PM +0800, Jonas Ådahl wrote: > > Use the fullscreen-shell protocol XML from the wayland-protocols > > installation, and remove the one we provide ourself. > > > > Signed-off-by: Jonas Ådahl > > > >

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-05 Thread Jonas Ådahl
On Thu, Nov 05, 2015 at 01:19:27PM +0200, Pekka Paalanen wrote: > On Wed, 4 Nov 2015 16:49:50 +0800 > Jonas Ådahl wrote: > > > Use the fullscreen-shell protocol XML from the wayland-protocols > > installation, and remove the one we provide ourself. > > > > Signed-off-by: Jonas Ådahl > > --- >

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-05 Thread Bill Spitzak
On Thu, Nov 5, 2015 at 11:46 AM, Bryce Harrington wrote: > > Hmm. With l's and 1's looking so similar in certain fonts, "shell1" is > going to look like a typo to some users. IMO it would be better to > distinguish this version number with at least an underscore. > "_shell_v1_" would feel more

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-05 Thread Bryce Harrington
On Wed, Nov 04, 2015 at 04:49:50PM +0800, Jonas Ådahl wrote: > Use the fullscreen-shell protocol XML from the wayland-protocols > installation, and remove the one we provide ourself. > > Signed-off-by: Jonas Ådahl > > diff --git a/clients/fullscreen.c b/clients/fullscreen.c > index 4fcca3d..be316

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-05 Thread Bryce Harrington
On Thu, Nov 05, 2015 at 01:19:27PM +0200, Pekka Paalanen wrote: > On Wed, 4 Nov 2015 16:49:50 +0800 > Jonas Ådahl wrote: > > > Use the fullscreen-shell protocol XML from the wayland-protocols > > installation, and remove the one we provide ourself. > > > > Signed-off-by: Jonas Ådahl > > --- >

Re: [PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-05 Thread Pekka Paalanen
On Wed, 4 Nov 2015 16:49:50 +0800 Jonas Ådahl wrote: > Use the fullscreen-shell protocol XML from the wayland-protocols > installation, and remove the one we provide ourself. > > Signed-off-by: Jonas Ådahl > --- > Makefile.am | 44 +--- > clients/fullscreen.c

[PATCH weston 01/10] Use fullscreen-shell.xml from wayland-protocols

2015-11-04 Thread Jonas Ådahl
Use the fullscreen-shell protocol XML from the wayland-protocols installation, and remove the one we provide ourself. Signed-off-by: Jonas Ådahl --- Makefile.am | 44 +--- clients/fullscreen.c| 60 +-- clients/simple-damage.c | 1