Re: [RFC] xserver: rip indexes out of the API/ABI

2012-04-25 Thread Dave Airlie
On Thu, Apr 26, 2012 at 1:10 AM, Aaron Plattner wrote: > On 04/10/2012 02:10 PM, Alan Coopersmith wrote: >> >> On 04/10/12 07:47 AM, Dave Airlie wrote: >>> >>> This patch series tears strips out of the current API, its not complete, >>> with further changes to ddc/i2c code I can remove all referen

Re: [PATCH synaptics 2/2] Reset scroll delta when finger is lifted

2012-04-25 Thread Peter Hutterer
On Wed, Apr 25, 2012 at 10:37:26PM -0700, Chase Douglas wrote: > On 04/25/2012 08:00 PM, Peter Hutterer wrote: > > Provides for a more consistent scrolling experience, otherwise delta > > leftovers may trigger extra events even when the actual scrolling action > > stays the same. > > > > Signed-of

Re: [PATCH synaptics 1/2] Don't unconditionally divide by scroll_dist_vert (#46617)

2012-04-25 Thread Peter Hutterer
On Wed, Apr 25, 2012 at 10:34:48PM -0700, Chase Douglas wrote: > On 04/25/2012 08:00 PM, Peter Hutterer wrote: > > Regression introduced in cddab79c408db3b13905a2be72aff4f7bf1406f8. > > > > If an event has a delta of less than scroll_dist_vert, the delta is > > unconditionally divided by the dista

[PATCH synaptics v2 1/2] Don't unconditionally divide by scroll_dist_vert (#46617)

2012-04-25 Thread Peter Hutterer
Regression introduced in cddab79c408db3b13905a2be72aff4f7bf1406f8. If an event has a delta of less than scroll_dist_vert, the delta is unconditionally divided by the distance, leaving some remainder close to 0 and never actually triggering the scroll amount. Fix this by working with the increment

Re: [PATCH 0/4] Configure fixes for Windows MinGW target

2012-04-25 Thread Ryan Pavlik
On Wed, Apr 25, 2012 at 8:41 AM, Michal Suchanek wrote: > Hello, > > what else needs to be patched to build on mingw? > > I could test this if it was a full patchset but have no opinion > whasoever configure changes for platform I don't know. > > Thanks > > Michal > I was testing by building usi

Re: [PATCH] TouchListenerAcceptReject: Warn and return early on bad listener index

2012-04-25 Thread Peter Hutterer
On Fri, Apr 20, 2012 at 11:08:15AM -0700, Chase Douglas wrote: > Signed-off-by: Chase Douglas merged, thanks. Cheers, Peter > --- > dix/touch.c |5 + > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/dix/touch.c b/dix/touch.c > index dd16367..401cb98 100644 > --- a

Re: [PATCH 1/2] Report logical button state in ProcXIQueryPointer

2012-04-25 Thread Peter Hutterer
On Fri, Apr 20, 2012 at 11:07:17AM -0700, Chase Douglas wrote: > Physical button state is usually meaningless to an X client. > > Signed-off-by: Chase Douglas > --- > Xi/xiquerypointer.c | 12 > 1 files changed, 4 insertions(+), 8 deletions(-) > > diff --git a/Xi/xiquerypointer.c

Re: [PATCH synaptics 2/2] Reset scroll delta when finger is lifted

2012-04-25 Thread Chase Douglas
On 04/25/2012 08:00 PM, Peter Hutterer wrote: > Provides for a more consistent scrolling experience, otherwise delta > leftovers may trigger extra events even when the actual scrolling action > stays the same. > > Signed-off-by: Peter Hutterer > --- > src/synaptics.c |2 ++ > 1 file changed,

Re: [PATCH synaptics 1/2] Don't unconditionally divide by scroll_dist_vert (#46617)

2012-04-25 Thread Chase Douglas
On 04/25/2012 08:00 PM, Peter Hutterer wrote: > Regression introduced in cddab79c408db3b13905a2be72aff4f7bf1406f8. > > If an event has a delta of less than scroll_dist_vert, the delta is > unconditionally divided by the distance, leaving some remainder close to 0 > and never actually triggering th

Re: [PATCH] dmx: Annotate dmxlog.c with _X_ATTRIBUTE_PRINTF

