Small cleanups

Fixed double set of velocity var

Added parenthesis to expression

Fixed warnings, removed unused attributes, broken audio output setting, and so on

Fixing some warnings of cppcheck

cppcheck --quiet --enable=warning,style,performance,portability Source/

Fixing some warnings

Cleaned up a bit Tutorial::active ifs in Person

Fix opacity for splintersprite and always-true condition

As spotted by @ignatenkobrain in #69.

Removed useless code in GameInitDispose

Settings: Fix string length for strncmp (again)

don't set same variable twice

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

fix length for strncmp()

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

remove recurring checks

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Fixed methods signatures to fix build

Moved doAI in Person class

Had to change weapons[i] for weapons[k) which might change behavior,
test needed. (but weapons[i] is bogus anyway, i was a person id)

CMake: Switch to --std=gnu++11 to fix builds with POWER

On POWER architectures (ppc64/ppc64le), the AltiVec SIMD instructions
overload bool with vector ints, causing builds to fail because vector
data cannot be cast to scalar data. However, this only happens on c++11
mode. In gnu+11 mode, the usage of AltiVec overload attributes is
contextual, so it doesn't cause builds to fail anymore.

Cleaned a bit mouse button shown names, added names for button 4 and 5.

Fixes #62 fixed button names display

Note that this cause problem because of the incomplete font we use:
I mapped the key «é» to draw weapon and it can’t be shown in the menu.
We should switch to a more complete font

Cleaned up a bit Tutorial::DrawTextInfo

Renamed method because DrawText is reserved on some windows build

Moved visibleloading check inside LoadingScreen

Merged identical switch cases in Tutorial

Moved things related to tutorial in new Tutorial class

Add script to fix hardcoded strings in maps

Usage instructions in the script.

Cleaned up TextureRes lifecycle to make sure it’s destroyed by RAII

Ditched unused private list in TextureRes

Object type is now an enum

Moved checkcollide methods in Object

Cleaned up checkcollide methods

Removed forgotten debug output

Removed unused method Checkcollide

Cleaned up Objects handling

Checkcollide in Object is unused.
Game::checkcollide is used instead, it should be cleaned and moved to
Object class.

Deleted duplicated unused method

Fixed a few ifs

Cleaned up console printing code, replaced global displaytext by local vars in Menu

Cleaned up a bit Text class. Using glPrintOutlined everywhere.

Improved error reporting of screenshot saving, fixed screenshot path.

Removed useless STUBBED define

CMake: Use correct path for Resources in app root for macOS

CMake: Fix SDL2 detection for macOS

Added some security to avoid dividing by 0 if joints vector is empty

Call to static method should be static

Seven Tasks: Revert task18 to unfixed version

Looks like dehardcoding the path to world.png broke it, I will have to
investigate deeper.

Add modders to the credits file

CMake: Decrease required version (3.0) and use pkgconfig to detect SDL2

Some distros (e.g. CentOS 7+EPEL) do not provide the sdl2-config.cmake file
necessary for `find_package`.

Others still provide CMake 3.0 which works just fine (at least for Linux
compilation).

Also install Seven Tasks readme.

Seven Tasks: Fix hardcoded clothes paths

Still unfixed in task15, various references to:
:Data:Textures.png

Removing them requires removing the whole chunks about clothes
information, so it is a bit more involved.

Add The Seven Tasks campaign by Philtron R./skraeling

It is licensed under CC-BY-SA 4.0 International with attribution to
Philtron R., as mentioned on the Lugaru forums:

> *Note:** This mod (campaign, maps and custom textures) is licensed under
> the Creative Commons Attribution - Share Alike license, version 4.0
> International (CC-BY-SA 4.0, https://creativecommons.org/licenses/by-sa/4.0/)
> Attributions can be made to "Philtron R.".

Permalink: https://web.archive.org/web/20161214063840/http://forums.wolfire.com/viewtopic.php?f=7&t=4633

Fix GCC5 warnings (-Wsign-compare and -Wmaybe-uninitialized)

Add original Temple readme and fix doc install

Move some doc to Docs/ and apply folder naming scheme

Add info for packagers and tweak Mac info

Also document previous commit.

Save progress upon level completion

Fixes #47.
Also made Folder::getUserDataPath() inline, hope that's good style.

Empire: Specify licensing in documentation

Also use permalink from the Wayback Machine for the Temple
license statement.

Also update release notes accordingly.

Empire: Fix hardcoded clothes paths

There is only one path I did not fixed, as it triggered a weird texture
bug on the main character, Ash (as Ben had told me by email). It can be
fixed with sed to trigger the bug:

# :Data:Textures:blackfur.png / Textures/Fur2.jpg (fix)
sed -i 's/\x1b\x3a\x44\x61\x74\x61\x3a\x54\x65\x78\x74\x75\x72\x65\x73\x3a\x62\x6c\x61\x63\x6b\x66\x75\x72\x2e\x70\x6e\x67/\x11\x54\x65\x78\x74\x75\x72\x65\x73\x2f\x46\x75\x72\x32\x2e\x6a\x70\x67/g' Data/Maps/jendraz*

Add the Lugaru: Empire campaign by Jendraz

It is licensed under CC-BY-SA 4.0 International with attribution to Jendraz,
as mentioned on the Lugaru forums:

> Note: This mod (campaign, maps and custom textures) is licensed under the
> Creative Commons Attribution - Share Alike license, version 4.0 International
> (CC-BY-SA 4.0, https://creativecommons.org/licenses/by-sa/4.0/)
> Attributions can be made to Jendraz.

Permalink: https://web.archive.org/web/20161213062036/http://forums.wolfire.com/viewtopic.php?f=7&t=3797

Add release notes for upcoming 1.1 release

Fixes #19.

Got rid of char[] uses in GameDraw.cpp

Trying to avoid crashes in Weapon::DoStuff

Fixes #41 It was hard to click on the next campaign level when it’s above an old one

Add self-made FurBW.jpg for Temple campaign

The furbw2.png file from the original mod was obviously a mix between
the Fur.jpg and Fur2.jpg of Lugaru, so I tried to redo it since furbw2.png
was sadly not licensed under CC-BY-SA 3.0.

The furwhite.png image was just the same as Fur.jpg, so I just changed
the hardcoded references to it.

Part of #26.

Add self-made replacement for Temple Cl*.png assets

The original assets were not licensed CC-BY-SA 3.0 and thus not added here.
To redo them, I used the AR PL UKai TW font under the Arphic Public License
(https://www.freedesktop.org/wiki/Software/CJKUnifonts/Download/).

Note: the Chinese characters are flipped horizontally so that they appear
properly on the back of players.

Part of #26.

Adjust macOS build information to new minimum OS version

Drop _EXT suffix from various GL_TEXTURE_* names

Add missing C standard library headers

abs() -> fabs()

Convert C-header includes to C++ variants in C++ files

Remove usage of deprecated gl(Un)LockArraysEXT()

Fix OpenGL and OpenAL header paths for macOS

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.