Fixed a lot of GCC warnings

Fix Lugaru.rc for previous header renaming

Rename all C++ headers with .hpp extension

Also improve header inclusion to follow this logic:
- "Own" header comes first (`Folders.hpp` in `Folder.cpp`)
- Lugaru headers come next, in alphabetical order
- System headers come next, using the `<header>` format

Sorted all source files in folders

Stopped using Account pointers, and removed general difficulty setting (difficulty from account is always used)

Was supposed to be two separate commits, sorry I forgot :-/

Account active is now stored as an integer to avoid weird loops to find the right id

Replaced all uses of Account::active outside of Account by call to active() method

Removed all modifications of Account active from outside Account

Moved Account::active in Account class. Maybe it should be changed for a method and we only store its index instead of a pointer.

Improve formatting of devtools docs

Re-write of DEVTOOLS.txt

Mention devtools option in man page

Rename debug mode to dev tools

The new config is then "Devtools" and the command line option is
--devtools, -d.

Updating newdetail more often to avoid storing bad value, and added a newline at the end of the config file

Reading debug mode from config again, but command line option overrides it

Oops, should be ifdef not ifndef

Disabling exception catching when building debug, to ease gdb use

Attempt at fixing the crash in DoAnimation

Using at() instead of operator[] in currentFrame to get more precise crashes

Fixed a bug loading all persons with the same ID. I hope it will fix weapon problems

Oops, committed by mistake a temporary commenting of code

Fix 7s offset between shown challenge time and high score

Also properly rounds the float time so that 26.89s does not end up as 26 but 27.

Removed useless Menu:: prefix, replaced uses of char by strings

Moved everything related to Menu is Menu.h,cpp and Campaign in Campaign.h,cpp

Replace MenuItem::init with a constructor

Changed namespace Menu for a class

Updated information in DEBUG-OPTIONS.txt

Alt+Y and alt+U are still undocumented.
Documentation about editor commands seems also inaccurate, and the code
seems buggy, shift+p will add a player AND a waypoint to this player it
seems to me.

Added a warning when loading a map with an old map version

Fix a crash when map contains a negative number of hotspots

Cleaned up a bit debug keys, ALT is always needed now except in editor.

Debug mode is now a comandline option

Declare Tempanim animation properly

Fixes #36.

CMake: Default to RelWithDebInfo build type when not specified

Also cosmetic changes.

CMake: Fix missing data install on Windows

Regression from 4afbc86d0abd136357e88d8c1fe2360a119289e3

Added a class for Hotspots to avoid using several arrays

Removed empty destructors. Joint::label is a bodypart

Removed unused methods from Skeleton

Replaced muscles and joints in Skeleton by vectors

Got rid of malloc in Muscle

Moved Joint and Muscle classes to their own files

Using the right type for animation height and type

Added convenient method to get target and current frames of Person animations

Cleaned up Animation by using vectors and classes

Moved Skeleton and Animation to their own folder.

Moved Animation class in Animation.* files.
Using a vector for animations.
Deactivated the deallocation for now, probable memory leaks, will be
fixed by futur commit

Install more docs (AUTHORS, DEBUG-OPTIONS.txt)

Readd -pedantic to CXX flags for all builds

It was removed a bit too fast in 20e924d, it's actually an interesting
flag to keep even for non-debug builds.

Silb assets are also under version 3.0 Unported of CC-BY-SA

At least judging by the version he linked in his forum statement.

Replaced as much char* as possible by std::strings

Moved all calls to skeleton Load to Person

Add AppStream appdata/metainfo file

This completes the set of metadata files installed for Linux systems.

Fixes #27.

Write config and user data to %APPDATA% on Windows

We may want to readd a PORTABLE option that reverts to loading the config
and user data from the binary's directory for portable binary distribution.

Last part of and fixes #22.

Fix CXX flags in various CMake build types

The flags we were defining for _DEBUG and _RELEASE were similar to the default
CMake flags for those types, which get appended to CMAKE_CXX_FLAGS, so we only
need to customize the latter.

The final flags will be for e.g. RelWithDebInfo:
${CMAKE_CXX_FLAGS} ${CXXFLAGS} ${CMAKE_CXX_FLAGS_RELWITHDEBINFO}
with ${CXXFLAGS} taken from the environment.

Add manpage for Linux system installation

Fixes #28.

Add and install desktop file and icon

Already added the install instruction for the appdata file for @Conan_Kudo.
Fixes most of #27.

Move distribution-related files to Dist

Also delete useless or obsolete files, as well as a binary .aps file
from Visual Studio.
Also fix .gitignore ignoring any lugaru*.* file :)

