[PATCH libXi v2] Fix duplicate sizeof in copy_classes

2011-11-11 Thread Peter Hutterer
sizeof(bla * sizeof()) is'nt right. Plus add some () to the next_block call too to emphasise that *nclasses is the multiplicator. Signed-off-by: Peter Hutterer Reviewed-by: Jeremy Huddleston --- Good catch, I must have read over this 50 times. I guess Jeremy's rev-by still stands so I left the

[PATCH:xfs 2/4] Fix printf format errors found by gcc now that it can check

2011-11-11 Thread Alan Coopersmith
Fixes: os/config.c: In function ‘config_parse_nameVal’: os/config.c:417:5: warning: too few arguments for format os/config.c: In function ‘config_parse_int’: os/config.c:458:6: warning: too few arguments for format Requires passing parm arguments around in order to provide parameter names with pro

[PATCH:xfs 4/4] Use config.h for autoconf output now that there's no conflict

2011-11-11 Thread Alan Coopersmith
Now that os/config.h is gone, no reason to special case and have xfs-config.h for the autoconf generated header. Signed-off-by: Alan Coopersmith --- .gitignore |3 --- configure.ac |3 +-- difs/atom.c|2 +- difs/cache.c |2 +- difs/charinfo.c|2

[PATCH:xfs 1/4] Make CONFIG_ERR_* values be strings instead of indexes in ConfigErrors array

2011-11-11 Thread Alan Coopersmith
Allows gcc to check printf format strings instead of just giving warnings. Since these #defines are only used in config.c, define them directly there, instead of in the config.h header that no one else includes. Signed-off-by: Alan Coopersmith --- os/config.c | 47 ++---

[PATCH:xfs 3/4] Move os/config.h into os/config.c

2011-11-11 Thread Alan Coopersmith
The only file that needed the only remaining definition was config.c, so there's no point keeping a header file around for a single #define Signed-off-by: Alan Coopersmith --- Makefile.am |1 - os/config.c |3 +++ os/config.h | 54 --

[PATCH:libxkbfile] Add const attributes to fix gcc -Wwrite-strings warnings

2011-11-11 Thread Alan Coopersmith
Does not fix all of them, as some affect the API and thus need more analysis. Signed-off-by: Alan Coopersmith --- include/X11/extensions/XKBfile.h |4 ++-- src/maprules.c |2 +- src/xkbbells.c |2 +- src/xkbconfig.c |2 +- src/

[PATCH:libFS 2/2] Mark pattern argument to FSListFonts* as const char *

2011-11-11 Thread Alan Coopersmith
Needed to fix gcc -Wwrite-strings warnings in clients such as fslsfonts Signed-off-by: Alan Coopersmith --- doc/FSlib.txt |4 ++-- include/X11/fonts/FSlib.h |4 ++-- src/FSFontInfo.c |2 +- src/FSFtNames.c |2 +- src/FSlibInt.c|4 ++-

[PATCH:libFS 1/2] Change FSOpenServer and FSServerName to take const char * args.

2011-11-11 Thread Alan Coopersmith
Matches the current versions of the XOpenDisplay & XDisplayName functions they're modeled on. Signed-off-by: Alan Coopersmith --- doc/FSlib.txt |6 +++--- include/X11/fonts/FSlib.h |4 ++-- src/FSOpenServ.c |4 ++-- src/FSServName.c |4 ++-- 4 files

[PATCH:xmodmap 3/3] Replace chk_malloc + sprintf with asprintf

2011-11-11 Thread Alan Coopersmith
Includes local private copy of asprintf for OS'es without it in libc. Removes chk_malloc as no callers remain anymore. Signed-off-by: Alan Coopersmith --- configure.ac |2 +- xmodmap.c| 69 + xmodmap.h|5 +++- 3 files chan

[PATCH:xmodmap 2/3] Free copy of input line at end of process_line instead of leaking it

