Re: [PATCH setxkbmap] -I syntax wrong in manpage

2013-07-17 Thread Stéphane Aulery
Hello Matt, De: Matt Dew mar...@osource.org Envoyé: Tue, 16 Jul 2013 19:56:51 +0200 (CEST) If not too late, Reviewed-By: Matt Dew mar...@osource.org On 06/30/2013 02:45 PM, Daniel Stone wrote: Reviewed-by: Daniel Stone dan...@fooishbar.org On 29 June 2013 00:04, Stéphane Aulery lk

Re: [PATCH:xrdb] Clarify .Xresources vs. .Xdefaults in man page

2013-07-17 Thread Stéphane Aulery
/xrdb/commit/?id=ed086df97fc816dfeb792ac0094eaa8795177bf5 Regards, -- Stéphane Aulery ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re : Re: [PATCH xmodmap] error in manpage example about swapping Control_L/Caps_Lock keys

2013-07-17 Thread Stéphane Aulery
07:48 PM, Stéphane Aulery wrote: Reported by Luca Capello at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=641588 Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xmodmap.man |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/man/xmodmap.man b/man/xmodmap.man

Re: [PATCH xsetmode] Remove dead code the-l option

2013-07-09 Thread Stéphane Aulery
De: Alan Coopersmith alan.coopersm...@oracle.com Envoyé: Tue, 09 Jul 2013 08:53:56 +0200 (CEST) On 07/ 8/13 03:55 PM, Stéphane Aulery wrote: Signed-off-by: Stéphane Aulery lk...@free.fr --- xsetmode.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) Dead code

Re: [PATCH xcalc 3/3] Reduce scope of i (cppcheck --verbose --enable=all)