2012-04-25 Thread Peter Hutterer
On Mon, Apr 23, 2012 at 07:02:29PM +0200, Michal Suchanek wrote: > From c3b73397fce271cefac7a72d5be498aed516dc59 Mon Sep 17 00:00:00 2001 > From: Michal Suchanek > Date: Mon, 23 Apr 2012 15:28:07 +0200 > Subject: [PATCH] dmx: Annotate dmxlog.c with _X_ATTRIBUTE_PRINTF and > _X_NORETURN > > * a

[PULL 1.12] touch fixes for 1.12

2012-04-25 Thread Peter Hutterer
Jeremy, These are a bunch of fixes for mainly the touch code. Quite a lot of them, but to be expected for a rather new feature. The following changes since commit 38e73f7fc29fe8e9defa8a0516a50ac3a84e10c2: configure.ac: Bump to 1.12.1 (2012-04-13 15:44:08 -0700) are available in the git reposi

Re: [PATCH synaptics] Init num_touches to 0 on start

2012-04-25 Thread Peter Hutterer
On Wed, Apr 25, 2012 at 08:01:57PM -0700, Chase Douglas wrote: > On 04/25/2012 07:56 PM, Peter Hutterer wrote: > > On Wed, Apr 25, 2012 at 06:22:07PM -0700, Chase Douglas wrote: > >> On 04/25/2012 05:36 PM, Peter Hutterer wrote: > >>> We implicitly rely on this already since we calloc the struct. D

[PATCH 2/2] xinput has dependancies on xrandr and xinerma

2012-04-25 Thread Benjamin Close
From: Benjamin Close Signed-off-by: Benjamin Close --- xorg.modules |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/xorg.modules b/xorg.modules index 929a6fb..b19b6e7 100644 --- a/xorg.modules +++ b/xorg.modules @@ -923,6 +923,8 @@ + +

[PATCH 1/2] libXrandr provides xrandr.pc which has a private dependancy on xrender

2012-04-25 Thread Benjamin Close
From: Benjamin Close For sanity sake we build xrender as a backup here so a system xrender isn't accidentally used when building apps singulary with: jhbuild build someapp Signed-off-by: Benjamin Close --- xorg.modules |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/xo

Re: [PATCH synaptics] Init num_touches to 0 on start

2012-04-25 Thread Chase Douglas
On 04/25/2012 07:56 PM, Peter Hutterer wrote: > On Wed, Apr 25, 2012 at 06:22:07PM -0700, Chase Douglas wrote: >> On 04/25/2012 05:36 PM, Peter Hutterer wrote: >>> We implicitly rely on this already since we calloc the struct. Do it >>> expliclity on DeviceOn(). >>> >>> Signed-off-by: Peter Huttere

[PATCH synaptics 2/2] Reset scroll delta when finger is lifted

2012-04-25 Thread Peter Hutterer
Provides for a more consistent scrolling experience, otherwise delta leftovers may trigger extra events even when the actual scrolling action stays the same. Signed-off-by: Peter Hutterer --- src/synaptics.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/src/synaptics.c b/src/synaptics

[PATCH synaptics 1/2] Don't unconditionally divide by scroll_dist_vert (#46617)

2012-04-25 Thread Peter Hutterer
Regression introduced in cddab79c408db3b13905a2be72aff4f7bf1406f8. If an event has a delta of less than scroll_dist_vert, the delta is unconditionally divided by the distance, leaving some remainder close to 0 and never actually triggering the scroll amount. Fix this by working with the increment

Re: [PATCH synaptics] Init num_touches to 0 on start

2012-04-25 Thread Peter Hutterer
On Wed, Apr 25, 2012 at 06:22:07PM -0700, Chase Douglas wrote: > On 04/25/2012 05:36 PM, Peter Hutterer wrote: > > We implicitly rely on this already since we calloc the struct. Do it > > expliclity on DeviceOn(). > > > > Signed-off-by: Peter Hutterer > > --- > > This half of the fix to 49161 a b

Re: [PATCH synaptics] ClickPad is most definitely a bool option.

2012-04-25 Thread Chase Douglas
On 04/25/2012 05:09 PM, Peter Hutterer wrote: > Signed-off-by: Peter Hutterer > --- > src/synaptics.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/synaptics.c b/src/synaptics.c > index d2fe960..2a43f95 100644 > --- a/src/synaptics.c > +++ b/src/synaptics.c > @@

Re: [PATCH synaptics] Init num_touches to 0 on start

2012-04-25 Thread Chase Douglas
On 04/25/2012 05:36 PM, Peter Hutterer wrote: > We implicitly rely on this already since we calloc the struct. Do it > expliclity on DeviceOn(). > > Signed-off-by: Peter Hutterer > --- > This half of the fix to 49161 a bit redundant but better safe than sorry, > they say. I have not problems wit

