Re: [PATCH weston] Remove a wrong closing “extern "C"” in shared/xalloc.c

2016-07-13 Thread Thiago Macieira
Em quinta-feira, 14 de julho de 2016, às 01:38:13 PDT, Emmanuel Gil Peyrot escreveu: > > Out of curiosity: why are you compiling C sources as C++? > > > > Is that the Arduino compiler? > > I wasn’t, I just noticed this leftover from a previous copy (likely the > file creation), and just fixed

Re: [PATCH weston] Remove a wrong closing “extern "C"” in shared/xalloc.c

2016-07-13 Thread Thiago Macieira
> -#ifdef __cplusplus > -} > -#endif Out of curiosity: why are you compiling C sources as C++? Is that the Arduino compiler? -- Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org Software Architect - Intel Open Source Technology Center

Re: [PATCH weston] Remove a wrong closing “extern "C"” in shared/xalloc.c

2016-07-13 Thread Emmanuel Gil Peyrot
On Wed, Jul 13, 2016 at 05:13:24PM -0700, Thiago Macieira wrote: > > -#ifdef __cplusplus > > -} > > -#endif > > Out of curiosity: why are you compiling C sources as C++? > > Is that the Arduino compiler? I wasn’t, I just noticed this leftover from a previous copy (likely the file creation),

Re: [PATCH weston] gl-renderer: Silence silly warning

2016-07-13 Thread Quentin Glidic
On 11/07/2016 23:26, Emil Velikov wrote: On 4 July 2016 at 15:00, Quentin Glidic wrote: From: Quentin Glidic Signed-off-by: Quentin Glidic --- libweston/gl-renderer.c | 2 +- 1 file changed, 1

[PATCH wayland-protocols] Introduce xdg-foreign protocol

2016-07-13 Thread Jonas Ådahl
xdg-foreign is a protocol meant to enable setting up inter surface relationships across clients. Potential use cases are out-of-process dialogs, such as file dialogs, meant to be used by sandboxed processes that may not have the access it needs to implement such dialogs. It works by enabling a

[PATCH weston] Revert "config-parser: Catch negative numbers assigned to unsigned config values"

2016-07-13 Thread Bryce Harrington
The reduction in range limits does have an effect for color values, which are expressed as hexadecimal values from 0x to 0x. By limiting the range to INT_MAX, color values of 0x8000 and up are in fact lost. This reverts commit 6351fb08c2e302f8696b2022830e5317e7219c39.

Re: [PATCH weston] Revert "config-parser: Catch negative numbers assigned to unsigned config values"

2016-07-13 Thread Bryce Harrington
On Wed, Jul 13, 2016 at 01:27:29PM -0700, Bryce Harrington wrote: > The reduction in range limits does have an effect for color values, > which are expressed as hexadecimal values from 0x to > 0x. By limiting the range to INT_MAX, color values of > 0x8000 and up are in fact

Re: [PATCH weston] Revert "config-parser: Catch negative numbers assigned to unsigned config values"

2016-07-13 Thread Yong Bakos
On Jul 13, 2016, at 2:10 PM, Bryce Harrington wrote: > > On Wed, Jul 13, 2016 at 01:27:29PM -0700, Bryce Harrington wrote: >> The reduction in range limits does have an effect for color values, >> which are expressed as hexadecimal values from 0x to >> 0x.

Re: [PATCH wayland-protocols v5 2/4] tablet: change all degree values from int to wl_fixed

2016-07-13 Thread Bryce Harrington
On Mon, Jul 11, 2016 at 05:13:34PM +0200, Carlos Garnacho wrote: > From: Peter Hutterer > > Signed-off-by: Peter Hutterer > Reviewed-by: Jason Gerecke > Reviewed-by: Carlos Garnacho Reviewed-by:

Re: [PATCH weston] Revert "config-parser: Catch negative numbers assigned to unsigned config values"

2016-07-13 Thread Michael Blumenkrantz
On Wed, 13 Jul 2016 14:27:17 -0700 Yong Bakos wrote: > On Jul 13, 2016, at 2:10 PM, Bryce Harrington wrote: > > > > On Wed, Jul 13, 2016 at 01:27:29PM -0700, Bryce Harrington wrote: > >> The reduction in range limits does have an effect for

Re: [PATCH weston] Revert "config-parser: Catch negative numbers assigned to unsigned config values"