2011-11-11 Thread Alan Coopersmith
Signed-off-by: Alan Coopersmith --- pf.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/pf.c b/pf.c index 0cc8101..0eb0f55 100644 --- a/pf.c +++ b/pf.c @@ -30,6 +30,7 @@ from The Open Group. #include #include #include +#include #include "xmodmap.h" #def

[PATCH:xmodmap 1/3] Fix gcc -Wwrite-strings warnings

2011-11-11 Thread Alan Coopersmith
Signed-off-by: Alan Coopersmith --- exec.c|2 +- handle.c |6 +++--- pf.c | 18 ++ wq.h |2 +- xmodmap.c |2 +- xmodmap.h |3 ++- 6 files changed, 18 insertions(+), 15 deletions(-) diff --git a/exec.c b/exec.c index 0508cd3..dbcb669 100644 ---

[PATCH vmmouse] Deal with opaque InputOption types in ABI 14

2011-11-11 Thread Peter Hutterer
Signed-off-by: Peter Hutterer --- I admit I didn't test this other than compile-test but this should be the right fix. Thomas, please give this some extra testing, I ran out of time, sorry. src/vmmouse.c | 49 ++--- 1 files changed, 34 insertions(+),

Re: [PATCH xdm] Re-introduce the ability of staticcally linking the greeter.

2011-11-11 Thread Alan Coopersmith
On 11/07/11 11:57, Matthieu Herrb wrote: This logically reverts 7e223d3ac6c0d549a7d6e4dcc86a053e19594028. There are still cases (in particular OpenBSD) where the shared greeter is not desired. As discussed elsewhere in this thread, I'm okay with the concept of this patch, though I have a couple

[PATCH xserver] docs: spell "X Server Version" consistently in titles. Add where missing.

2011-11-11 Thread Gaetan Nadon
Reviewed-by: Alan Coopersmith Signed-off-by: Gaetan Nadon --- doc/Xinput.xml|2 +- doc/Xserver-spec.xml |2 +- doc/dtrace/Xserver-DTrace.xml |2 +- hw/dmx/doc/dmx.xml|2 ++ hw/dmx/doc/scaled.xml |2 ++ hw/xfree86/doc/ddxDesign.xml

Re: [PATCH xserver] docs: spell "X Server Version" consistently in titles. Add where missing.

2011-11-11 Thread Alan Coopersmith
On 11/11/11 13:51, Gaetan Nadon wrote: Signed-off-by: Gaetan Nadon --- doc/Xinput.xml|2 +- doc/Xserver-spec.xml |2 +- doc/dtrace/Xserver-DTrace.xml |2 +- hw/dmx/doc/dmx.xml|2 ++ hw/dmx/doc/scaled.xml |2 ++ hw/xfree86/doc

Re: [PATCH libXi] Add some () to shut up coverity

2011-11-11 Thread Josh Triplett
Yes, this is an array of *nclasses pointers. Signed-off-by: Peter Hutterer --- src/XExtInt.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/XExtInt.c b/src/XExtInt.c index aa341e9..372956e 100644 --- a/src/XExtInt.c +++ b/src/XExtInt.c @@ -1528,7 +1528,7 @@ copy_cl

[PATCH xserver] docs: spell "X Server Version" consistently in titles. Add where missing.

2011-11-11 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon --- doc/Xinput.xml|2 +- doc/Xserver-spec.xml |2 +- doc/dtrace/Xserver-DTrace.xml |2 +- hw/dmx/doc/dmx.xml|2 ++ hw/dmx/doc/scaled.xml |2 ++ hw/xfree86/doc/ddxDesign.xml |2 +- 6 files changed, 8

Re: Glamor update

2011-11-11 Thread Jamey Sharp
Hello! On Fri, Nov 11, 2011 at 04:42:32PM +0800, Zhigang Gong wrote: > During the last discussion about glamor’s plan in this list, we got a > conclusion that to extract glamor from xorg and build a separate > glamor library to be used by any possible DDX driver. And Eric > suggested I can increme

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

