Re: [PATCH:xf86-video-ast] Use uint32_t for a variable that's passed to a function expecting a uint32_t.

2013-06-07 Thread Alan Coopersmith
On 06/ 2/13 02:36 PM, Thomas Klausner wrote: --- src/ast_vgatool.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ast_vgatool.c b/src/ast_vgatool.c index 513cf7b..378822a 100644 --- a/src/ast_vgatool.c +++ b/src/ast_vgatool.c @@ -2709,7 +2709,7 @@ void

Re: [PATCH:libX11 6/8] Check for symbol existence with #ifdef, not #if.

2013-06-07 Thread Alan Coopersmith
On 06/ 2/13 11:49 AM, Thomas Klausner wrote: --- src/XlibInt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/XlibInt.c b/src/XlibInt.c index b06e57b..92a4340 100644 --- a/src/XlibInt.c +++ b/src/XlibInt.c @@ -239,7 +239,7 @@ void _XSeqSyncFunction( static int

Re: [PATCH:libX11 8/8] Deal with the limited range of VAX floating point numbers when compiling for VAX.

2013-06-07 Thread Alan Coopersmith
On 06/ 2/13 04:19 PM, Thomas Klausner wrote: On Sun, Jun 02, 2013 at 04:11:08PM -0700, Alan Coopersmith wrote: People still compile X for VAX? I had no idea... Yes, NetBSD-6.1 was released for vax, including X. http://netbsd.org/ports/vax/ Of course, it was cross-compiled. Okay - but

Re: [PATCH:libX11 2/8] Let libX11 load and make available the newer (X11R6) callback-based API for XIM (expected by emacs).

2013-06-07 Thread Alan Coopersmith
For patches 2, 4, 5 from this series, while I have vague memories that I once knew how those files worked, I certainly don't have access to those brain cells any more, so will leave these for someone with more familiarity with the Xlib i18n bits to throw out a reviewed-by tag - I'm willing to

Re: [PATCH:xrandr 2/2] Use __UNCONST to work around const problem.

2013-06-07 Thread Alan Coopersmith
On 06/ 2/13 05:15 PM, Thomas Klausner wrote: On Mon, Jun 03, 2013 at 01:37:31AM +0200, Thomas Klausner wrote: On Sun, Jun 02, 2013 at 04:35:51PM -0700, Alan Coopersmith wrote: On 06/ 2/13 12:30 PM, Thomas Klausner wrote: XXX: should probably be fixed differently. --- xrandr.c | 2 +- 1

A clerical error in the document Xlib - C Language X Interface X11R7.7

2013-06-07 Thread ZHANG Zhaolong
hi, In this page: http://www.x.org/releases/X11R7.7/doc/libX11/libX11/libX11.html#Image_Format_Functions_and_Macros To obtain the pixmap format information for a given display, use XListPixmapFormats. -ImageByteOrder(display) - int XImageByteOrder(Display *display, int

Re: Current DRI3 specification

2013-06-07 Thread Arthur Huillet
Hello, On Tue, 04 Jun 2013 16:51:33 -0700 Keith Packard kei...@keithp.com wrote: ┌─── DRI3PixmapFromBuffer pixmap: PIXMAP drawable: DRAWABLE size: CARD32 width, height, stride: CARD16 depth, bpp: CARD8 buffer: FD └─── Errors: Alloc,

Re: [PATCH:libX11 6/8] Check for symbol existence with #ifdef, not #if.

2013-06-07 Thread Jamey Sharp
Since every other XTHREADS check in that earlier patch uses #ifdef, it looks like it was just a mistake. For this fix: Reviewed-by: Jamey Sharp ja...@minilop.net Jamey On 06/ 2/13 11:49 AM, Thomas Klausner wrote: --- src/XlibInt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH:libX11 8/8] Deal with the limited range of VAX floating point numbers when compiling for VAX.

2013-06-07 Thread Thomas Klausner
On Thu, Jun 06, 2013 at 11:14:07PM -0700, Alan Coopersmith wrote: On 06/ 2/13 04:19 PM, Thomas Klausner wrote: On Sun, Jun 02, 2013 at 04:11:08PM -0700, Alan Coopersmith wrote: People still compile X for VAX? I had no idea... Yes, NetBSD-6.1 was released for vax, including X.

Re: [PATCH] xvfb: add randr support

2013-06-07 Thread Michal Srb
On Saturday 12 of January 2013 12:55:58 Mike Frysinger wrote: From: Lambros Lambrou lambroslamb...@google.com Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=26391 Signed-off-by: Lambros Lambrou lambroslamb...@google.com Signed-off-by: Mike Frysinger vap...@gentoo.org ---

Re: [PATCH] Revert DRI2: re-allocate DRI2 drawable if pixmap serial changes

2013-06-07 Thread Chris Wilson
On Wed, Jun 05, 2013 at 12:34:24PM -0700, Eric Anholt wrote: This reverts commit 3209b094a3b1466b579e8020e12a4f3fa78a5f3f. After a long debug session by Paul Berry, it appears that this was the commit that has been producing sporadic failures in piglit front buffer rendering tests for the

[PATCH] initial aarch64 support

2013-06-07 Thread Mark Salter
Basic suport for AArch64 (64bit ARM) architecture. This is enough to get things to build. Hardware is not yet available for testing, so a more thorough port will be needed once hardware does become available. Signed-off-by: Mark Salter msal...@redhat.com --- configure.ac | 3 +++

[PATCH:libX11] specs/libX11: correct prototype for XListPixmapFormats/XImageByteOrder

2013-06-07 Thread Alan Coopersmith
The XListPixmapFormats arguments was being shown with XImageByteOrder's name and return types. Appears to have been a glitch in the nroff - docbook conversion. Reported-by: ZHANG Zhaolong zhangzl2...@126.com Signed-off-by: Alan Coopersmith alan.coopersm...@oracle.com --- specs/libX11/CH02.xml

Re: A clerical error in the document Xlib - C Language X Interface X11R7.7

2013-06-07 Thread Alan Coopersmith
On 06/ 7/13 02:00 AM, ZHANG Zhaolong wrote: hi, In this page: http://www.x.org/releases/X11R7.7/doc/libX11/libX11/libX11.html#Image_Format_Functions_and_Macros To obtain the pixmap format information for a given display, use XListPixmapFormats. -ImageByteOrder(display) - int

Re: [PATCH] xvfb: add randr support

2013-06-07 Thread Keith Packard
Mike Frysinger vap...@gentoo.org writes: +static Bool +vfbRRScreenSetSize(ScreenPtr pScreen, CARD16 width, CARD16 height, + CARD32 mmWidth, CARD32 mmHeight) +{ This can be a whole lot shorter if you use SetRootClip, take a look at hw/xwin/winrandr.c. Should be something

Re: Current DRI3 specification

2013-06-07 Thread Keith Packard
Arthur Huillet arthur.huil...@free.fr writes: Perhaps is this just me but I find The pixels [...] may not be to be ambiguous. What about: The pixels within the buffer are not required to be arranged in [...] It is slightly longer but it eliminates the ten seconds it took me to figure out

Re: Current DRI3 specification

2013-06-07 Thread Keith Packard
James Jones jajo...@nvidia.com writes: Yeah, I think the semantics are compatible. We allocate these buffers on the server-side, but I don't think that affects the interaction with Present. DRI2 allocates them server-side as well for GLX compliance, but with the way Intel hardware does

Re: Current DRI3 specification

2013-06-07 Thread James Jones
On 06/07/2013 10:10 AM, Keith Packard wrote: * PGP Signed by an unknown key James Jones jajo...@nvidia.com writes: Yeah, I think the semantics are compatible. We allocate these buffers on the server-side, but I don't think that affects the interaction with Present. DRI2 allocates them

Re: [PATCH:libX11] specs/libX11: correct prototype for XListPixmapFormats/XImageByteOrder

2013-06-07 Thread Jamey Sharp
Reviewed-by: Jamey Sharp ja...@minilop.net On Fri, Jun 07, 2013 at 09:32:31AM -0700, Alan Coopersmith wrote: The XListPixmapFormats arguments was being shown with XImageByteOrder's name and return types. Appears to have been a glitch in the nroff - docbook conversion. Reported-by: ZHANG

Re: glx support in Xephyr

2013-06-07 Thread Daniel Stone
Hi, On 5 June 2013 12:23, Laurent Carlier lordhea...@gmail.com wrote: Any plans to apply the patch provided in https://bugs.freedesktop.org/show_bug.cgi?id=62346 ? Indentation is totally broken, but with that fixed (and sent to the mailing list): Acked-by: Daniel Stone dan...@fooishbar.org

[PATCH] ephyr: restore GLX support

2013-06-07 Thread Laurent Carlier
It was removed since version 1.13 Signed-off-by: Sebastien Bacher seb...@ubuntu.com Acked-by: Daniel Stone dan...@fooishbar.org --- hw/kdrive/ephyr/ephyrinit.c | 20 1 file changed, 20 insertions(+) diff --git a/hw/kdrive/ephyr/ephyrinit.c b/hw/kdrive/ephyr/ephyrinit.c

A clerical error in the document Xlib - C Language X Interface X11R7.7

2013-06-07 Thread ZHANG Zhaolong
hi, In this page: http://www.x.org/releases/X11R7.7/doc/libX11/libX11/libX11.html#Image_Format_Functions_and_Macros To obtain the pixmap format information for a given display, use XListPixmapFormats. -ImageByteOrder(display) - int XImageByteOrder(Display *display, int

Re: [PATCH:libXrandr] Add some casts for gcc with more warnings enabled.

2013-06-07 Thread Alan Coopersmith
On 06/ 2/13 01:17 PM, Thomas Klausner wrote: --- src/XrrProvider.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/XrrProvider.c b/src/XrrProvider.c index 309e321..cc339cd 100644 --- a/src/XrrProvider.c +++ b/src/XrrProvider.c @@ -77,7 +77,7 @@

Re: [PATCH:libpciaccess 1/3] Protect config.h like usual.

2013-06-07 Thread Alan Coopersmith
On 06/ 2/13 01:38 PM, Thomas Klausner wrote: --- src/common_bridge.c | 3 +++ src/common_device_name.c | 3 +++ 2 files changed, 6 insertions(+) diff --git a/src/common_bridge.c b/src/common_bridge.c index 82a13fd..b4b5d7e 100644 --- a/src/common_bridge.c +++ b/src/common_bridge.c @@

Re: [PATCH:libpciaccess 3/3] Update NetBSD support. From Michael Lorenz macal...@netbsd.org.

2013-06-07 Thread Alan Coopersmith
On 06/ 2/13 01:38 PM, Thomas Klausner wrote: --- configure.ac | 4 +- src/netbsd_pci.c | 471 +++ 2 files changed, 342 insertions(+), 133 deletions(-) Since that's a substantial amount of code, any chance of Signed-off-by tags to

Re: [PATCH:libpciaccess 2/3] Remove useless extra const in const sometype const * var = value; . Found using clang -Wduplicate-decl-specifier.

2013-06-07 Thread Alan Coopersmith
On 06/ 2/13 01:38 PM, Thomas Klausner wrote: --- src/common_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/common_interface.c b/src/common_interface.c index 6dccf8e..8eace4e 100644 --- a/src/common_interface.c +++ b/src/common_interface.c @@ -268,7 +268,7

Re: [PATCH libXt 1/2] Fix cross-compilation

2013-06-07 Thread Alan Coopersmith
On 01/ 3/13 01:16 AM, Thierry Reding wrote: When cross-compiling against a sysroot for instance, a --sysroot option is typically passed via the CPPFLAGS variable. This can cause the build to break in the util subdirectory which requires the CPPFLAGS to be set for the build system. Solve this by