Re: Spinning in _XReply

2011-02-23 Thread Rami Ylimäki
On 02/22/2011 11:26 PM, xm...@karlt.net wrote: Although wrong, our code has been working well enough for us to get useful information. If someone is able to point out a reason why this now works less successfully than previously, then that would bump up the priority at our end. By

Re: Current tinderbox regression (xserver, randr)

2011-02-23 Thread Colin Harrison
Hi, I think Keith was waiting for someone to review this patch... http://lists.x.org/archives/xorg-devel/2011-February/019390.html and RandR 1.4 fixes... http://lists.x.org/archives/xorg-devel/2011-February/019389.html The patch makes the code compile for me (MinGW for Windows)..but I

Re: [PATCH] Unbreak the .m.c rule

2011-02-23 Thread Paulo Zanoni
Great! Now I can build xts on newer distros =) Tested-by: Paulo Zanoni pzan...@mandriva.com 2011/2/23 Peter Hutterer peter.hutte...@who-t.net: On Tue, Feb 22, 2011 at 05:57:26PM -0500, Adam Jackson wrote: Some builds of gnu make apparently know how to go straight from .m to .o. That's great

Re: [PATCH] Unbreak the .m.c rule

2011-02-23 Thread Dan Nicholson
On Tue, Feb 22, 2011 at 9:04 PM, Peter Hutterer peter.hutte...@who-t.net wrote: On Tue, Feb 22, 2011 at 05:57:26PM -0500, Adam Jackson wrote: Some builds of gnu make apparently know how to go straight from .m to .o. That's great if your .m file is Objective C source, but for us, it's not.

Re: Spinning in _XReply

2011-02-23 Thread Michal Suchanek
On 23 February 2011 11:15, Rami Ylimäki rami.ylim...@vincit.fi wrote: On 02/22/2011 11:26 PM, xm...@karlt.net wrote: Although wrong, our code has been working well enough for us to get useful information.  If someone is able to point out a reason why this now works less successfully than

[PATCH] libXfixes v5: Pointer barriers

2011-02-23 Thread Adam Jackson
Signed-off-by: Adam Jackson a...@redhat.com --- configure.ac|2 +- include/X11/extensions/Xfixes.h | 15 ++ src/Cursor.c| 58 +++ 3 files changed, 74 insertions(+), 1 deletions(-) diff --git

[PATCH] dri2: Initialize padding in DRI2GetBuffers reply

2011-02-23 Thread Pauli
From: Pauli Nieminen ext-pauli.niemi...@nokia.com This fixes following valgrind reported error about padding being uninitialised. ==== Syscall param writev(vector[...]) points to uninitialised byte(s) ====at 0x4AB7054: writev (writev.c:51) ====by 0x5E237: _XSERVTransWritev

Re: [PATCH] Unbreak the .m.c rule

2011-02-23 Thread Adam Jackson
On 2/23/11 8:53 AM, Dan Nicholson wrote: Peter, did we discuss some other way to fix this? Either way, this looks like the right thing to do. Reviewed-by: Dan Nicholsondbn.li...@gmail.com Pushed, thanks all. - ajax ___ xorg-devel@lists.x.org:

Re: [Patch 0/8] Pointer barriers

2011-02-23 Thread Adam Jackson
On 2/21/11 10:51 PM, Peter Hutterer wrote: Notes on the PB support in general: - having a pointer barrier specify which direction it doesn't work is very confusing. Both from the client-side and whithin the server it was counter-intuitive. I'd prefer an interface like this

Re: [PATCH] [xserver] record: avoid crash when calling RecordFlushReplyBuffer recursively

2011-02-23 Thread Keith Packard
On Mon, 14 Feb 2011 19:59:13 +0200, Oliver McFadden oliver.mcfad...@nokia.com wrote: I don't think this has been merged into master yet (and we should apply it to the 1.9.xxx branches too), so I've added my Reviewed-by below. Good catch! I'll merge this to master for 1.10. --

Re: [PATCH] randr: Add 'set' fields to SetCrtcConfigs request

2011-02-23 Thread Aaron Plattner
On Wed, Feb 16, 2011 at 10:59:22PM -0800, Keith Packard wrote: This permits clients to perform incremental configuration changes instead of requiring a complete new configuration to be written. Signed-off-by: Keith Packard kei...@keithp.com --- randr/rrcrtc.c | 332

[PULL] 1.10: warning fixes, ABI fix, and minor Fedora merge

2011-02-23 Thread ajax
The following changes since commit 93a73993708b1345c86ec3ec06b02ed236595673: test: write some event → XI1 conversion tests. (2011-02-22 08:08:55 +1000) are available in the git repository at: ssh://people.freedesktop.org/~ajax/xserver ..BRANCH.NOT.VERIFIED.. Adam Jackson (13):

[PULL] 1.10: warning fixes, ABI fix, and minor Fedora merge

2011-02-23 Thread ajax
The following changes since commit 93a73993708b1345c86ec3ec06b02ed236595673: test: write some event → XI1 conversion tests. (2011-02-22 08:08:55 +1000) are available in the git repository at: ssh://people.freedesktop.org/~ajax/xserver for-keithp Adam Jackson (13): glxproxy: warning