2016-07-13 Thread Yong Bakos
On Jul 13, 2016, at 1:27 PM, Bryce Harrington wrote: > > The reduction in range limits does have an effect for color values, > which are expressed as hexadecimal values from 0x to > 0x. By limiting the range to INT_MAX, color values of > 0x8000 and up

Re: [PATCH weston] Revert "config-parser: Catch negative numbers assigned to unsigned config values"

2016-07-13 Thread Bryce Harrington
On Wed, Jul 13, 2016 at 02:12:43PM -0700, Yong Bakos wrote: > On Jul 13, 2016, at 1:27 PM, Bryce Harrington wrote: > > > > The reduction in range limits does have an effect for color values, > > which are expressed as hexadecimal values from 0x to > > 0x.

Re: [PATCH wayland-protocols v5 1/4] tablet: add v2 of the tablet protocol

2016-07-13 Thread Bryce Harrington
On Mon, Jul 11, 2016 at 05:13:33PM +0200, Carlos Garnacho wrote: > From: Peter Hutterer > > This is a straightforward copy/paste with a _v1 -> _v2 rename. No functional > changes otherwise. > > Signed-off-by: Peter Hutterer > Reviewed-by:

Re: [PATCH wayland-protocols v5 3/4] tablet: restrict the cursor surface to one per tool

2016-07-13 Thread Bryce Harrington
On Mon, Jul 11, 2016 at 05:13:35PM +0200, Carlos Garnacho wrote: > From: Peter Hutterer > > The initial approach was to allow one surface to be re-used between tools, > seats and even used together as wl_pointer cursor surface. This has a few > drawbacks, most of which

[PATCH weston 1/2] config-parser: Add weston_config_section_get_color

2016-07-13 Thread Bryce Harrington
Previously weston_config_section_get_uint was serving dual purpose for parsing both unsigned decimal integer values (ids, counts, seconds, etc.) and hexadecimal values (colors), by relying on strtoul's auto-detection mechanism. However, this usage is unable to catch certain kinds of error

[PATCH weston 2/2] Re-apply "config-parser: Catch negative numbers assigned to unsigned config values"

2016-07-13 Thread Bryce Harrington
[With hexadecimal color values now handled via their own routine, re-introduce the negative unsigned numbers fix.] strtoul() has a side effect that when given a string representing a negative number, it returns a negated version as the value, and does not flag an error. IOW, strtoul("-42", )

Re: [PATCH weston] Revert "config-parser: Catch negative numbers assigned to unsigned config values"

2016-07-13 Thread Bryce Harrington
On Wed, Jul 13, 2016 at 02:27:17PM -0700, Yong Bakos wrote: > On Jul 13, 2016, at 2:10 PM, Bryce Harrington wrote: > > > > On Wed, Jul 13, 2016 at 01:27:29PM -0700, Bryce Harrington wrote: > >> The reduction in range limits does have an effect for color values, > >> which

Re: [PATCH libinput] touchpad: reduce middle button size on Dell touchpads to 10mm

2016-07-13 Thread Hans de Goede
Hi, On 13-07-16 01:51, Peter Hutterer wrote: All Dell touchpas appear to have a visual marker on their touchpads. With a visible marker our middle button can (and should) be much smaller since we can rely on users to hit the button precisely. https://bugs.freedesktop.org/show_bug.cgi?id=96710

Re: Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

2016-07-13 Thread Pekka Paalanen
On Sun, 10 Jul 2016 14:34:28 +0200 (CEST) Jan Engelhardt wrote: > On Sunday 2016-07-10 13:13, Quentin Glidic wrote: > > > > If we install only one .pc file: > > - You cannot develop against an old version. > > I do not feel that is true. If you have Berkeley DB 4.5 in tarball

Re: Touch events not reviving with wayland-ivi-extenssion 1.4.0 and wayland video sink

2016-07-13 Thread Vikas Patil
Dear All, Seeing the below wayland log without above workaround focus is going 0 to surface. I think it means surface is failing to grab the touch focus. What could be the reasons of focus not getting setup? I think this [3687363.407] ivi_input@19.input_focus(90, 4, 0) need to be [3687363.407]

Re: Weston versioning (Re: [PATCH weston 6/6] libweston: do not use weston version in libweston.pc)

2016-07-13 Thread Jan Engelhardt
On Wednesday 2016-07-13 13:54, Pekka Paalanen wrote: > >I think Quentin raised a good point, though. In source-based >distros, well, in Gentoo at least which I use almost exclusively, >there are no separate -devel packages. A package is, abstractly, merely a selected subset of `make install`