Re: [PATCH] Remove leftover fbSaveAreas fbRestoreAreas defines from wfbrename.h

2010-10-06 Thread Jeremy Huddleston
Checked grep to be sure... Reviewed-by: Jeremy Huddleston jerem...@apple.com Tested-by: Jeremy Huddleston jerem...@apple.com On Oct 5, 2010, at 18:05, Alan Coopersmith wrote: The fb functions they try to rename were deleted in 2007 by commit ae7f71a8b3d6756161e55d998d6eec37d2695c98

Re: patch:libXt-1.0.8 fix possible NULL access

2010-10-06 Thread walter harms
Alan Coopersmith schrieb: walter harms wrote: You appear to have attached a different version of the patch than in the body of your e-mail - neither git nor patch can apply either one to the current git tree. After a bit of manual editing I finally got one to work, so I fixed the

[PATCH libXi] Fix typo when converting raw events from the wire.

2010-10-06 Thread Carlos Garnacho
From: Carlos Garnacho carl...@gnome.org The raw values were being miscalculated, containing only the integral part of the FP3232, meanwhile normal valuators were mistakenly added the fractional part of its corresponding raw value. Signed-off-by: Carlos Garnacho carl...@gnome.org ---

[PATCH] DRI2: Add ReuseBufferNotify hook

2010-10-06 Thread Pauli Nieminen
ReuseBufferNotify hook is called whenever old buffer is reused in DRI2 code. Driver can use this hook to rewrite the buffer name if hardware requires shared buffers. Shared buffer might be some hardware limited resources like framebuffer that is preallocated in boot. Signed-off-by: Pauli

[PATCH] DRI2: Expose API to set drawable swap limit.

2010-10-06 Thread Pauli Nieminen
This allows ddx to set swap_limit if there is more than one back buffer for drawable. Setting swap_limit has to also check if change affects a client that is blocked. This can be used to implement N-buffering in driver with minimal logic in allocation and selecting next back. Signed-off-by:

[PATCH 1/2] DRI2: Avoid call to NULL pointer

2010-10-06 Thread Pauli Nieminen
DDX driver may implement schedule swap without GetMSC. In that case we can't call GetMSC in DRI2SwapBuffers. Signed-off-by: Pauli Nieminen ext-pauli.niemi...@nokia.com --- hw/xfree86/dri2/dri2.c | 11 +++ 1 files changed, 7 insertions(+), 4 deletions(-) diff --git

[PATCH 2/2] DRI2: Add error message when working around driver bug

2010-10-06 Thread Pauli Nieminen
There isn't API that allows application atomically query for msc changes and schedule swaps. If msc changes dramatically between query and scheduling application would schedule swap to happen at wrong time. Because of API limitations driver has to make msc increment for each vblank affecting the

[PATCH] render: repack PictureRec

2010-10-06 Thread Adam Jackson
Eliminate the unused dither field, move filter and stateChanges into the bitfield, and reorder elements to pack holes on LP64. sizeof(PictureRec) ILP32 LP64 before:84152 after: 72120 Signed-off-by: Adam Jackson a...@redhat.com

Re: Disabling monitors with bad EDIDs?!?!

2010-10-06 Thread Richard Hughes
On 24 September 2010 07:07, Kai-Uwe Behrmann k...@gmx.de wrote: Manufacturers have typical good colour measurement equipement and appear to put some reasonable colorimetric data inside the EDID. The colorimetric precission is of course rough, given that the gamma curves are represented by just

Re: Disabling monitors with bad EDIDs?!?!

2010-10-06 Thread Kai-Uwe Behrmann
Am 06.10.10, 16:54 +0100 schrieb Richard Hughes: On 24 September 2010 07:07, Kai-Uwe Behrmann k...@gmx.de wrote: Manufacturers have typical good colour measurement equipement and appear to put some reasonable colorimetric data inside the EDID. The colorimetric precission is of course rough,

Re: [PATCH libXi] Fix typo when converting raw events from the wire.

2010-10-06 Thread Jeremy Huddleston
Reviewed-by: Jeremy Huddleston jerem...@apple.com On Oct 6, 2010, at 02:04, Carlos Garnacho wrote: From: Carlos Garnacho carl...@gnome.org The raw values were being miscalculated, containing only the integral part of the FP3232, meanwhile normal valuators were mistakenly added the

[PATCH] Xorg restarted each session after some activity (20-30 minutes).

2010-10-06 Thread Sebasian Glita
From: Sebastian Glita gs...@cs.upt.ro A simple fixup in dixutils.c:_CallCallbacks: - by moving recursion count check *before* list iteration and actual callbacks' invocation; - can say for sure, though, whether counters (member `int inCallback' from include/dixstruct.h:171:CallbackListRec

Re: [PATCH modular] Process a user-supplied list of module/components.

2010-10-06 Thread Gaetan Nadon
On Tue, 2010-10-05 at 17:35 -0400, Trevor Woerner wrote: From: Trevor Woerner twoer...@gmail.com New feature. Using the new --modlist file option, the user can specify a file which contains a list of the module/component items to process. The file can contain blank lines and comment

Re: [PATCH modular] Process a user-supplied list of module/components.

