Age | Commit message (Collapse) | Author |
|
Support for xprint was removed in commit 0ce52bc9ed.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
This silences all "function might be candidate for attribute 'noreturn'"
warnings. Add to HandleEvent as well, as that function never returns.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
This silences all "function '...' might be a candidate for 'gnu_printf'
format attribute" warnings.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
yylex is already declared in (the generated) lex.c. This silences
a "redundant redeclaration of 'yylex'" warning.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
yyparse is already declared in (the generated) gram.h. This silences
a "redundant redeclaration of 'yyparse'" warning.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
yylineno is already declared in (the generated) gram.h. This silences
a "redundant redeclaration of 'yylineno'" warning.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Silence some "initialization discards 'const' qualifier from pointer
target type" warnings.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
The variable `n` is written to every iteration of the loop, but never
read. It is overwritten (and read) in the next loop though. Remove the
dead store.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
xprint is deprecated and was removed from xorg in 2008.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Names that begin with an underscore ('_') are reserved. Follow the usual
practice of naming the include guard of file "foo.h" "FOO_H", as is already
done in "icons.h".
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Found by using:
codespell --builtin clear,rare,usage,informal,code,names
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
adapt the suggestion to use XFetchName to work around a system whose locale
support is broken, but rather than break existing configurations, use that
call as a fallback when the existing/working calls fail.
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
useful; a subsequent X11R6 change made the behavior a little worse.
This change repairs that old damage.
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
does not handle those files).
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
since -q option can be used to silence those if wanted
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
name as a prefix
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
also bump version to 1.0.11.1, reflecting ongoing work since release
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
and document (except for the print-extension) all of the options
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
report/patch by Preston Crow for improved window placement
(reindented to fit with current sources)
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
because the case lacked a break-statement
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
https://invisible-island.net/cindent/
That is essentially a script which adds typedefs to the x-indent.sh script.
No object-code changed in this reformatting.
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
patch submitted by Preston Crow
modified to check if xrandr development header/library is available
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
obsolete and less portable
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
https://gitlab.freedesktop.org/xorg/app/twm/issues/4
patch submitted by Jes Bodi Klinke
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
fixed in that case.
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
twm.c:544:20: warning: assignment discards ‘const’ qualifier from
pointer target type [-Wdiscarded-qualifiers]
Scr->iconmgr.name = "TWM";
^
twm.c:545:25: warning: assignment discards ‘const’ qualifier from
pointer target type [-Wdiscarded-qualifiers]
Scr->iconmgr.icon_name = "Icons";
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
${rcdir} is ${datadir}/X11/twm by default, but can be overridden when
running make. If it is overridden, then the code should read from the
same place as where "make rcdir=FOO install" puts the file.
(Obviously overriding rcdir= for "make install" requires overriding it
for "make" as well. But at least this way it's possible to get it to
work. :-) )
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Found by Oracle's Parfait 2.2 static analyzer:
Error: Memory leak
Memory leak [memory-leak] (CWE 401):
Memory leak of pointer sc allocated with malloc(32)
at line 421 of src/util.c in function 'InsertRGBColormap'.
sc allocated at line 397 with malloc(32)
sc leaks when replace != 0 at line 405.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Found by Oracle's Parfait 2.2 static analyzer:
Error: Use after free
Use after free [use-after-free] (CWE 416):
Use after free of pointer Scr
at line 421 of src/util.c in function 'InsertRGBColormap'.
Invalid pointer accessible via global Scr at line 105 of src/twm.c
maps escapes to sc->maps at line 419 of src/util.c in function 'InsertRGBColormap'
maps freed with XFree at line 406
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
It now always allocates memory, so remove some unnecessary checks.
While here, improve handling of an error case.
Signed-off-by: Thomas Klausner <wiz@NetBSD.org>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|