2013-07-08 Thread Stéphane Aulery
Le lundi 08 juillet 2013 à 02:53:09, Matthieu Herrb a écrit : On Sun, Jul 07, 2013 at 10:50:30PM +0200, Stéphane Aulery wrote: Signed-off-by: Stéphane Aulery lk...@free.fr diff --git a/math.c b/math.c index a8c4a5c..b09b94a 100644 --- a/math.c - for (i=1; i XCALC_MEMORY; i

[PATCH xsetpointer] Impove help

2013-07-08 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xsetpointer.man | 30 -- xsetpointer.c |6 +- 2 files changed, 29 insertions(+), 7 deletions(-) diff --git a/man/xsetpointer.man b/man/xsetpointer.man index c75666f..92ac92e 100644 --- a/man

[PATCH xsetmode] Remove dead code the-l option

2013-07-08 Thread Stéphane Aulery
Remove dead code the-l option that seems copied from another utility (xsetpointer) but never executed. Signed-off-by: Stéphane Aulery lk...@free.fr --- xsetmode.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/xsetmode.c b/xsetmode.c index 2aead84..bdfe862

[PATCH xscope] Improve help and manpage.

2013-07-08 Thread Stéphane Aulery
Add some details on the verbosity and options -T and -A. These two options are not very clear to me. Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xscope.man | 89 +--- scope.c| 33 - 2 files changed, 84

[PATCH xcalc 1/3] Reduce scope of tmp (cppcheck --verbose --enable=all)

2013-07-07 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- math.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/math.c b/math.c index de33f7e..7332372 100644 --- a/math.c +++ b/math.c @@ -175,10 +175,10 @@ static void DrawDisplay(void) { if ((int) strlen(dispstr) 12

[PATCH xcalc 3/3] Reduce scope of i (cppcheck --verbose --enable=all)

2013-07-07 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- math.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/math.c b/math.c index a8c4a5c..b09b94a 100644 --- a/math.c +++ b/math.c @@ -751,13 +751,12 @@ void offf(void) { /* full reset */ - int i; ResetCalc

[PATCH xsetroot] Improve manpage and -help option.

2013-07-07 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xsetroot.man | 44 ++-- xsetroot.c | 32 2 files changed, 42 insertions(+), 34 deletions(-) diff --git a/man/xsetroot.man b/man/xsetroot.man index b58b39d

[PATCH xcalc] Drop CRAY support

2013-07-06 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- math.c | 27 --- 1 file changed, 27 deletions(-) diff --git a/math.c b/math.c index b290beb..7f81757 100644 --- a/math.c +++ b/math.c @@ -18,33 +18,6 @@ #include xcalc.h -#ifdef _CRAY /* kludge around Cray

[PATCH xedit] Drop CRAY support

2013-07-06 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- commands.c |5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/commands.c b/commands.c index 90a64ec..ae81a12 100644 --- a/commands.c +++ b/commands.c @@ -29,9 +29,6 @@ #include X11/Xfuncs.h #include X11/Xos.h #include

[PATCH xsm] Drop CRAY support

2013-07-06 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- signals.c |7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/signals.c b/signals.c index 53a0168..248e9ca 100644 --- a/signals.c +++ b/signals.c @@ -130,8 +130,7 @@ sig_child_handler (int sig) { int pid, olderrno

Re: [PATCH xsetroot] Improve manpage and -help option.

2013-07-06 Thread Stéphane Aulery
Le samedi 06 juillet 2013 à 09:37:59, Alan Coopersmith a écrit : On 07/ 3/13 04:42 PM, Stéphane Aulery wrote: [-cursor_name \fIcursorname\fP] -.IP \fB-cursor_name\fP \fIcursorname\fP +.IP \fB-cursor_name\fP \fIcursorfontname\fP Did you mean to leave those unmatched or did you just miss

[PATCH xcalc 1/2] Use the new M_PI symbol from math.h

2013-07-06 Thread Stéphane Aulery
Use the new M_PI symbol from math.h and fix and a value of the same accuracy if it's necessary to redefine M_PI. Signed-off-by: Stéphane Aulery lk...@free.fr --- math.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/math.c b/math.c index

[PATCH xcalc 2/2] Use the new M_E symbol from math.h

2013-07-06 Thread Stéphane Aulery
Use the new M_E symbol from math.h and fix and a value of the same accuracy if it's necessary to redefine M_E. Signed-off-by: Stéphane Aulery lk...@free.fr --- math.c |8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/math.c b/math.c index bb31b14..63e24b2 100644

[PATCH xwd] Ensure consistency between the manpage and the output of the -help option

2013-07-06 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xwd.man | 14 +- xwd.c | 23 ++- 2 files changed, 27 insertions(+), 10 deletions(-) diff --git a/man/xwd.man b/man/xwd.man index 4dc9041..76e07d7 100644 --- a/man/xwd.man +++ b/man/xwd.man @@ -28,8

[PATCH xvinfo] Use real program name for help message

2013-07-06 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- xvinfo.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/xvinfo.c b/xvinfo.c index 8df7aa6..f1a6056 100644 --- a/xvinfo.c +++ b/xvinfo.c @@ -10,10 +10,12 @@ #include string.h #include ctype.h +static char *progname

[PATCH xwd] Use real program name for help message and fix a typo

2013-07-06 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- xwd.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/xwd.c b/xwd.c index eb60a07..fd914c9 100644 --- a/xwd.c +++ b/xwd.c @@ -89,6 +89,8 @@ in this Software without prior written authorization from The Open Group

[PATCH xsetroot] The name of the programe should be replaced once.

2013-07-06 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- xsetroot.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xsetroot.c b/xsetroot.c index 2eb3a40..73ab1c5 100644 --- a/xsetroot.c +++ b/xsetroot.c @@ -73,7 +73,7 @@ static Pixmap ReadBitmapFile(char *filename, unsigned int

Re: [PATCH xwd] Use real program name for help message and fix a typo

2013-07-06 Thread Stéphane Aulery
Le samedi 06 juillet 2013 à 07:04:02, Alan Coopersmith a écrit : On 07/ 6/13 06:18 PM, Stéphane Aulery wrote: Signed-off-by: Stéphane Aulery lk...@free.fr --- xwd.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/xwd.c b/xwd.c index eb60a07..fd914c9 100644

[PATCH 1/2 xvidtune] Use real program name for help message

2013-07-03 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- xvidtune.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/xvidtune.c b/xvidtune.c index ac339e4..18b67d5 100644 --- a/xvidtune.c +++ b/xvidtune.c @@ -55,6 +55,8 @@ static int dot_clock, mode_flags; static unsigned long

[PATCH 2/2 xvidtune] Ensure consistency between the manpage and the output of the -h option

2013-07-03 Thread Stéphane Aulery
Mention -timeout option in manpage. Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xvidtune.man |6 ++ xvidtune.c |4 ++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/man/xvidtune.man b/man/xvidtune.man index 30379b6..413af99 100644 --- a/man

[PATCH xstdcmap] Use the same parameter names in the help.

2013-07-03 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xstdcmap.man |4 ++-- xstdcmap.c |2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/man/xstdcmap.man b/man/xstdcmap.man index 1685764..a875baf 100644 --- a/man/xstdcmap.man +++ b/man/xstdcmap.man @@ -43,7 +43,7

[PATCH xsm] Improve help messages.

2013-07-03 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xsm.man |6 +++--- xsm.c |2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/man/xsm.man b/man/xsm.man index 8549ab2..5da459a 100644 --- a/man/xsm.man +++ b/man/xsm.man @@ -31,7 +31,7 @@ xsm \- X Session Manager

[PATCH xsetroot] Improve manpage and -help option.

2013-07-03 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xsetroot.man | 41 - xsetroot.c | 30 +++--- 2 files changed, 39 insertions(+), 32 deletions(-) diff --git a/man/xsetroot.man b/man/xsetroot.man index b58b39d..1139842

[PATCH xwininfo] Improve manpage and -h option

2013-07-02 Thread Stéphane Aulery
- Rename windows and display args. - Mention of short (-d) -display option. Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xwininfo.man | 14 +++--- xwininfo.c | 36 ++-- 2 files changed, 25 insertions(+), 25 deletions(-) diff --git

Re: [PATCH xwud] Adds the missing option -dumpheader and improves the display option -h

2013-07-02 Thread Stéphane Aulery
Le mardi 02 juillet 2013 à 09:55:06, walter harms a écrit : Improving help is a good idea ! Reviewed-by: wha...@bfs.de If this patch goes in the repository and you are interested in this kind of change, I would spend in reviewing all applications. -- Stéphane Aulery

Re: [PATCH setxkbmap] -I syntax wrong in manpage

2013-07-01 Thread Stéphane Aulery
to be consistent? Regards, -- Stéphane Aulery ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

[PATCH setxkbmap] Ensures consistency between the manpage and the output of the -h option in addition to the correction [1] for bug #524510 reported by Debian [2].

2013-07-01 Thread Stéphane Aulery
[1] http://cgit.freedesktop.org/xorg/app/setxkbmap/commit/?id=2011f509c56b73cc62105c689d702f279f522017 [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=524510 Signed-off-by: Stéphane Aulery lk...@free.fr --- setxkbmap.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH xwud] Adds the missing option -dumpheader and improves the display option -h

2013-07-01 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery lk...@free.fr --- xwud.c | 21 + 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/xwud.c b/xwud.c index 9c6808c..32489a5 100644 --- a/xwud.c +++ b/xwud.c @@ -78,10 +78,23 @@ static void DumpHeader(const XWDFileHeader *header

[PATCH setxkbmap] -I syntax wrong in manpage

2013-06-30 Thread Stéphane Aulery
cf. syntax in source code file : http://cgit.freedesktop.org/xorg/app/setxkbmap/tree/setxkbmap.c#n248 Fix Debian report Bug #524510 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=524510 Signed-off-by: Stéphane Aulery lk...@free.fr --- man/setxkbmap.man |2 +- 1 file changed, 1 insertion

[PATCH xrandr] Mention of --brightness with -h option

2013-06-30 Thread Stéphane Aulery
Reported by jidanni at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=709569 Signed-off-by: Stéphane Aulery lk...@free.fr --- xrandr.c |1 + 1 file changed, 1 insertion(+) diff --git a/xrandr.c b/xrandr.c index da786cb..0fffaa0 100644 --- a/xrandr.c +++ b/xrandr.c @@ -139,6 +139,7

[PATCH xmodmap] error in manpage example about swapping Control_L/Caps_Lock keys

2013-06-30 Thread Stéphane Aulery
Reported by Luca Capello at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=641588 Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xmodmap.man |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/man/xmodmap.man b/man/xmodmap.man index f864e6e..1ed1ca1 100644 --- a/man

[PATCH xrdb] ~/.Xdefaults is deprecated, ~/.Xresources should be used instead

2013-06-30 Thread Stéphane Aulery
Reported by John Feuerstein at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=649187 Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xrdb.man |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/man/xrdb.man b/man/xrdb.man index e30b0d5..7582712 100644 --- a/man/xrdb.man

[PATCH xrdb] ~/.Xdefaults is deprecated, ~/.Xresources should be used instead

2013-06-29 Thread Stéphane Aulery
Reported by John Feuerstein at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=649187 Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xrdb.man |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/man/xrdb.man b/man/xrdb.man index e30b0d5..7582712 100644 --- a/man/xrdb.man

[PATCH xrandr] Mention of --brightness with -h option

2013-06-29 Thread Stéphane Aulery
Reported by jidanni at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=709569 Signed-off-by: Stéphane Aulery lk...@free.fr --- xrandr.c |1 + 1 file changed, 1 insertion(+) diff --git a/xrandr.c b/xrandr.c index da786cb..0fffaa0 100644 --- a/xrandr.c +++ b/xrandr.c @@ -139,6 +139,7

[PATCH xmodmap] error in manpage example about swapping Control_L/Caps_Lock keys

2013-06-29 Thread Stéphane Aulery
Reported by Luca Capello at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=641588 Signed-off-by: Stéphane Aulery lk...@free.fr --- man/xmodmap.man |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/man/xmodmap.man b/man/xmodmap.man index f864e6e..1ed1ca1 100644 --- a/man

[PATCH setxkbmap] -I syntax wrong in manpage

2013-06-29 Thread Stéphane Aulery
cf. syntax in source code file : http://cgit.freedesktop.org/xorg/app/setxkbmap/tree/setxkbmap.c#n248 Fix Debian report Bug #524510 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=524510 Signed-off-by: Stéphane Aulery lk...@free.fr --- man/setxkbmap.man |2 +- 1 file changed, 1 insertion