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

2013-07-08 Thread Alan Coopersmith
On 07/ 8/13 03:55 PM, Stéphane Aulery wrote: Remove dead code the-l option that seems copied from another utility (xsetpointer) but never executed. Signed-off-by: Stéphane Aulery --- xsetmode.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) Dead code in a dead prog

Re: [PATCH xsetpointer] Impove help

2013-07-08 Thread Alan Coopersmith
On 07/ 8/13 03:35 PM, Stéphane Aulery wrote: Signed-off-by: Stéphane Aulery --- man/xsetpointer.man | 30 -- xsetpointer.c |6 +- 2 files changed, 29 insertions(+), 7 deletions(-) Pushed, though it's mostly pointless as xsetpointer has not had a re

Re: [PATCH:xrdb] Add a command line flag -cppargs which allows passing arguments to cpp (while still complying with the access(2) check for the cpp executable).

2013-07-08 Thread Alan Coopersmith
On 08/23/12 07:48 AM, Thomas Klausner wrote: See thread starting at http://mail-index.netbsd.org/tech-x11/2009/12/01/msg000687.html That thread seems to already be resolved by the patch Egbert pushed earlier this year: http://cgit.freedesktop.org/xorg/app/xrdb/commit/?id=420347005ccf1886dfbb2e

[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 --- man/xscope.man | 89 +--- scope.c| 33 - 2 files changed, 84 insertions(+),

[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 --- xsetmode.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/xsetmode.c b/xsetmode.c index 2aead84..bdfe862 100644 --- a/x

[PATCH xsetpointer] Impove help

2013-07-08 Thread Stéphane Aulery
Signed-off-by: Stéphane Aulery --- 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/xsetpointer.man +++ b/

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 > > > > diff --git a/math.c b/math.c > > index a8c4a5c..b09b94a 100644 > > --- a/math.c > > - for (i=1; i < XCALC_MEMORY; i++) >

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

2013-07-08 Thread Matthieu Herrb
On Sun, Jul 07, 2013 at 10:50:30PM +0200, Stéphane Aulery wrote: > Signed-off-by: Stéphane Aulery > --- > 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 @@ voi

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

2013-07-08 Thread walter harms
Am 07.07.2013 22:50, schrieb Stéphane Aulery: > Signed-off-by: Stéphane Aulery > --- > 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) >