`char` is unsigned on some platforms, which breaks the assumption it
can hold value -1. The simple fix is to replace it by int8_t, which
gives compatibility with x86 platforms where Descent3 usually was
delivered on.
Descent3/attach.cpp:668:47: warning: array subscript has type "char" [-Wchar-subscripts]
This type of console is enabled only with `-svgalib` and `-dedicated` options. Since there no svgalib direct support (which is pretty old and too specific to Linux tech), it's better completely remove this code for simplicity and reducing external dependencies.
Removed `-svgalib` option as unused now.
This definition was used to control the accessibility of some class members, changing protected qualifiers to public. This introduced unnecessary coupling between components and headers.
All conditional access specifiers have been set to public, which should not be a problem given the low number of classes that actually used affected members. Another albeit more complex solution could have been to use friend classes.
Join the license header with historical comments using a separator so IDEs can correctly parse the initial header.
Also use .gitattributes to ensure all files are LF.
