Why track window activeness? (Re: [PATCH weston v3 2/8] compositor: Track inhibition state in weston_surface)

2016-06-30 Thread Pekka Paalanen
On Wed, 29 Jun 2016 16:57:31 -0700 Bryce Harrington wrote: > On Tue, Jun 28, 2016 at 12:15:49PM +0300, Pekka Paalanen wrote: > > On Thu, 23 Jun 2016 17:32:50 -0700 > > Bryce Harrington wrote: > > > > > On Thu, May 26, 2016 at 06:01:27PM +0300,

Re: [PATCH weston v3 2/8] compositor: Track inhibition state in weston_surface

2016-06-29 Thread Bryce Harrington
On Tue, Jun 28, 2016 at 12:15:49PM +0300, Pekka Paalanen wrote: > On Thu, 23 Jun 2016 17:32:50 -0700 > Bryce Harrington wrote: > > > On Thu, May 26, 2016 at 06:01:27PM +0300, Pekka Paalanen wrote: > > > On Thu, 7 Apr 2016 16:44:17 -0700 > > > Bryce Harrington

Re: [PATCH weston v3 2/8] compositor: Track inhibition state in weston_surface

2016-06-28 Thread Pekka Paalanen
On Thu, 23 Jun 2016 17:32:50 -0700 Bryce Harrington wrote: > On Thu, May 26, 2016 at 06:01:27PM +0300, Pekka Paalanen wrote: > > On Thu, 7 Apr 2016 16:44:17 -0700 > > Bryce Harrington wrote: > > > > > Signed-off-by: Bryce Harrington

Re: [PATCH weston v3 2/8] compositor: Track inhibition state in weston_surface

2016-06-23 Thread Bryce Harrington
On Thu, May 26, 2016 at 06:01:27PM +0300, Pekka Paalanen wrote: > On Thu, 7 Apr 2016 16:44:17 -0700 > Bryce Harrington wrote: > > > Signed-off-by: Bryce Harrington > > Hi, > > the commit message could use an explanation of what the 'bool active'

Re: [PATCH weston v3 2/8] compositor: Track inhibition state in weston_surface

2016-05-26 Thread Pekka Paalanen
On Thu, 7 Apr 2016 16:44:17 -0700 Bryce Harrington wrote: > Signed-off-by: Bryce Harrington Hi, the commit message could use an explanation of what the 'bool active' will actually be useful for. What do we need the "is in active use" for? The

[PATCH weston v3 2/8] compositor: Track inhibition state in weston_surface

2016-04-07 Thread Bryce Harrington
Signed-off-by: Bryce Harrington --- v3: Rename inhibit_screensaving to inhibit_idling desktop-shell/shell.c | 4 +++- fullscreen-shell/fullscreen-shell.c | 25 +++--- ivi-shell/ivi-shell.c | 4 +++- src/compositor.c