Add proper system-wide install instructions for Linux

The behaviour is triggered by the SYSTEM_INSTALL option (currently defaults to OFF).
Fixes #13.

(cherry picked from commit 46cd66c888bd9cc5df4f6eb59abce429fdd6cd56)

Moved as much as possible init in Person constructor

Fixed weird broken if/else/if

Fixes #8 Removed glstubs.h and its uses

Fix hardcoded clothes paths in maps

Also fix loading those paths with the proper Folder function.
Remove bogus piece of cloth "1.0 1.0 1.0.png".
Part of #21.

Trying to handle a bit better missing files and avoid segfaults

Also using multiplatform SDL_ShowSimpleMessageBox instead of Windows
 only MessageBox

Data files use title case consistently

The data lookup is now case sensitive, so we make it so that all files
use a title case (e.g. ShirtFemale.png).

Caveat: Paths to clothes are hardcoded in every binary level file as
:Data:Textures:shirtfemale.png or similar, they need to be fixed somehow.

Fix regression in fallback logic on Linux

Regression from 11ff9aec2c3206a5b9df5bf8815051458628c45a

Rename DATADIR to DATA_DIR to avoid conflict

On MinGW it would error out with:

Source/Utils/Folders.cpp:33:45: error: expected primary-expression before ';' token
 const std::string Folders::dataDir = DATADIR;
                                             ^

The issue seems to be that MinGW (and likely MSVC) is case insensitive
and would see DATADIR as conflicting with dataDir.

Trying to add implementations for MacOS and Windows

Dirs are now created if missing under GNU/Linux

Switched all data access to the new methods

Removed the ugly define on fopen
Still missing:
- Creating the folders we want to write in
- Finding the path of the binary and use it as base on Linux, with locateCorrectFile

First step towards using clean methods and correct paths for datas and config

Merge branch 'master' of gitlab.com:osslugaru/lugaru

Remove license info obsoleted by relicensing

51203b531b2054a8a7c40356e2108a5ec7b9f366 effectively relicensed all
Wolfire assets under CC-BY-SA 3.0, so this last paragraph is obsolete
and was just forgotten during the relicensing.

Rename "Debug\ Readme" to DEBUG-OPTIONS.txt and move out of Data

Also mac2unix :)

Main campaign: Delete unused German translation of April scene

DialogBox is protected on Windows, rename to DialogScene

Also adapted variable names accordingly and even the fields
in the dialog data files.

Convert campaigns and dialogues to Unix format

Also add .gitattributes to enforce LF line endings.

Fix libpng iCCP warning about invalid sRGB profile

Using ImageMagick or pngcrush seemed to introduce issues, so I had in the end to
fix the sRGB profile manually for each problematic image in GIMP.

Removed weird test for dialogues. Only attackkey should be used.

Cleaned up dialog handling using a Dialog class

The fixdialogue command will behave a tiny bit differently as it will
set participantfacing value as the dialogue would.
The previous behavior would crash if the number of boxes changed so I
could not keep it.
I kept the limitation of participants of 10 for now, does not seem like
a big deal.

Tiny clean

Cleaned a bit OPENAL_SetFrequency, fixing some clang warnings

It would be best to check if calls to OPENAL_SetFrequency are really needed

Person::occluded should be an int

Removed useless code

Fixed clang warnings

Cleaned up FootLand method

Moved Person loading from file into a Person constructor

Removed unused member of Weapon class

Added pedantic to debug build options to avoid problems such as the VLA one in the future

Removed ununused field hasAlpha

Avoid using Variable Length Arrays (which are refused by clang)

Fixed some small sonar warnings

Fixed a clang warning

Replaced hardcoded environment values by appropriate define

Cleaned a bit flashing by using a function

Cleaned up handling of envsounds array through a function

Removed unused globals

Removed unused members of Person class

Added a constructor in Person class

For now it mostly put 0 in all members to avoid uninitialized int,
 later it should put default values instead of GameTick.cpp

CI: Remove use of copr for mingw-openal-soft

It's now part of the Fedora repositories

Switch to using «The Lean Mean C++ Option Parser»

Options now requires two dash before long option name.
Some option names were slightly changed, you can see all of them with "lugaru -h"
See http://optionparser.sourceforge.net/index.html for info on the parser used
Fixes #14

We may want to add a --debug option to avoid having to edit the
config.txt file for this?

Renamed OpenGL_Windows.cpp into main.cpp as it contains main()

Removed ctrl+q binding. All OS have their own binding for window closing.