Re: Input and games.

2013-04-23 Thread David Herrmann
Hi Todd On Tue, Apr 23, 2013 at 4:12 PM, Todd Showalter wrote: > On Tue, Apr 23, 2013 at 7:25 AM, Pekka Paalanen wrote: > >> what you describe here is very much a keymap-like database for game >> controllers: translating from button and axis indices to labels or >> symbols. However, having a bri

Re: [PATCH 2/3] Add a 'serial' property on weston_output

2013-04-23 Thread Richard Hughes
On 23 April 2013 16:37, Rob Bradford wrote: > When I read the commit message I wondered "why does he need a serial" > for the wl_output. I was of course thinking about the wayland use of > "serial" as a sequence number. Ooops, my bad. > Are you intending to expose this through the wayland protoc

Re: [PATCH 3/3] Parse the EDID when outputs are added

2013-04-23 Thread Richard Hughes
On 23 Apr 2013 16:36, "Rob Bradford" wrote: > Please can you clarify what I think this statement implies. That is that > you are certifying that you have the rights to license this code under > the MIT license as carried by Weston. Correct, i.e. I read the VESA EDID specification document. :) Ric

Re: Pointer lock and warping

2013-04-23 Thread Bill Spitzak
On 04/23/2013 12:11 AM, Pekka Paalanen wrote: Here's a lot simpler solution for non-jittery dragging of objects: just hide the pointer cursor, when starting the move. If you still want the pointer cursor visible, draw it yourself with the object you are moving. That is already possible today, an

Re: [PATCH 2/3] Add a 'serial' property on weston_output

2013-04-23 Thread Rob Bradford
On Mon, Apr 22, 2013 at 12:57:03PM +0100, Richard Hughes wrote: > --- > src/compositor-drm.c | 1 + > src/compositor.h | 2 +- > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/compositor-drm.c b/src/compositor-drm.c > index 61ef97e..a454676 100644 > --- a/src/compositor-d

Re: [PATCH 3/3] Parse the EDID when outputs are added

2013-04-23 Thread Rob Bradford
On Mon, Apr 22, 2013 at 12:57:04PM +0100, Richard Hughes wrote: > At the moment we're only extracting interesting strings. We have to be quite > careful parsing the EDID data, as vendors like to do insane things. > > The original EDID parsing code was written by me for gnome-color-manager. Hi Ric

Re: [PATCH weston 2/6] weston: Allow relative paths for modules

2013-04-23 Thread John Kåre Alsaker
You should drop the code changes since they already allow relative paths. You just changed it to only allow base filenames. On Tue, Apr 23, 2013 at 2:54 PM, Quentin Glidic < sardemff7+wayl...@sardemff7.net> wrote: > From: Quentin Glidic > > Signed-off-by: Quentin Glidic > --- > man/weston.man

Re: [PATCH weston 2/6] weston: Allow relative paths for modules

2013-04-23 Thread Bill Spitzak
On 04/23/2013 05:54 AM, Quentin Glidic wrote: From: Quentin Glidic - if (name[0] != '/') + if (!strchr(name, '/')) snprintf(path, sizeof path, "%s/%s", MODULEDIR, name); else snprintf(path, sizeof path, "%s", name); This disallows any subd

Re: [PATCH weston 1/6] shared/option-parser: Allow spaced options

2013-04-23 Thread Bill Spitzak
On 04/23/2013 05:54 AM, Quentin Glidic wrote: +You can specify short options having an argument with a following space. Long +options with argument can be specified either with or without an equal sign. -static void +static bool handle_option(const struct weston_option *option, char *value)

Re: Input and games.

2013-04-23 Thread Todd Showalter
On Tue, Apr 23, 2013 at 7:25 AM, Pekka Paalanen wrote: > what you describe here is very much a keymap-like database for game > controllers: translating from button and axis indices to labels or > symbols. However, having a brief chat with Daniel Stone, it seems we > should not need these. > > Tak

Re: [PATCH weston] compositor: add a layer clipping mechanism

2013-04-23 Thread Pekka Paalanen
On Tue, 23 Apr 2013 15:01:18 +0200 Giulio Camuffo wrote: > 2013/4/23 Pekka Paalanen > > > On Mon, 8 Apr 2013 14:31:53 +0200 > > Giulio Camuffo wrote: > > > > > this adds a mechanism to clip the surfaces belonging to a layer > > > to an arbitrary rect > > > > Hi Giulio, > > > > I don't have an

Re: [PATCH weston] compositor: add a layer clipping mechanism

2013-04-23 Thread Giulio Camuffo
2013/4/23 Pekka Paalanen > On Mon, 8 Apr 2013 14:31:53 +0200 > Giulio Camuffo wrote: > > > this adds a mechanism to clip the surfaces belonging to a layer > > to an arbitrary rect > > Hi Giulio, > > I don't have anything against the idea of clipping layers, but the > implementation below is puz

[PATCH weston 6/6] headless-backend: Init fake pointer and keyboard

2013-04-23 Thread Quentin Glidic
From: "U. Artie Eoff" Signed-off-by: Quentin Glidic --- src/compositor-headless.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/compositor-headless.c b/src/compositor-headless.c index 4720329..507d4bf 100644 --- a/src/compositor-headless.c +++ b/src/compositor-headless.c @@ -173,6 +

