Re: [PATCH 09/10] xfree86: remove xf86EnableAccess

2010-05-06 Thread Mark Kettenis
From: Keith Packard kei...@keithp.com Date: Wed, 05 May 2010 20:21:29 -0700 On Wed, 5 May 2010 23:57:00 +0200 (CEST), Mark Kettenis mark.kette...@xs4all.nl wrote: You can probabaly use a version smaller than 9 as there have been a few ABI bumps since xf86EnableAccess() became a no-op.

RE: How the Server do the SW rendering?

2010-05-06 Thread Huang, FrankR
I used a gtk progressbar application to trace the X server. I found that the driver’s composite function will be triggered several times. How the client to do the XRenderComposite()? Several times to do so ? Thanks, Frank From:

Re: [PATCH xinit] Use AC_PROG_SED to find sed

2010-05-06 Thread Julien Cristau
On Wed, May 5, 2010 at 23:55:43 -0500, Yaakov (Cygwin/X) wrote: From: Yaakov Selkowitz yselkow...@users.sourceforge.net AC_PROG_SED sets SED as the path to a fully-functional 'sed' (which may also be called 'gsed' if GNU sed is installed alongside a proprietary version). Signed-off-by:

X-JHBuild: A thin JHBuild wrapper with plug-ins.

2010-05-06 Thread Dirk Wallenstein
I wasn't aware of the benefits of JHBuild for a long time. Therefore, to also address people like the former-me: Why would you want to use JHBuild and X-JHBuild? JHBuild offers general and per-module settings for configure switches and make

Re: X-JHBuild: A thin JHBuild wrapper with plug-ins.