Re: [PATCH] Unbreak the .m.c rule

2011-02-23 Thread Peter Hutterer
On Wed, Feb 23, 2011 at 05:53:57AM -0800, Dan Nicholson wrote: On Tue, Feb 22, 2011 at 9:04 PM, Peter Hutterer peter.hutte...@who-t.net wrote: On Tue, Feb 22, 2011 at 05:57:26PM -0500, Adam Jackson wrote: Some builds of gnu make apparently know how to go straight from .m to .o. That's

Re: [PATCH] randr: Add 'set' fields to SetCrtcConfigs request

2011-02-23 Thread Luc Verhaegen
On Wed, Feb 23, 2011 at 10:16:46AM -0800, Aaron Plattner wrote: On Wed, Feb 16, 2011 at 10:59:22PM -0800, Keith Packard wrote: Other than the protocol doc comments and the existing bug noted above, Reviewed-by: Aaron Plattner aplatt...@nvidia.com However, I do need to object strongly to

Re: [PATCH] libXfixes v5: Pointer barriers

2011-02-23 Thread Julien Cristau
On Wed, Feb 23, 2011 at 11:52:02 -0500, Adam Jackson wrote: diff --git a/include/X11/extensions/Xfixes.h b/include/X11/extensions/Xfixes.h index 0c00310..481531c 100644 --- a/include/X11/extensions/Xfixes.h +++ b/include/X11/extensions/Xfixes.h @@ -1,5 +1,6 @@ /* * Copyright (c) 2006,

Re: Spinning in _XReply

2011-02-23 Thread xmail
Rami Ylimäki writes: On 02/22/2011 11:26 PM, xm...@karlt.net wrote: Although wrong, our code has been working well enough for us to get useful information. If someone is able to point out a reason why this now works less successfully than previously, then that would bump up the priority at

Re: [PATCH] randr: Add 'set' fields to SetCrtcConfigs request

2011-02-23 Thread Keith Packard
On Wed, 23 Feb 2011 10:16:46 -0800, Aaron Plattner aplatt...@nvidia.com wrote: randrproto.txt says nothing about SetCrtcMode. ... randrproto.txt says nothing about SetCrtcOutputs. The interplay encoded here probably ought to be described in the protocol doc. Yeah, randrproto.txt is missing

Re: [PATCH] randr: Add 'set' fields to SetCrtcConfigs request

2011-02-23 Thread Keith Packard
On Wed, 23 Feb 2011 23:12:15 +0100, Luc Verhaegen l...@skynet.be wrote: I believe this all got commited a while ago. There was even a day or so when the proto and xserver were out of sync, and there was a tinderbox complaint and at least one bug. Only a single SOB was present. I just

Re: [PATCH] randr: Add 'set' fields to SetCrtcConfigs request

2011-02-23 Thread Luc Verhaegen
On Wed, Feb 23, 2011 at 03:49:58PM -0800, Keith Packard wrote: On Wed, 23 Feb 2011 23:12:15 +0100, Luc Verhaegen l...@skynet.be wrote: I believe this all got commited a while ago. There was even a day or so when the proto and xserver were out of sync, and there was a tinderbox complaint

Re: [PULL] 1.10: warning fixes, ABI fix, and minor Fedora merge

2011-02-23 Thread Peter Hutterer
On Wed, Feb 23, 2011 at 01:48:23PM -0500, a...@redhat.com wrote: The following changes since commit 93a73993708b1345c86ec3ec06b02ed236595673: test: write some event → XI1 conversion tests. (2011-02-22 08:08:55 +1000) are available in the git repository at:

Re: [PULL] 1.10: warning fixes, ABI fix, and minor Fedora merge

2011-02-23 Thread Adam Jackson
On 2/23/11 6:57 PM, Peter Hutterer wrote: On Wed, Feb 23, 2011 at 01:48:23PM -0500, a...@redhat.com wrote: Peter Hutterer (1): Add mode field to ConstrainCursorHarder just in case that wasn't clear to all, this breaks the ABI. But we don't have any users for it yet anyway, so... Or

Re: [PULL] 1.10: warning fixes, ABI fix, and minor Fedora merge

2011-02-23 Thread Keith Packard
On Wed, 23 Feb 2011 13:43:54 -0500, a...@redhat.com wrote: are available in the git repository at: ssh://people.freedesktop.org/~ajax/xserver ..BRANCH.NOT.VERIFIED.. BRANCH.NOT.VERIFIED? -- keith.pack...@intel.com pgpE1vzCzY7zp.pgp Description: PGP signature

Re: [PATCH] include: document list interface.

2011-02-23 Thread Peter Hutterer
On Tue, Feb 22, 2011 at 12:56:41PM -0800, Alan Coopersmith wrote: + * Looping through the list requires a 'struct foo' as iterator and the + * name of the field the subnodes use. Looks like this should this say: a 'struct foo' as iterator, the head of the list, and the name of

[PATCH v2] include: document list interface.

2011-02-23 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- Changes since v1: typos pointed out by Alan - initilise - initialize - Seems like the word in should be deleted from that sentence. successfully found the sentence to deleted the work in from and executed said task. it shall not be