[PATCH weston 5/6] tests: Drop weston-tests-env

2013-04-23 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- tests/.gitignore | 10 +++-- tests/Makefile.am | 56 -- tests/weston-tests-env | 39 --- 3 files changed, 44 insertions(+), 61 deletions(-) dele

[PATCH weston 4/6] shared/option-parser: Add string list options

2013-04-23 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- man/weston.man | 6 +++--- shared/config-parser.h | 1 + shared/option-parser.c | 13 + src/compositor.c | 27 --- tests/weston-tests-env | 6 -- 5 files changed, 45 insertions(+), 8

[PATCH weston 3/6] weston-test: Get the test client path from args

2013-04-23 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- tests/weston-test.c| 18 -- tests/weston-tests-env | 3 ++- 2 files changed, 14 insertions(+), 7 deletions(-) diff --git a/tests/weston-test.c b/tests/weston-test.c index 55c5da4..c19db3a 100644 --- a/tests/weston-test

[PATCH weston 1/6] shared/option-parser: Allow spaced options

2013-04-23 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- man/weston.man | 3 +++ shared/option-parser.c | 25 - 2 files changed, 19 insertions(+), 9 deletions(-) diff --git a/man/weston.man b/man/weston.man index 39d854b..a25e619 100644 --- a/man/weston.man +++ b

[PATCH weston 2/6] weston: Allow relative paths for modules

2013-04-23 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- man/weston.man | 4 ++-- src/compositor.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/man/weston.man b/man/weston.man index a25e619..97db3c8 100644 --- a/man/weston.man +++ b/man/weston.man @@ -102,7 +102,7 @@ L

Re: [PATCH weston] compositor: add a layer clipping mechanism

2013-04-23 Thread Pekka Paalanen
On Mon, 8 Apr 2013 14:31:53 +0200 Giulio Camuffo wrote: > this adds a mechanism to clip the surfaces belonging to a layer > to an arbitrary rect Hi Giulio, I don't have anything against the idea of clipping layers, but the implementation below is puzzling. > --- > > I'm using this functional

Re: Input and games.

2013-04-23 Thread Pekka Paalanen
On Mon, 22 Apr 2013 15:32:50 -0400 Todd Showalter wrote: > On Mon, Apr 22, 2013 at 1:40 PM, Pekka Paalanen > wrote: > > >> Gamepads, by contrast, are all mostly the same these days, much > >> like mice. You can find oddball ones like that PC gamepad that > >> was up on Kickstarter recently

Re: [DRM] Patch for Compositor Drm

2013-04-23 Thread Ander Conselvan de Oliveira
Hi, Thanks for fixing this. Just a small nitpick about the commit message. We keep the summary line below 76 characters because of git log, and we also prefix it with the component to which the change was made. In this particular case, I would go with something like compositor-drm: Check re

Re: [PATCH] Extract and parse the EDID when outputs are added

2013-04-23 Thread Ander Conselvan de Oliveira
Hi, On 04/22/2013 02:57 PM, Richard Hughes wrote: At the moment we're only extracting interesting strings. We have to be quite careful parsing the EDID data, as vendors like to do insane things. You should add some information of why this patch is necessary to the commit message. It seems yo

Re: [PATCH 1/3] Extract the EDID blob when adding a DRM output

2013-04-23 Thread Richard Hughes
On 23 April 2013 08:32, Kai-Uwe Behrmann wrote: > Client side parsing is a different issue indeed. However the patch author > mentioned in the initial post GNOME CMS alias colord as a plugin inside the > Weston CMS. So other possible CMS plugins like Argyll and Oyranos have > different requirement

Re: [PATCH] evdev: Handle SYN_DROPPED, keep transaction buffer

2013-04-23 Thread Pekka Paalanen
On Sat, 6 Apr 2013 00:27:10 +0200 Martin Minarik wrote: > Each evdev device keeps a key press bitmap, > the incoming events are filtered to the following constraints: > > 1. only notify releases for previously pressed keys > 2. only notify presses for previously released keys > 3. on device dest

RE: [RFC] [tests] Running the wayland tests against compositor-headless

2013-04-23 Thread Eoff, Ullysses A
> -Original Message- > From: Sam Spilsbury [mailto:smspil...@gmail.com] >> >> >> > Ah interesting. I guess the point of the tests is to verify the > backends themselves as opposed to testing the core compositor (well, > it tests that as an incident, but I guess its not the point?). Does it

Re: [PATCH 1/3] Extract the EDID blob when adding a DRM output

2013-04-23 Thread Kai-Uwe Behrmann
Am 23.04.2013 07:49, schrieb David Herrmann: On Apr 23, 2013 6:38 AM, "Graeme Gill" wrote: David Herrmann wrote: Patch 3/3 parses the blob and keeps the parsed values. Why should we keep the binary blob, too? If we need more EDID values, we simply extend the parser? This would seem to impl

Re: Pointer lock and warping

2013-04-23 Thread Pekka Paalanen
On Mon, 22 Apr 2013 19:48:14 -0700 Bill Spitzak wrote: > Thinking about Todd Showalter's requests for pointer warping, I think > in fact a correctly-behaving Wayland app will always want pointer > warping and incremental update when the mouse is held down. This > would require a major change in t