Internationalize xload

2010-01-15 Thread Alan Coopersmith
Upstreaming some changes we've had in the Solaris xload for almost 10 years, now that I've finally gotten around to integrating them with the autotools build. I know this doesn't have the full whizbang integration found in many packages that utilize the GNU gettextize autotools support, but I've n

[PATCH:xload] Integrate gettext support with autotools build system

2010-01-15 Thread Alan Coopersmith
Mostly cribbed from prior work for libXpm Signed-off-by: Alan Coopersmith --- Makefile.am |9 + acinclude.m4 | 47 +++ configure.ac | 20 xload.c | 14 +++--- 4 files changed, 87 insertions(+), 3 del

[PATCH:xload] Internationalize xload: Call gettext on strings printed to stderr

2010-01-15 Thread Alan Coopersmith
From: Jay Hobson Originally done for Solaris 9 to fix Sun bug 4365629 http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=4365629 Signed-off-by: Alan Coopersmith --- COPYING | 23 +++ xload.c | 93 --- 2 files c

Re: [PATCH video-qlx 0/7] Apply latest configuration patches

2010-01-15 Thread Gaetan Nadon
[PATCH video-qlx 0/7] : typo: make that video-qxl. On Fri, 2010-01-15 at 17:47 -0500, Gaetan Nadon wrote: > The initial public release of the QXL virtual GPU was based on a level of > configuration > that was since improved through several patches. Some are maintenance items > applied > to all

[PATCH video-qlx 7/7] config: add INSTALL and ChangeLog #23814

2010-01-15 Thread Gaetan Nadon
All modules are required to have an INSTALL file and a ChangeLog file. Signed-off-by: Gaetan Nadon --- Makefile.am | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/Makefile.am b/Makefile.am index 586638a..781f29b 100644 --- a/Makefile.am +++ b/Makefile.am @@ -1

[PATCH video-qlx 6/7] make: Automake 'foreign' option is specified in configure.ac. #24206

2010-01-15 Thread Gaetan Nadon
Remove from Makefile.am Signed-off-by: Gaetan Nadon --- Makefile.am |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/Makefile.am b/Makefile.am index c47e58a..586638a 100644 --- a/Makefile.am +++ b/Makefile.am @@ -18,5 +18,4 @@ # IN AN ACTION OF CONTRACT, TORT OR OTHERWI

[PATCH video-qlx 5/7] configure.ac: remove -I$(prefix)/include from INCLUDES #24676

2010-01-15 Thread Gaetan Nadon
Using $prefix for any purpose will yield incorrect results as not all modules uses the same prefix and that modules can use different directory which is configurable. The main include dir comes from XORG_CFLAGS Reported-By: Michael Olbrich Signed-off-by: Gaetan Nadon --- configure.ac |2 +-

[PATCH video-qlx 4/7] configure.ac: sdkdir usage duplicates the sdk include dir

2010-01-15 Thread Gaetan Nadon
The sdkdir variable provides a duplicate copy of the include/xorg directory. The statement is removed as this was it's only used. In the Makefile, there is now only one instance of the -I sdkdir The sdkdir is provided in XORG_CFLAGS. Signed-off-by: Gaetan Nadon --- configure.ac |3 +-- 1 fil

[PATCH video-qlx 3/7] git: .gitignore: use common defaults with custom section # 24239

2010-01-15 Thread Gaetan Nadon
Using common defaults will reduce errors and maintenance. Only the very small or inexistent custom section need periodic maintenance when the structure of the component changes. Do not edit defaults. Signed-off-by: Gaetan Nadon --- .gitignore | 78 ++

[PATCH video-qlx 2/7] packaging: add COPYING and README

2010-01-15 Thread Gaetan Nadon
http://wiki.x.org/wiki/NewModuleGuidelines Signed-off-by: Gaetan Nadon --- COPYING | 19 +++ README | 26 ++ 2 files changed, 45 insertions(+), 0 deletions(-) create mode 100644 COPYING create mode 100644 README diff --git a/COPYING b/COPYING new f

[PATCH video-qlx 1/7] configure.ac: deploy the new XORG_DEFAULT_OPTIONS #24242

2010-01-15 Thread Gaetan Nadon
This macro aggregate a number of existing macros that sets commmon X.Org components configuration options. It shields the configuration file from future changes. Signed-off-by: Gaetan Nadon --- configure.ac | 26 ++ 1 files changed, 14 insertions(+), 12 deletions(-) di

[PATCH video-qlx 0/7] Apply latest configuration patches

2010-01-15 Thread Gaetan Nadon
The initial public release of the QXL virtual GPU was based on a level of configuration that was since improved through several patches. Some are maintenance items applied to all X.Org modules while others are drivers specific patches. All have been reviewed against the other drivers. Gaetan Nad

Re: [PATCH 08/11] xfree86: replace True/False with TRUE/FALSE.

