Age | Commit message (Collapse) | Author |
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Add a note to the readme file to clarify that the default build system is still
automake.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
AM_CPPFLAGS defines several symbols that do not have to be there. By providing
default values in twm.h, this list can be reduced in size while still allowing
to override these values.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
These files define only a single variable that is used in only one place.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
The logic to generate deftwmrc.c is currently split between Makefile.am and
deftwmrc.sed. Consolidate into a single script.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Provide declaration of yyparse(), and alternate declaration of yylex().
This works with bison and byacc, as well as different versions of flex.
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Replace the extern declaration of yylex() in gram.y with a
definition/use in parse.h, improving portability.
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
A recent change removed the declaration of yylineno from lex.l,
making the program depend on the version of flex.
Restore the declaration with an ifdef.
Also, remove unused declaration in parse.c
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
all of the users of (generated) gram.h rely upon (custom) parse.h,
and the order of those headers affects portability.
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
|
|
Still valid c, but now also valid c++.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
'new' is a keyword in c++. Its usage confuses some tools that work
with c and c++ source code.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
'class' is a keyword in c++. Its usage confuses some tools that work
with c and c++ source code.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
'class' is a keyword in c++. Its usage confuses some tools that work
with c and c++ source code.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
'class' is a keyword in c++. Its usage confuses some tools that work
with c and c++ source code.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
'class' is a keyword in c++. Its usage confuses some tools that work
with c and c++ source code.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
'delete' is a keyword in c++. Its usage confuses some tools that work
with c and c++ source code.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
'register' is an optimization hint to the compiler that is generally
not necessary and needlessly prevents using a c++ compiler to compile
twm.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Replace with a local variable following the naming scheme from
Identify() in src/menus.c.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Replace with a local variable following the naming scheme from
Identify() in src/menus.c.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Replace with a local variable following the naming scheme from
Identify() in src/menus.c.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
twm defines several "junk" variables to use with functions like
XQueryPointer or XGetGeometry. In some instances, the returned
values are actually used, which makes the code confusing and hard
to reason about. Use dedicated variables in those cases.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
twm defines several "junk" variables to use with functions like
XQueryPointer or XGetGeometry. In some instances, the returned
values are actually used, which makes the code confusing and hard
to reason about. Use dedicated variables in those cases.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
twm defines several "junk" variables to use with functions like
XQueryPointer or XGetGeometry. In some instances, the returned
values are actually used, which makes the code confusing and hard
to reason about. Use dedicated variables in those cases.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
twm defines several "junk" variables to use with functions like
XQueryPointer or XGetGeometry. In some instances, the returned
values are actually used, which makes the code confusing and hard
to reason about. Use dedicated variables in those cases.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
twm defines several "junk" variables to use with functions like
XQueryPointer or XGetGeometry. In some instances, the returned
values are actually used, which makes the code confusing and hard
to reason about. Use dedicated variables in those cases.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
twm defines several "junk" variables to use with functions like
XQueryPointer or XGetGeometry. In some instances, the returned
values are actually used, which makes the code confusing and hard
to reason about. Use dedicated variables in those cases.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
twm defines several "junk" variables to use with functions like
XQueryPointer or XGetGeometry. In some instances, the returned
values are actually used, which makes the code confusing and hard
to reason about. Use dedicated variables in those cases.
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>
|
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
Preparation for the removal of FirstScreen: Separate code that is
executed for all screens from code that is executed for only the
first screen.
Signed-off-by: Tim Wiederhake <twied@gmx.net>
|
|
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>
|