2011-11-11 Thread Gaetan Nadon
On Fri, 2011-10-07 at 13:28 -0700, Jeremy Huddleston wrote: > >> On Oct 6, 2011, at 1:09 PM, Gaetan Nadon wrote: > >>> The srcipt runs 'make dist' to create the tarball. We don't have to prompt > >>> the user for the tar name, the version number or the section name. > >> > >> Should we 'make dist

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

2011-11-11 Thread Gaetan Nadon
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 from the module git URL - It only releases tarballs it has created from the git module - Timely feedback, e

Re: [PULL] Xext: XCopyArea does not work in Xinerama mode.#25113

2011-11-11 Thread Keith Packard
On Thu, 10 Nov 2011 21:05:47 +0530, Arvind Umrao wrote: > XCopyArea does not copy areas of the screen across physical displays. This is not a protocol violation. Throughout the spec, it is made clear that the X server is free to discard window contents at any time, as long as suitable expose or

Re: Changes causes application

2011-11-11 Thread Jamey Sharp
On 11/11/11, Ratin wrote: > On Thu, Nov 10, 2011 at 12:57 PM, Jamey Sharp wrote: >> What application is it? Does it make X requests from more than one >> thread? > > Hi Jamey, Thanks for the insightful question. Its a bit complex video > wall application ... [interesting details snipped] > > As f

Re: s3switch maintenance

2011-11-11 Thread Guido Günther
On Thu, Nov 10, 2011 at 11:37:44PM +0100, Tormod Volden wrote: > [s3switch is a tool for switching outputs on S3 Savage cards] > > Hi, > Due to popular request, I would like to revive the s3switch tool and > maintain it. It is still shipped in Fedora, based on a patch set from > Debian and a coupl

Re: [PULL] Xext: XCopyArea does not work in Xinerama mode.#25113

2011-11-11 Thread Arvind Umrao
New pull request, removed 2011BRB_Reviewed tag. The following changes since commit 6601a0337f093fb56e995db92b1074d45c9003f2: Xext: XCopyArea does not work in Xinerama mode.#25113 (2011-11-11 21:13:33 +0530) are available in the git repository at: git://github.com/akumrao/xserver.git mast

Re: [PATCH libXi] Include config.h from source files

2011-11-11 Thread Gaetan Nadon
On Fri, 2011-11-11 at 15:33 +1000, Peter Hutterer wrote: > Signed-off-by: Peter Hutterer > --- > src/XAllowDv.c |3 +++ > src/XChgDCtl.c |4 > src/XChgFCtl.c |4 > src/XChgKMap.c |4 > src/XChgKbd.c|4 > src/XChgPnt.c

Re: Changes causes application

2011-11-11 Thread Ratin
On Thu, Nov 10, 2011 at 12:57 PM, Jamey Sharp wrote: > On Wed, Oct 26, 2011 at 06:36:06AM -0700, Ratin wrote: >> Hi I have an X11 based application that renders video on the monitor >> upon launch, sometimes when launched, the video does not start >> rendering until mouse click event happens. The

Glamor update

2011-11-11 Thread Zhigang Gong
Hi folks, During the last discussion about glamor’s plan in this list, we got a conclusion that to extract glamor from xorg and build a separate glamor library to be used by any possible DDX driver. And Eric suggested I can incrementally merge glamor into Intel video driver. Now here is the

Re: [PATCH libXi] Stop unnecessary calls to size_classes

2011-11-11 Thread Jeremy Huddleston
Why can't you just use malloc() ... do we really need an Xmalloc macro any more? Either way, all 3 are: Reviewed-by: Jeremy Huddleston On Nov 10, 2011, at 8:50 PM, Peter Hutterer wrote: > Xmalloc is a macro evaluating its argument twice. > > Signed-off-by: Peter Hutterer > --- > src/XIQueryDe

Re: moving rootless out of the server

2011-11-11 Thread Jeremy Huddleston
On Nov 10, 2011, at 2:44 PM, Jamey Sharp wrote: > Just to let you all know: Getting students to build xf86-video-nested > worked out well, so we're starting another X-related PSU Capstone > project. This time Jeremy is joining Josh and me as project sponsors. > > The goal this time around is to