2010-01-15 Thread Peter Hutterer
On Fri, Jan 15, 2010 at 08:35:19AM -0800, Dan Nicholson wrote: > On Thu, Jan 14, 2010 at 9:32 PM, Peter Hutterer > wrote: > > xf86Xinput.c relied on xkbsrv.h's definition of True/False which seems odd > > at first and weird on second glance. > > Hey, I put those in! I personally like True/False o

Re: [PULL] udev input-hotplug backend

2010-01-15 Thread Stephan Raue
thank you Tested-by: Stephan Raue Am 15.01.2010 18:10, schrieb Dan Nicholson: > On Mon, Jan 11, 2010 at 6:51 AM, Dan Nicholson wrote: > >> On Sun, Jan 10, 2010 at 11:52 PM, Stephan Raue >> wrote: >> >>> Hi Dan, >>> >>> do you dont forgot this? (before release) >>> >> No, I h

Re: [PATCH v2] Allow driver to call DeleteInputDeviceRequest during UnInit

2010-01-15 Thread Oldřich Jedlička
Hi Peter, Simon, On Friday 15 of January 2010 at 10:40:02, Simon Thum wrote: > Oldřich Jedlička wrote: > > When the input driver (like xf86-input-wacom) removes it's devices > > during a call to UnInit, the CloseDownDevices() cannot handle it. The > > "next" variable can become a pointer to freed

[PATCH xserver] packaging: provide a default README file #24206

2010-01-15 Thread Gaetan Nadon
All modules should have a README file. The bottom URL section is found in all X.Org README files. Reviewed-by: Peter Hutterer Signed-off-by: Gaetan Nadon --- README | 38 ++ 1 files changed, 38 insertions(+), 0 deletions(-) create mode 100644 README diff

Re: [PATCH xserver] packaging: provide a default README file #24206

2010-01-15 Thread Gaetan Nadon
On Fri, 2010-01-15 at 11:33 +1000, Peter Hutterer wrote: > On Thu, Jan 14, 2010 at 04:08:46PM -0500, Gaetan Nadon wrote: > > All modules should have a README file. > > The bottom URL section is found in all X.Org README files. > > > > Signed-off-by: Gaetan Nadon > > --- > > README | 34 ++

Re: [PATCH v3] dix: move cursor realize code to its own function

2010-01-15 Thread Keith Packard
On Fri, 15 Jan 2010 19:34:29 +0200, Tiago Vignatti wrote: > The semantic remains, only code was moved: reuse chunk of code to realize > cursor on both AllocARGBCursor and AllocGlyphCursor. > > (sh*t, git-diff got cracked and is confusing!) > > Signed-off-by: Tiago Vignatti Reviewed-by: Keith

[PATCH v3] dix: move cursor realize code to its own function

2010-01-15 Thread Tiago Vignatti
The semantic remains, only code was moved: reuse chunk of code to realize cursor on both AllocARGBCursor and AllocGlyphCursor. (sh*t, git-diff got cracked and is confusing!) Signed-off-by: Tiago Vignatti --- changes since v2: applied keithp suggestion and changed comments accordingly. dix/curs

[PATCH:libX11] Add %S substitutions to Compose man page

2010-01-15 Thread Alan Coopersmith
Commit 9df349a7894725f9469b106af645f57f7f3f9af3 added support for %S in Compose file include statements - this documents it in the Compose file man page. Also changes the existing substitution documentation to list format instead of a paragraph format to allow easier adding of %S. Signed-off-by:

Re: [PULL] udev input-hotplug backend

2010-01-15 Thread Dan Nicholson
On Mon, Jan 11, 2010 at 6:51 AM, Dan Nicholson wrote: > On Sun, Jan 10, 2010 at 11:52 PM, Stephan Raue > wrote: >> Hi Dan, >> >> do you dont forgot this? (before release) > > No, I haven't, but I've been busy and I wasn't able to reproduce it in > the short time I was playing with it. I also jus

Re: [PATCH 01/11] xkb: remove _XkbTyped*alloc

2010-01-15 Thread Dan Nicholson
On Thu, Jan 14, 2010 at 9:32 PM, Peter Hutterer wrote: > Please no extension-specific macros for memory allocation. > > Signed-off-by: Peter Hutterer Or ridiculous unnecessary casts. Reviewed-by: Dan Nicholson ___ xorg-devel mailing list xorg-devel@l

Re: [PATCH v2.1] xkb: Add XKM file format description.

2010-01-15 Thread Dan Nicholson
On Thu, Jan 14, 2010 at 7:24 PM, Peter Hutterer wrote: > Signed-off-by: Peter Hutterer I applaud you for actually taking the time to document this. Maybe someday it won't even be necessary. Acked-by: Dan Nicholson ___ xorg-devel mailing list xorg-dev

Re: [PATCH 11/11] xkb: unexport xkbDevicePrivateKey and xkbUnwrapProc.

