[hatari-devel] patch: Add some missing include directives

[ Thread Index | Date Index | More lists.tuxfamily.org/hatari-devel Archives ]


This is a multi-part message in MIME format.
Hi,

while experimenting with emscripten, i found some minor issues not directly 
related to emscripten itself, but merely due to the fact that it is using a 
different c-library. In some places, macros/functions where used without 
excplicitly including the corresponding header files. Attached patch should 
fix that.

There are also some minor things that could be fixed:

- in .gitignore, Debug/* should be added, just like Release/*
- the configure script deletes `dirname $0`/CMakeCache.txt but imho that is 
nonsense. CMakeCache.txt is written to the current directory, not to the 
directory configure resides in. If you build in a dedicated directory like 
Release, that won't remove anything. Same for the CMakeFiles directory.

The clang compiler used there also spits out several warnings, especially in 
the debug build, but those are not addressed yet.






Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/