[PATCH 2/2] Add test for XIQueryPointer button mask when physical touch is active

2012-04-25 Thread Chase Douglas
Signed-off-by: Chase Douglas --- The functions to wait for specific events should probably be moved to xorg-gtest eventually, but I want to make sure they are generic enough for all use cases before doing that. configure.ac | 14 + test/integration/.gitign

[PATCH 1/2] Add xorg-gtest integration test framework

2012-04-25 Thread Chase Douglas
Signed-off-by: Chase Douglas --- configure.ac| 21 ++- m4/xorg-gtest.m4| 110 +++ test/integration/Makefile-xorg-gtest.am | 61 + test/integration/Makefile.am|4 + 4 files chan

[PATCH synaptics] Init num_touches to 0 on start

2012-04-25 Thread Peter Hutterer
We implicitly rely on this already since we calloc the struct. Do it expliclity on DeviceOn(). Signed-off-by: Peter Hutterer --- This half of the fix to 49161 a bit redundant but better safe than sorry, they say. src/eventcomm.c |1 + 1 file changed, 1 insertion(+) diff --git a/src/eventco

Re: [PATCH:xmodmap] include config.h before stdio.h & other system headers

2012-04-25 Thread Peter Hutterer
On Tue, Apr 24, 2012 at 03:16:57PM -0700, Alan Coopersmith wrote: > Ensures definitions like _GNU_SOURCE are visible when needed. > > Signed-off-by: Alan Coopersmith Reviewed-by: Peter Hutterer Cheers, Peter > --- > exec.c|4 > pf.c |4 > xmodmap.c |4 >

Re: [PATCH:libXtst 1/2] Move -I flags from AM_CFLAGS to AM_CPPFLAGS

2012-04-25 Thread Peter Hutterer
On Tue, Apr 24, 2012 at 03:20:53PM -0700, Alan Coopersmith wrote: > Ensures local copy of headers takes precedence over any -I flags the > builder may have passed in CPPFLAGS. > > Signed-off-by: Alan Coopersmith > --- Reviewed-by: Peter Hutterer for both Cheers, Peter > src/Makefile.am |

Re: [RFC] xserver: rip indexes out of the API/ABI

2012-04-25 Thread Aaron Plattner
On 04/10/2012 02:10 PM, Alan Coopersmith wrote: On 04/10/12 07:47 AM, Dave Airlie wrote: This patch series tears strips out of the current API, its not complete, with further changes to ddc/i2c code I can remove all references to xf86Screens from drivers. I've got a bit more work to stop them pe

[PATCH synaptics] ClickPad is most definitely a bool option.

2012-04-25 Thread Peter Hutterer
Signed-off-by: Peter Hutterer --- src/synaptics.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/synaptics.c b/src/synaptics.c index d2fe960..2a43f95 100644 --- a/src/synaptics.c +++ b/src/synaptics.c @@ -702,7 +702,7 @@ static void set_default_parameters(InputInfoPtr

Re: [PATCH 1/4] configure.ac: On MinGW force --disable-xorg and --disable-xv, and --disable-dmx by default

2012-04-25 Thread Yaakov (Cygwin/X)
On 2012-04-25 07:11, Jon TURNEY wrote: diff --git a/configure.ac b/configure.ac index 4afac82..868e086 100644 --- a/configure.ac +++ b/configure.ac @@ -715,6 +715,9 @@ case $host_os in XV=no SYMBOL_VISIBILITY=no ;; + mingw*) + XV

Re: [PATCH modular] Release.sh: batch release and autotagging of modules

2012-04-25 Thread Gaetan Nadon
On 12-04-24 10:49 PM, Alan Coopersmith wrote: > On 02/ 7/12 03:58 PM, Gaetan Nadon wrote: >> Features: >> - Visit the supplied paths to git modules >> - Accepts a list of paths in the format produced by build.sh -L >> - Tag and version names picked-up from Makefile >> - Determine the web section fr

Re: [PATCH] Include on Solaris for bzero definition

2012-04-25 Thread Alan Coopersmith
On 04/25/12 01:59 PM, Guillem Jover wrote: > On Sat, 2012-02-25 at 10:29:49 -0800, Alan Coopersmith wrote: >> SVR4 OS'es put the ANSI standard funcs (memcpy, memset, etc) in >> and the BSD compatibility funcs (bcopy, bzero, etc) in >> >> If you look in you'll see other OS'es needed this as well

