Re: [PATCH wayland] scanner: Replace #define tab with space

2016-05-26 Thread Quentin Glidic
On 24/05/2016 21:55, Yong Bakos wrote: From: Yong Bakos wayland-client-protocol.h and wayland-server-protocol.h use a tab between the identifier and token of generated #defines for request/event opcodes and versions. While this sometimes enables vertical alignment, it more frequently doesn't, a

[PATCH libinput 2/2] doc: add a check to compare local files with those in the makefile

2016-05-26 Thread Peter Hutterer
Avoid forgetting about adding svgs/dotfiles to the Makefile.am Signed-off-by: Peter Hutterer --- doc/Makefile.am | 10 ++ 1 file changed, 10 insertions(+) diff --git a/doc/Makefile.am b/doc/Makefile.am index 9a0bca1..8266e3b 100644 --- a/doc/Makefile.am +++ b/doc/Makefile.am @@ -85,6 +8

[PATCH libinput 1/2] doc: ship the doc sources even when not building with docs

2016-05-26 Thread Peter Hutterer
No effect so far because the dist-hook prevents us from making a tarball without the sources anyway. But for correctness split the two up. Signed-off-by: Peter Hutterer --- doc/Makefile.am | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/doc/Makefile.am b/doc/M

Re: [PATCH 0/6] RDP compositor needs love v2

2016-05-26 Thread Hardening
Le 26/04/2016 23:34, David Fort a écrit : > Compared to the first version: > * compilation with FreeRDP 2.0 is now prefered; > * trivial patch using constant instead of hard coded value has its own commit; > * I have splitted the output resized patch so that the shell and the core > weston > part

Re: [PATCH weston v3 1/8] compositor: Track idle as a per-seat property

2016-05-26 Thread Pekka Paalanen
On Thu, 7 Apr 2016 16:44:16 -0700 Bryce Harrington wrote: > Instead of having a single global idle tracker, track idling separately > for each seat. Still treat inhibition on any one seat as inhibiting > the screensaver globally, for now. Hi, the screensaver? The commit message should explai

Re: [PATCH weston v3 8/8] desktop-shell: Enable per-output fade animations

2016-05-26 Thread Pekka Paalanen
On Thu, 7 Apr 2016 16:44:23 -0700 Bryce Harrington wrote: > Instead of creating a single global fade surface across all outputs, > create a separate surface for each output. This lets us individually > fade each output (or block fading if the output has an inhibiting > surface). Hi, sounds go

Re: [PATCH weston v3 6/8] ivi-shell: Support tracking active surfaces

2016-05-26 Thread Pekka Paalanen
On Thu, 7 Apr 2016 16:44:21 -0700 Bryce Harrington wrote: > Activity for ivi-shell follows either click or touch. Only a single > surface can be active in the shell at a time. > > Signed-off-by: Bryce Harrington > --- > Makefile.am | 4 ++- > configure.ac | 2 ++ > ivi-s

Re: [PATCH weston v3 5/8] fullscreen-shell: Support tracking active surfaces

2016-05-26 Thread Pekka Paalanen
On Thu, 7 Apr 2016 16:44:20 -0700 Bryce Harrington wrote: > Surface activity is determined by what surface is being displayed > fullscreen. Only a single surface can be active in the shell. Hi, only a single surface can be active? But there is a surface for every output. Shouldn't every visi

Re: [PATCH weston v3 7/8] Define the screensaver inhibitor client interface

2016-05-26 Thread Pekka Paalanen
On Thu, 7 Apr 2016 16:44:22 -0700 Bryce Harrington wrote: > Hook up the API defined in wayland-protocols to allow client screensaver > inhibition requests. > > Signed-off-by: Bryce Harrington > --- > v2: Update for protocol rename > > Makefile.am | 9 +++-- > clients/simple-shm

Re: [PATCH weston v3 4/8] desktop-shell: Support tracking active surfaces

2016-05-26 Thread Pekka Paalanen
On Thu, 7 Apr 2016 16:44:19 -0700 Bryce Harrington wrote: > Activity for desktop-shell simply follows keyboard focus. Only a single > surface can be active at a time. > > Signed-off-by: Bryce Harrington > --- > desktop-shell/shell.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a

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 comment in the code does not really tell me either

Re: [PATCH weston v3 3/8] compositor: If the output is inhibited, don't idle it off

2016-05-26 Thread Pekka Paalanen
On Thu, 7 Apr 2016 16:44:18 -0700 Bryce Harrington wrote: > Adds a helper routine weston_output_inhibited_outputs() which returns a > mask of outputs that should inhibit screen idling. > > Use this routine to check for inhibiting outputs for handling of idle > behaviors in core: In sleep mode,

Re: Protocol for window previews/thumbnails

2016-05-26 Thread adlo
> On 24 May 2016, at 16:06, Pekka Paalanen wrote: > > The "private to weston" also means it should be a privileged interface: > arbitrary clients must not be able to use it, as it's none of their > concern and could be a security concern. > What about Wayland Security Modules (libwsm)? In what