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

2016-07-14 Thread Emil Velikov
On 13 July 2016 at 15:53, Jan Engelhardt wrote: > > 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.

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

2016-07-14 Thread Yong Bakos
On Jul 13, 2016, at 7:01 PM, Bryce Harrington wrote: > > 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 >

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

2016-07-14 Thread Bryce Harrington
On Thu, Jul 14, 2016 at 09:16:41AM +0200, Quentin Glidic wrote: > On 14/07/2016 04:01, Bryce Harrington wrote: > >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),

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

2016-07-14 Thread Bryce Harrington
On Thu, Jul 14, 2016 at 01:35:09PM +0100, Eric Engestrom wrote: > On Wed, Jul 13, 2016 at 07:01:28PM -0700, Bryce Harrington wrote: > > Previously weston_config_section_get_uint was serving dual purpose for > > parsing both unsigned decimal integer values (ids, counts, seconds, > > etc.) and

[RFC weston] Add strtoint() helper

2016-07-14 Thread Bryce Harrington
Adds a safe strtol helper function, modeled loosely after Wayland scanner's strtouint. This encapsulates the various quirks of strtol behavior, and streamlines the interface to just handling base-10 numbers with a simple true/false error indicator. Signed-off-by: Bryce Harrington

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

2016-07-14 Thread Eric Engestrom
On Wed, Jul 13, 2016 at 07:01:28PM -0700, Bryce Harrington wrote: > 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

[PATCH wayland-protocols v4] Add screensaver idle inhibitor protocol

2016-07-14 Thread Bryce Harrington
This interface allows disabling of screensaver/screenblanking on a per-surface basis. As long as the surface remains visible and non-occluded it blocks the screensaver, etc. from activating on the output(s) that the surface is visible on. To uninhibit, simply destroy the inhibitor object.

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

2016-07-14 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 treats it as a high value hexadecimal. IOW, strtoul("-42", ) sets val to 0xffd6. This

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

2016-07-14 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

Re: [RFC weston] Add strtoint() helper

2016-07-14 Thread Peter Hutterer
On Thu, Jul 14, 2016 at 01:03:34PM -0700, Bryce Harrington wrote: > Adds a safe strtol helper function, modeled loosely after Wayland > scanner's strtouint. This encapsulates the various quirks of strtol > behavior, and streamlines the interface to just handling base-10 numbers > with a simple

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

2016-07-14 Thread Jan Engelhardt
On Thursday 2016-07-14 17:33, Emil Velikov wrote: > >The keypoint here is that one should _not_ need to uninstall >libdb-4_5-devel in order to have libdb-4_8-devel and vice-versa. >This is what parallel installability is all about (afaict). It is indeed what it is about. But is it _necessary_ to

Re: [RFC weston] Add strtoint() helper

2016-07-14 Thread Thiago Macieira
On quinta-feira, 14 de julho de 2016 13:03:34 PDT Bryce Harrington wrote: > + ret = strtol(str, , 10); Here you made a cast from long to int, which may be a loss of data. ret = strtol("4294967296", , 10); Will produce no errno since it's in the valid range of long on LP64 systems,

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

2016-07-14 Thread Quentin Glidic
On 14/07/2016 04:01, Bryce Harrington wrote: 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

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

2016-07-14 Thread Bryce Harrington
On Thu, Jul 14, 2016 at 09:40:46AM -0700, Yong Bakos wrote: > On Jul 13, 2016, at 7:01 PM, Bryce Harrington wrote: > > > > Previously weston_config_section_get_uint was serving dual purpose for > > parsing both unsigned decimal integer values (ids, counts, seconds, > >

Re: [PATCH wayland-protocols v5 4/4] tablet: Add pad support to the tablet protocol

2016-07-14 Thread Peter Hutterer
On Mon, Jul 11, 2016 at 05:13:36PM +0200, Carlos Garnacho wrote: > The pad's interface is similar to the tool interface, a client is notified of > the pad after the tablet_added event. > > The pad has three functionalities: buttons, rings and strips. > Buttons are fairly straightforward, rings

[PATCH libinput 2/2] touchpad: don't init a horizontal scroll area on touchpads <50mm high

2016-07-14 Thread Peter Hutterer
We simply don't have enough space on those touchpads to have an area carved out for horizontal scrolling. Given that horizontal scrolling is rarely needed anyway users of these touchpads will just have to cling to scroll bars or use two-finger scrolling. Exception are small clickpads because they