Adjust all includes outside of ui/.

Add flags in Makefile.am to make all compile.

There were code paths within CreateLine where the method did return a value,
and others where it didn't.  No caller of CreateLine actually considered
the return values at all.  Hence we may as well decide to not return anything
at all.  This should fix an issue reported by the undefined-behavior sanitizer.

Thanks to Carsten Gräser for pointing this out.

To get rid of tabs, mainly.  These two files were left out when moving
to git, because uncrustify wouldn't take them.  With a bit a manual
arm-twisting uncrustify now processes these files.

The (very) old DDD debugger with the (very) old motif gui cannot handle umlauts.

Compilers are much smarter than programmers, and know best when to use
CPU registers.  Leave the decision to them and do not clutter the code
with the 'register' keyword.

This used to be legal, but is not anymore.  In particular, clang
refuses to compile it.  This patch fixes the issue.

The explicit list of 8 bit numbers in the files tool0, tool1, etc
were stored in arrays of char. However, they contain values larger than 127,
which is illegal in C++11 -- the type must be unsigned char.
On the other hand, the char arrays are handed to X, which really does
expect signed char.
The patch fixes the problem by changing the types to unsigned char,
and explicitly casting the arguments of the X calls.