Re: [PATCH] Include on Solaris for bzero definition

2012-04-25 Thread Guillem Jover
On Sat, 2012-02-25 at 10:29:49 -0800, Alan Coopersmith wrote: > SVR4 OS'es put the ANSI standard funcs (memcpy, memset, etc) in > and the BSD compatibility funcs (bcopy, bzero, etc) in > > If you look in you'll see other OS'es needed this as well > in the past, but Solaris is the only one of th

Re: xserver patches - remove redundant declarations

2012-04-25 Thread Michal Suchanek
On 24 April 2012 20:44, Chase Douglas wrote: > Hi Michal, > > Either your email client or my thunderbird is munging the patch > attachments. They aren't coming across correctly here. > > However, even if they were, it is hard to review patches as attachments. > Please send them as individual emai

Re: [Tigervnc-devel] [PATCH:tigervnc-trunk] Make fb header transformations work with Xorg 1.12.1

2012-04-25 Thread Pierre Ossman
On Sat, 14 Apr 2012 14:10:20 -0700 Alan Coopersmith wrote: > The reformatting of all the Xserver sources in the 1.12.1 release changed > the headers so now some of them have multiple instances of the C++ "and" > and "xor" keywords on the same line, so the hack to sanitize them needs > to use the

Re: [PATCH 0/4] Configure fixes for Windows MinGW target

2012-04-25 Thread Michal Suchanek
Hello, what else needs to be patched to build on mingw? I could test this if it was a full patchset but have no opinion whasoever configure changes for platform I don't know. Thanks Michal ___ xorg-devel@lists.x.org: X.Org development Archives: http:/

Re: [PULL to discuss] Remove kdrive, Xnest, and Xvfb

2012-04-25 Thread Jon TURNEY
On 03/04/2012 22:08, Yaakov Selkowitz wrote: > On 2012-04-03 13:58, Jamey Sharp wrote: >> I'd be really interested to know how much of the Xorg DDX relies on >> POSIX and how much is straight C. The module loader is an obvious >> candidate for pain, but once you turn off stuff like the PCI layer >>

[PATCH 4/4] configure.ac: on MinGW, link with ws2_32 instead of winsock2

2012-04-25 Thread Jon TURNEY
From: Ryan Pavlik ws2_32 is the correct name for the libary (even on 64 bit Windows :-)) Reviewed-by: Jon TURNEY --- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index 4cb01f0..7c056fc 100644 --- a/configure.ac +++ b/configu

[PATCH 3/4] configure.ac: MinGW doesn't have setuid binaries either.

2012-04-25 Thread Jon TURNEY
From: Ryan Pavlik If the target platform isn't in a list of platforms we know don't use a setuid binary, we try to test if we can chown something to root. This test possibly won't give the right answer if we are cross-compiling, which is common for the MinGW target. This patch adds MinGW to the

[PATCH 2/4] configure.ac: auto-disable MITSHM if we lack IPC

2012-04-25 Thread Jon TURNEY
From: Ryan Pavlik The MITSHM extension uses SYSV IPC, but even if configure's test for IPC failed, MITSHM was still enabled by default, breaking MinGW builds by default. Unfortunately, this exposes the fact that the HAVE_SYSV_IPC test wasn't being used for anything before and so we hadn't notice

[PATCH 1/4] configure.ac: On MinGW force --disable-xorg and --disable-xv, and --disable-dmx by default

2012-04-25 Thread Jon TURNEY
Signed-off-by: Jon TURNEY --- configure.ac |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/configure.ac b/configure.ac index 4afac82..868e086 100644 --- a/configure.ac +++ b/configure.ac @@ -715,6 +715,9 @@ case $host_os in XV=no SYMBOL

[PATCH 0/4] Configure fixes for Windows MinGW target

2012-04-25 Thread Jon TURNEY
Configure fixes for Windows MinGW target. (More fixes are needed to build and work, but one thing at a time...) Jon TURNEY (1): configure.ac: On MinGW force --disable-xorg and --disable-xv, and --disable-dmx by default Ryan Pavlik (3): configure.ac: auto-disable MITSHM if we lack IPC co

Re: [Xcb] xb/proto portion of [PULL v3] XResource extension v1.2

2012-04-25 Thread Erkki Seppala
On 24.04.2012 15:57, Peter Harris wrote: On Tue, Apr 24, 2012 at 6:35 AM, Erkki Seppala wrote: / + length 3 4 or not, hopefully someone can chime in. If it really is a list of CARD32s, with the length given in bytes (unlike most of the re