2010-01-15 Thread Dan Nicholson
On Thu, Jan 14, 2010 at 9:32 PM, Peter Hutterer wrote: > Signed-off-by: Peter Hutterer > --- >  include/xkbsrv.h |    4 ++-- >  1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/xkbsrv.h b/include/xkbsrv.h > index d5a811d..758ea80 100644 > --- a/include/xkbsrv.h > +++ b/in

Re: [PATCH 10/11] xkb: remove XkbFoobar -> SrvXkbFoobar defines.

2010-01-15 Thread Dan Nicholson
On Thu, Jan 14, 2010 at 9:32 PM, Peter Hutterer wrote: > These functions have the same names as the client-side functions as a legacy > of the shared header files. Remove the define, rename all functions to > XkbSrvFoobar. XkbSrvFoobar is preferable to SrvXkbFoobar for namespacing. > > This techni

Re: [PATCH 09/11] xkb: sed True -> TRUE and False -> FALSE

2010-01-15 Thread Dan Nicholson
On Thu, Jan 14, 2010 at 9:32 PM, Peter Hutterer wrote: > Signed-off-by: Peter Hutterer Reviewed-by: Dan Nicholson ___ xorg-devel mailing list xorg-devel@lists.x.org http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH 07/11] xkb: remove unused _XkbIsPressEvent and _XkbIsReleaseEvent defines

2010-01-15 Thread Dan Nicholson
On Thu, Jan 14, 2010 at 9:32 PM, Peter Hutterer wrote: > Signed-off-by: Peter Hutterer > --- >  include/xkbsrv.h |    3 --- >  1 files changed, 0 insertions(+), 3 deletions(-) > > diff --git a/include/xkbsrv.h b/include/xkbsrv.h > index bc74eef..4575a8a 100644 > --- a/include/xkbsrv.h > +++ b/inc

Re: [PATCH 08/11] xfree86: replace True/False with TRUE/FALSE.

2010-01-15 Thread Dan Nicholson
On Thu, Jan 14, 2010 at 9:32 PM, Peter Hutterer wrote: > xf86Xinput.c relied on xkbsrv.h's definition of True/False which seems odd > at first and weird on second glance. Hey, I put those in! I personally like True/False or true/false better than all caps, but am not religious about it. Reviewed

Re: [PATCH 06/11] xkb: purge _XkbLibError.

2010-01-15 Thread Dan Nicholson
On Thu, Jan 14, 2010 at 9:32 PM, Peter Hutterer wrote: > This is a leftover from the shared header files and was defined away anyway. > > Signed-off-by: Peter Hutterer > --- >  include/xkbsrv.h |    1 - >  xkb/xkbout.c     |   28 ++-- >  xkb/xkmread.c    |  194 +--

Re: [PATCH 05/11] xkb: remove XkbAtomGetString, replace with NameForAtom.

2010-01-15 Thread Fernando Carrijo
Peter Hutterer wrote: > XKB really XKBdoes not XKBneed its own XKBdefines for XKBeverything. > > Signed-off-by: Peter Hutterer > --- > include/xkbsrv.h |2 -- > xkb/xkbfmisc.c | 10 +- > xkb/xkbout.c |2 +- > xkb/xkbtext.c|4 ++-- > 4 files changed, 8 insertions

Re: [PATCH 04/11] xkb: remove IsKeypadKey define, only used in two places.

2010-01-15 Thread Fernando Carrijo
Peter Hutterer wrote: > Signed-off-by: Peter Hutterer > --- > include/xkbsrv.h |3 --- > xkb/XKBMisc.c|2 +- > 2 files changed, 1 insertions(+), 4 deletions(-) Reviewed-by: Fernando Carrijo ___ xorg-devel mailing list xorg-devel@lists.x

Re: [PATCH 02/11] xkb: remove _XkbClearElems, a memset will do.

2010-01-15 Thread Fernando Carrijo
Peter Hutterer wrote: > Bonus point - it's easier to understand what's actually being done with the > memory. > > Signed-off-by: Peter Hutterer > --- > include/xkbsrv.h |2 -- > xkb/XKBAlloc.c | 13 +++-- > 2 files changed, 7 insertions(+), 8 deletions(-) Reviewed-by: Fernando

Re: [PATCH 01/11] xkb: remove _XkbTyped*alloc

2010-01-15 Thread Fernando Carrijo
Peter Hutterer wrote: > Please no extension-specific macros for memory allocation. > > Signed-off-by: Peter Hutterer > --- > include/xkbsrv.h |4 --- > xkb/XKBAlloc.c | 33 + > xkb/XKBGAlloc.c |2 +- > xkb/XKBMAlloc.c | 71 +++

Re: [PATCH v2] Allow driver to call DeleteInputDeviceRequest during UnInit

2010-01-15 Thread Simon Thum
Oldřich Jedlička wrote: > When the input driver (like xf86-input-wacom) removes it's devices > during a call to UnInit, the CloseDownDevices() cannot handle it. The > "next" variable can become a pointer to freed memory. > > The patch introduces order-independent device freeing mechanism by > reme