2010-05-06 Thread Joel Feiner
On Thu, May 6, 2010 at 6:36 AM, Dirk Wallenstein hals...@t-online.dewrote: snip Layout -- The attached archive contains the following tree structure that has to be unpacked and placed in you home directory. ~/.x-jhbuild |-- bin | `-- x-jhbuild |-- config |

Re: X-JHBuild: A thin JHBuild wrapper with plug-ins.

2010-05-06 Thread Dirk Wallenstein
On Thu, May 06, 2010 at 09:13:10AM -0400, Joel Feiner wrote: On Thu, May 6, 2010 at 6:36 AM, Dirk Wallenstein hals...@t-online.dewrote: snip Layout -- The attached archive contains the following tree structure that has to be unpacked and placed in you home directory.

Re: [PATCH 0/2] Composite window resize performance tweaks

2010-05-06 Thread Adam Jackson
On Wed, 2010-05-05 at 16:25 -0400, Adam Jackson wrote: Uncomposited: 1.8 sec Composited, unpatched:12.4 sec Composited, patched: 9.3 sec Which is decent. Note that the uncomposited number is actually kind of depressing, 9ms for a synchronous resize is rather a lot of a

[PATCHv2] dix: add 3x3 transformation matrix xinput property for multi-head handling

2010-05-06 Thread Peter Korsgaard
For absolute input devices (E.G. touchscreens) in multi-head setups, we need a way to bind the device to an randr output. This adds the infrastructure to the server to allow us to do so. positionSprite() scales input coordinates to the dimensions of the shared (total) screen frame buffer, so to

Re: [PATCH 2/2] composite: Don't backfill non-MapWindow allocations

2010-05-06 Thread Adam Jackson
On Wed, 2010-05-05 at 17:20 -0700, Aaron Plattner wrote: Just to make sure I understand this right, does this work because the compReallocPixmap callers always go through the compCopyWindow path and use CopyArea to initialize the new pixmap? It would be nice if that were described in the

Re: [PATCH] xf86: allow for no outputs connected at startup operation.

2010-05-06 Thread Adam Jackson
On Thu, 2010-05-06 at 13:18 +1000, Dave Airlie wrote: From: Dave Airlie airl...@redhat.com When nothing is connected at startup and we canGrow, allow the server to start with a 1024x768 framebuffer, and when the drivers send hotplug events this will expand to the correct size dynamically.

Re: [PATCHv2] dix: add 3x3 transformation matrix xinput property for multi-head handling

2010-05-06 Thread Éric Piel
Op 06-05-10 16:34, Peter Korsgaard schreef: For absolute input devices (E.G. touchscreens) in multi-head setups, we need a way to bind the device to an randr output. This adds the infrastructure to the server to allow us to do so. positionSprite() scales input coordinates to the dimensions

Re: [PATCHv2] dix: add 3x3 transformation matrix xinput property for multi-head handling

2010-05-06 Thread Peter Korsgaard
Éric == Éric Piel e.a.b.p...@tudelft.nl writes: Éric Hi, Éric I really like the idea of the patch. Do I understand correctly Éric that the transformation applies only to absolute input devices? Éric If so, it would still be a nice step towards allowing rotation of Éric touchpads (and

Re: [PULL libXdmcp] Cleanups: getting rid of X{alloc, free, realloc}, others

2010-05-06 Thread Alan Coopersmith
Mikhail Gusarov wrote: The following changes since commit 482b19329fac5e311fe0423e58f3e8c573b66114: Gaetan Nadon (1): config: update AC_PREREQ statement to 2.60 are available in the git repository at: git://anongit.freedesktop.org/~dottedmag/libXdmcp cleanups I've pushed to

Re: [PATCHv2] dix: add 3x3 transformation matrix xinput property for multi-head handling

2010-05-06 Thread Keith Packard
On Thu, 06 May 2010 16:58:22 +0200, Éric Piel e.a.b.p...@tudelft.nl wrote: It would only require to find the best way to apply the matrix on the values received from relative devices, right? It seems like the only sensible plan for a projective transform would be to apply the transform after

Re: [PATCH] xf86: allow for no outputs connected at startup operation.

2010-05-06 Thread Keith Packard
On Thu, 06 May 2010 10:40:34 -0400, Adam Jackson a...@nwnk.net wrote: It keeps the logic out of the drivers, which I strongly approve of. Where does this leave you after startup? Framebuffer with no CRTCs connected seems logical enough, but it might not be something the drivers are prepared

Re: [PATCH 2/2] composite: Don't backfill non-MapWindow allocations

2010-05-06 Thread Aaron Plattner
On Thu, May 06, 2010 at 07:34:58AM -0700, Adam Jackson wrote: On Wed, 2010-05-05 at 17:20 -0700, Aaron Plattner wrote: Just to make sure I understand this right, does this work because the compReallocPixmap callers always go through the compCopyWindow path and use CopyArea to initialize the

[PATCH 1/3] Introduce dixLookupFontable for FONT or GC parameters.

2010-05-06 Thread Jamey Sharp
Signed-off-by: Jamey Sharp ja...@minilop.net --- Xext/xf86bigfont.c | 14 ++ dix/dispatch.c | 26 -- dix/dixutils.c | 18 ++ glx/xfont.c| 17 +++-- include/dix.h |6 ++ 5 files changed, 33

[PATCH 3/3] Return BadPicture, not BadPixmap, if alpha-map is invalid.

2010-05-06 Thread Jamey Sharp
I believe this is what was intended. Signed-off-by: Jamey Sharp ja...@minilop.net --- render/picture.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/render/picture.c b/render/picture.c index 18bfea2..48693b8 100644 --- a/render/picture.c +++ b/render/picture.c @@

[PATCH 2/3] VERIFY_PICTURE always returns BadPicture. Don't bother specifying.

2010-05-06 Thread Jamey Sharp
Same goes for VERIFY_ALPHA, VERIFY_XIN_PICTURE, and VERIFY_XIN_ALPHA. Signed-off-by: Jamey Sharp ja...@minilop.net --- hw/dmx/dmxpict.c|6 +-- render/picturestr.h |8 ++-- render/render.c | 131 +- xfixes/region.c |6 +-- 4

[PATCH resend] trivial unreviewed patches

2010-05-06 Thread Jamey Sharp
I found these cleanups while trying to simplify the use of dixLookupResourceByType, but I think they stand on their own and I thought they'd be easy to review. Anyone? Jamey ___ xorg-devel@lists.x.org: X.Org development Archives:

Re: [PATCH 1/3] Introduce dixLookupFontable for FONT or GC parameters.

2010-05-06 Thread Julien Cristau
On Thu, May 6, 2010 at 11:00:37 -0700, Jamey Sharp wrote: Signed-off-by: Jamey Sharp ja...@minilop.net Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___ xorg-devel@lists.x.org: X.Org development Archives:

Re: [PATCH 2/2] composite: Don't backfill non-MapWindow allocations

2010-05-06 Thread Adam Jackson
On Thu, 2010-05-06 at 10:47 -0700, Aaron Plattner wrote: On Thu, May 06, 2010 at 07:34:58AM -0700, Adam Jackson wrote: On Wed, 2010-05-05 at 17:20 -0700, Aaron Plattner wrote: Just to make sure I understand this right, does this work because the compReallocPixmap callers always go through

Re: [PATCH 2/3] VERIFY_PICTURE always returns BadPicture. Don't bother specifying.

2010-05-06 Thread Julien Cristau
On Thu, May 6, 2010 at 11:00:38 -0700, Jamey Sharp wrote: Same goes for VERIFY_ALPHA, VERIFY_XIN_PICTURE, and VERIFY_XIN_ALPHA. Signed-off-by: Jamey Sharp ja...@minilop.net Reviewed-by: Julien Cristau jcris...@debian.org Cheers, Julien ___

Re: [PATCH 09/10] xfree86: remove xf86EnableAccess

2010-05-06 Thread Vignatti Tiago (Nokia-D/Helsinki)
On Thu, May 06, 2010 at 09:58:25AM +0200, ext Mark Kettenis wrote: From: Keith Packard kei...@keithp.com Date: Wed, 05 May 2010 20:21:29 -0700 On Wed, 5 May 2010 23:57:00 +0200 (CEST), Mark Kettenis mark.kette...@xs4all.nl wrote: You can probabaly use a version smaller than 9 as

Re: [PATCH resend] trivial unreviewed patches

2010-05-06 Thread Adam Jackson
On Thu, 2010-05-06 at 11:00 -0700, Jamey Sharp wrote: I found these cleanups while trying to simplify the use of dixLookupResourceByType, but I think they stand on their own and I thought they'd be easy to review. Anyone? Reviewed-by: Adam Jackson a...@redhat.com for the series. - ajax

[PATCH] modes: Combine xf86DefaultModes and DMTModes

2010-05-06 Thread Adam Jackson
DMTModes is what xf86DefaultModes was always meant to be. Smash the two together, leaving aliases in place for the old names to make it easy on external modules. Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/common/Makefile.am| 10 +--- hw/xfree86/common/extramodes | 27

[PATCH] clientErrorValue is never used outside dix. Stop importing it.

2010-05-06 Thread Jamey Sharp
Signed-off-by: Jamey Sharp ja...@minilop.net --- Xext/panoramiXprocs.c |7 --- Xext/xvmain.c |2 -- glx/xfont.c |2 -- 3 files changed, 0 insertions(+), 11 deletions(-) diff --git a/Xext/panoramiXprocs.c b/Xext/panoramiXprocs.c index 6635db9..7098617 100644 ---

Re: [PATCH] xf86: allow for no outputs connected at startup operation.

2010-05-06 Thread Dave Airlie
On Fri, May 7, 2010 at 12:40 AM, Adam Jackson a...@nwnk.net wrote: On Thu, 2010-05-06 at 13:18 +1000, Dave Airlie wrote: From: Dave Airlie airl...@redhat.com When nothing is connected at startup and we canGrow, allow the server to start with a 1024x768 framebuffer, and when the drivers send

Re: [PATCH 2/2] composite: Don't backfill non-MapWindow allocations

2010-05-06 Thread Keith Packard
On Thu, 6 May 2010 10:47:29 -0700, Aaron Plattner aplatt...@nvidia.com wrote: I can't help but think that this would all be simpler if Composite could wrap PaintWindow. ;) The old pixmap will be gone by the time PaintWindow is called. And, you can't exactly keep it around as PaintWindow may

Re: [PATCH] clientErrorValue is never used outside dix. Stop importing it.

2010-05-06 Thread Keith Packard
On Thu, 6 May 2010 13:12:03 -0700, Jamey Sharp ja...@minilop.net wrote: -/* Various of the DIX function interfaces were not designed to allow - * the client-errorValue to be set on BadValue and other errors. - * Rather than changing interfaces and breaking untold code we introduce - * a new

[PATCH] mi: Delete wide ellipse arc cache

2010-05-06 Thread Adam Jackson
This does make wide ellipses slower, by a factor of twoish: 946000.0 521000.0 ( 0.55) 10-pixel wide ellipse 98300.049900.0 ( 0.51) 100-pixel wide ellipse 17700.0 9310.0 ( 0.53) 500-pixel wide ellipse 16900.0 7980.0 ( 0.47) 100-pixel wide dashed ellipse 16100.0

Re: Proposal for per-CRTC pixmaps in RandR

2010-05-06 Thread Keith Packard
Here's a patch to randrproto.txt that specifies the protocol level semantics for the proposed changes: From 1265b555ebc49e7e5bc23de7441e7686748e97e1 Mon Sep 17 00:00:00 2001 From: Keith Packard kei...@keithp.com Date: Thu, 6 May 2010 15:11:52 -0700 Subject: [PATCH] Proposed RandR changes for

[PATCH: libXdmcp] Use stdint.h instead of limits.h for UINT*_MAX definitions

2010-05-06 Thread Alan Coopersmith
While Solaris allows either one, C99 only requires them in stdint.h and some platforms don't include them via limits.h Corrects tinderbox reported errors on Fedora 11 build machine: http://tinderbox.x.org/builds/2010-05-06-0019/logs/libXdmcp/#build AA32.c: In function 'XdmcpAllocARRAY32':

Re: [PATCH: libXdmcp] Use stdint.h instead of limits.h for UINT*_MAX definitions

2010-05-06 Thread Alan Coopersmith
Alan Coopersmith wrote: While Solaris allows either one, C99 only requires them in stdint.h and some platforms don't include them via limits.h BTW, again I only compile tested on OpenSolaris, so if someone wants to verify this fixes the build problem on Linux, that would be good. (I've got a

Re: [PATCH] mi: Delete wide ellipse arc cache

2010-05-06 Thread Keith Packard
On Thu, 6 May 2010 17:58:29 -0400, Adam Jackson a...@redhat.com wrote: This does make wide ellipses slower, by a factor of twoish: Signed-off-by: Adam Jackson a...@redhat.com Reviewed-by: Keith Packard kei...@keithp.com -- keith.pack...@intel.com pgpKjrKM66hVc.pgp Description: PGP

Re: [PATCH: libXdmcp] Use stdint.h instead of limits.h for UINT*_MAX definitions

2010-05-06 Thread Gaetan Nadon
On Thu, 2010-05-06 at 15:42 -0700, Alan Coopersmith wrote: Alan Coopersmith wrote: While Solaris allows either one, C99 only requires them in stdint.h and some platforms don't include them via limits.h BTW, again I only compile tested on OpenSolaris, so if someone wants to verify this

[PATCH synaptics 00/12] Cleanup patches

2010-05-06 Thread Peter Hutterer
There's not a lot of functional change in this series, it's mostly grabbing some blocks and moving them into their own function to make the code easier to read and understand. The only actual feature change is in the last patch, the up/down and left/right scroll buttons are made conditional on

[PATCH synaptics 01/12] Add FIXME: where does the 13 come from?

2010-05-06 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/synaptics.c b/src/synaptics.c index 2f432ce..78eb03f 100644 --- a/src/synaptics.c +++ b/src/synaptics.c @@ -1613,6 +1613,7 @@

[PATCH synaptics 02/12] Change condition the finger requirement obvious.

2010-05-06 Thread Peter Hutterer
The need for a finger down is for all conditions, move it to the front of the condition to make it more obvious. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/synaptics.c

[PATCH synaptics 03/12] Move SHM hardware state update into a separate function.

2010-05-06 Thread Peter Hutterer
No functional changes, this is just to move a slab of code out of mind when reading. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c | 56 -- 1 files changed, 33 insertions(+), 23 deletions(-) diff --git

[PATCH synaptics 04/12] Factor out up/down and left/right button scrolling from HandleState.

2010-05-06 Thread Peter Hutterer
There are not a lot of touchpads that even have these buttons anyway, move the code out of the way for clarity. No functional changes. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c | 65 +- 1 files changed, 40

[PATCH synaptics 05/12] When scrolling, hardcode the button/press release cycle.

2010-05-06 Thread Peter Hutterer
Not sure why the values for !hw-up and hw-up were used here instead, but this could potentially lead to unreleased buttons. Also, it's harder to read and generally confusing, since the same wasn't used for left/right scrolling. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net ---

[PATCH synaptics 06/12] Move scroll button handling out of HandleState.

2010-05-06 Thread Peter Hutterer
Again, just to make the code easier to read, no functional changes. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c | 44 ++-- 1 files changed, 26 insertions(+), 18 deletions(-) diff --git a/src/synaptics.c b/src/synaptics.c

[PATCH synaptics 07/12] Use post_button_click to post doubleclicks.

2010-05-06 Thread Peter Hutterer
Function introduced with the factoring out of post_scroll_events. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c | 16 +--- 1 files changed, 5 insertions(+), 11 deletions(-) diff --git a/src/synaptics.c b/src/synaptics.c index c3a4ebc..9a286b5 100644

[PATCH synaptics 08/12] Factor out updating the hardware state.

2010-05-06 Thread Peter Hutterer
No functional changes. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c | 55 --- 1 files changed, 32 insertions(+), 23 deletions(-) diff --git a/src/synaptics.c b/src/synaptics.c index 9a286b5..650fbc2 100644 ---

[PATCH synaptics 09/12] Rename HandleClickWithFingers to handle_clickfinger.

2010-05-06 Thread Peter Hutterer
The option is called ClickFinger, the man page talks this way about it, sync up the code with it. No functional change. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/synaptics.c

[PATCH synaptics 10/12] Add NO_EDGE to the edge enum instead of using hardcoded 0.

2010-05-06 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/synaptics.c b/src/synaptics.c index aa3e848..907335a 100644 --- a/src/synaptics.c +++ b/src/synaptics.c @@ -83,6 +83,7 @@ #endif typedef

[PATCH synaptics 11/12] move history storage from ComputeDeltas into HandleState.

2010-05-06 Thread Peter Hutterer
It's well hidden in there, but not useful. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- src/synaptics.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/synaptics.c b/src/synaptics.c index 907335a..88c5237 100644 --- a/src/synaptics.c +++

[PATCH synaptics 12/12] Make scrollbuttons processing and property conditional on their existence.

2010-05-06 Thread Peter Hutterer
There are not a lot of touchpads that have extra physical scroll buttons anymore. For those that don't have them, don't initalize the properties and conditionalize some of the code (moved into its own functions). Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- man/synaptics.man |