2010-10-06 Thread Trevor Woerner
On Wed, Oct 6, 2010 at 3:59 PM, Gaetan Nadon mems...@videotron.ca wrote: On Tue, 2010-10-05 at 17:35 -0400, Trevor Woerner wrote: The script attempts to process any unknown items at the end of the known list. Any reasons why this is better for the user? I think 99 out of a 100 users will

Re: [PATCH modular] Process a user-supplied list of module/components.

2010-10-06 Thread Trevor Woerner
On Wed, Oct 6, 2010 at 4:16 PM, Trevor Woerner twoer...@gmail.com wrote: Shouldn't all the projects be known to the build.sh script? Whoops, one more thought I forgot to add: how would the script differentiate between an unlisted sub-project the user wants the script to build and a known

Re: [PATCH modular] Process a user-supplied list of module/components.

2010-10-06 Thread Gaetan Nadon
On Wed, 2010-10-06 at 16:16 -0400, Trevor Woerner wrote: I agree... but to be honest I think the more pertinent question is: why would a user be trying to build unknown sub-projects? Shouldn't all the projects be known to the build.sh script? Should the script even support unknown projects?

[PATCH modular] build.sh usage text: remove annoying backslashes

2010-10-06 Thread Gaetan Nadon
This is text intended for the user to read, not the shell interpreter. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- build.sh | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/build.sh b/build.sh index 1e31c2a..f9b581d 100755 --- a/build.sh +++ b/build.sh

Re: [PATCH modular] Process a user-supplied list of module/components.

2010-10-06 Thread Trevor Woerner
On Wed, Oct 6, 2010 at 5:21 PM, Gaetan Nadon mems...@videotron.ca wrote: I could be wrong, but I thought that having a custom list would go a long way in making build.sh more useful. That's why I still don't think that sorting a list of modules to build them in order in terribly useful. The

Re: [PATCH modular] build.sh usage text: remove annoying backslashes

2010-10-06 Thread Trevor Woerner
On Wed, Oct 6, 2010 at 5:30 PM, Gaetan Nadon mems...@videotron.ca wrote: This is text intended for the user to read, not the shell interpreter. Yes, that's why there are backslashes, so the interpreter doesn't replace them when displaying the help text. For example the user is told they can

Re: [PATCH modular] Process a user-supplied list of module/components.

2010-10-06 Thread Gaetan Nadon
On Wed, 2010-10-06 at 17:37 -0400, Trevor Woerner wrote: The fact is you have a lot more experience with this stuff than I do. So if you say building them in the user-specified order is more important than sorting them, I can easily create a new patch which implements this behaviour. And

Re: [PATCH 2/2] DRI2: Add error message when working around driver bug

2010-10-06 Thread Mario Kleiner
On Oct 6, 2010, at 1:05 PM, Pauli Nieminen wrote: There isn't API that allows application atomically query for msc changes and schedule swaps. If msc changes dramatically between query and scheduling application would schedule swap to happen at wrong time. Because of API limitations driver

Re: [PATCH modular] build.sh usage text: remove annoying backslashes

2010-10-06 Thread Gaetan Nadon
On Wed, 2010-10-06 at 17:46 -0400, Trevor Woerner wrote: On Wed, Oct 6, 2010 at 5:30 PM, Gaetan Nadon mems...@videotron.ca wrote: This is text intended for the user to read, not the shell interpreter. Yes, that's why there are backslashes, so the interpreter doesn't replace them when

Re: [PATCH 1/2] DRI2: Avoid call to NULL pointer

2010-10-06 Thread Mario Kleiner
On Oct 6, 2010, at 1:05 PM, Pauli Nieminen wrote: DDX driver may implement schedule swap without GetMSC. In that case we can't call GetMSC in DRI2SwapBuffers. I don't think this check is neccessary. Afaik if the ds-GetMSC entry point isn't defined then ds-ScheduleSwap isn't defined either,

Re: [PATCH libXi] Fix typo when converting raw events from the wire.

2010-10-06 Thread Peter Hutterer
On Wed, Oct 06, 2010 at 12:27:15PM -0700, Jeremy Huddleston wrote: Reviewed-by: Jeremy Huddleston jerem...@apple.com On Oct 6, 2010, at 02:04, Carlos Garnacho wrote: From: Carlos Garnacho carl...@gnome.org The raw values were being miscalculated, containing only the integral part

[PATCH font-util] fontutil.m4: Add XORG_FONT_FC_CONFDIR to find fontconfig's confdir

2010-10-06 Thread Jeremy Huddleston
Signed-off-by: Jeremy Huddleston jerem...@apple.com --- fontutil.m4.in | 34 ++ 1 files changed, 34 insertions(+), 0 deletions(-) diff --git a/fontutil.m4.in b/fontutil.m4.in index c867ea6..b9ad59c 100644 --- a/fontutil.m4.in +++ b/fontutil.m4.in @@ -256,6

[PATCH bh-ttf] Use XORG_FONT_FC_CONFDIR from font-util 1.2 to find fontconfig's confdir

2010-10-06 Thread Jeremy Huddleston
Signed-off-by: Jeremy Huddleston jerem...@apple.com --- Makefile.am |2 +- configure.ac |7 --- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/Makefile.am b/Makefile.am index 34d9819..f1fc0cc 100644 --- a/Makefile.am +++ b/Makefile.am @@ -36,7 +36,7 @@ FONT_FILES = \