Fix a warning introduced by c5b4e541749c50805519ce040d98a0a8e5e0ec03

Fix a warning reported by clang

minetest/src/script/cpp_api/s_player.h:27:1: warning: struct 'PlayerHPChangeReason' was previously declared as a class [-Wmismatched-tags]
struct PlayerHPChangeReason;

Fix various Client class functions not marked as override (virtual)

Also switch some of them which are reported by clang-tidy

Load dependencies and description from mod.conf

Add reasons to on_dieplayer and on_hpchange

Add formspec theming using prepended strings

Update jsoncpp to 1.8.4 (#7172)

Guard sound manager initialization with "enable_sound" (#7167)

macOS: don't require X11 libraries during compilation (#7149)

The xxf86vm needs to be removed from Apple builds to avoid CMake Error XXF86VM_LIBRARY is NOTFOUND

Mapgen: Remove unused mgv7 code and some unused biometypes

Make sounds stop playing when entering game or mainmenu

Cleanup sound manager class (#7158)

* Cleanup sound manager client

* Use some const refs
* Use auto on iterators
* Drop unused parameters
* Move sound_openal.* to client folder
* Move sound.cpp + OnDemandSoundFetcher to client/ folder + reorganize includes properly

Print the mod loading time (#7137)

* Print the mod loading time

Fix some gcc 7.3 reported warnings

```
src/client/joystick_controller.h:85:2: warning: duplicate 'virtual' declaration specifier [-Wduplicate-decl-specifier]
        virtual bool isTriggered(const irr::SEvent::SJoystickEvent &ev) const;
/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/7.3.1/../../../../include/c++/7.3.1/ext/new_allocator.h:140:22: warning: destructor called on non-final 'JoystickAxisCmb' that has virtual functions but non-virtual destructor [-Wdelete-non-virtual-dtor]
                destroy(_Up* __p) { __p->~_Up(); }
/usr/bin/../lib64/gcc/x86_64-pc-linux-gnu/7.3.1/../../../../include/c++/7.3.1/ext/new_allocator.h:140:22: warning: destructor called on non-final 'JoystickButtonCmb' that has virtual functions but non-virtual destructor [-Wdelete-non-virtual-dtor]
        destroy(_Up* __p) { __p->~_Up(); }
```

Global initialization of sound using SoundManagerGlobal (#7063)

* Global initialization of sound using SoundManagerGlobal

Update mesh collector and move it to a separate file (#6904)

* Update MeshCollector

* Simplify MeshCollector

Gitlab CI: Disable currently broken ubuntu 16.10 & 17.04 builds

I will try to repair them later

Lua_api.txt: Improve noise documentation

Minetest ASCII art: Move from actionstream to rawstream

Mgfractal / chunksize: Improve documentation

Lua_api.txt: Fix hardware colouring documentation

More 'draw type' -> 'paramtype2' changes, missing from a previous commit.

Add src/unittest/test_world/world.mt to gitignore

Fixes #7146

Add missing file to android build

Fix node-nodebox lighting difference in direct sunlight (#7061)

.gitignore: ignore more files (android, unittests)

Lua_api.txt: Split long lines part 6

Fix a crash or random memory leak when reseting saved environment variable in test_servermodmanager.cpp

Drop Server::m_enable_rollback_recording it's only used in server constructor

ServerEnvironment::loadDefaultMeta: Loading default meta is only possible for ServerEnv itself

Forgot to remove obsolete Server::m_mods

Server: delegate mod management & config to ServerModConfiguration (#7131)

* Server: delegate mod management & config to ServerModConfiguration (rename it to ServerModManager)

* Use c++11 range based loops
* Add unittests + experimental/default mod as a test case to permit testing mod loading in future tests

Fix an alone if to be with a missing else

Lua_api.txt: Split long lines part 5, fix hardware colouring docs

Biomes: Fix vertical biome blend

README: Fix Fedora dependencies (#7133)

Lua_api.txt: Split long lines part 4

Rename 'subgame' to 'game'

Lua_api.txt: Improve and complete drawtype documentation

Move documentation of mesh and plantlike_rooted drawtypes into main section.

Lua_api.txt: Split long lines part 3

Biome API: Add 'get_biome_name(biome_id)' API

Change name of default biome to a more suitable lowercase 'default'.

Add Voxelarea unittests (#7121)

* Add voxelarea unittests

Android: typo fix since recent gradle update, as output path changed

Finally fix android build

Fix Android build, but there is a remaining linking issue in guiConfirmRegistration

* Also fix variable name overloading in guiConfirmRegistration

Android build fixes for c++11

Android: update tools (gradle, NDK) & prepare to switch to clang

Drop texture file list cache (#6660)

Fix typo in lighting code since bcdb3d5

Lua_api.txt: Split long lines part 2

Spawn level: Add 'get_spawn_level(x, z)' API

Returns a suitable player spawn y co-ordinate for unmodified terrain.

Formspecs: Unify textarea and field parsing functions, fix wrong fallback text

* textarea[], field[]: Unify function, fix wrong fallback text
* Remove apparently superflous mainmenumanager.h incldue
* intlGUIEditBox.cpp: make read-only boxes really read-only
* Use elseif (trivial)

VoxelArea: add_{x,y,z,p} must be static

Fix some documentations issues
Use getNodeNoCheck(v3s16, ...) in some cases instead of getNodeNoCheck(x, y, z, ...)

Drop unused struct ModLicenseInfo & ModAuthorInfo + default constructor in ModMetadata

ServerEnvironment & StaticObject cleanups

* isFreeServerActiveObjectId is now part of ServerEnvironment
* getFreeServerActiveObjectId is now part of ServerEnvironment
* StaticObject constructor now take ServerActiveObject instead of type + string. This permits to remove a big string copy in some code parts

Drop unused Server::m_ignore_map_edit_events_peer_id

Variable name fix + structure creation unrolling in lighting code

Drop less performant Server::setBlockNotSent for ClientInterface::markBlockposAsNotSent

Lua_api.txt: Split long lines part 1

Gennotify docs: Clarify returned positions for decorations

Biomes: Document xyz biome limits

Getv3intfield: Fix logic of return bool

Biomes: Add 'min_pos'/'max_pos' xyz biome limits

'y_min' and 'y_max' are still accepted for compatibility.

Cleanup & bugfix

* ObjectRef::set_local_animation: fix wrong lua return (should push a boolean, currently returns nil)
* ObjectRef::set_eye_offset: fix wrong lua return (should push a boolean, currently returns nil)
* Fix various Server functions which depends on RemotePlayer objet and return true/false when player object is nil whereas it's a caller implementation error. Change those bool functions to void and add sanitize_check call instead. Current callers are always checking player object validity
* Optimize Server::setClouds : use CloudParams object ref instead of attribute deserialization from structure & perform RemotePlayer::setCloudParams directly in server class like many other calls
* Optimize Server::SendCloudParams: use CloudParams object ref instead of deserialized attributes

Cleanup: drop Server::hudGetHotbarSelectedImage()

Call directly accessible RemotePlayer::getHotbarSelectedImage() from server api

Cleanup: drop Server::hudGetHotbarImage()

Call directly accessible RemotePlayer::getHotbarImage() from server api & make it const ref

Cleanup: drop Server::hudGetHotbarItemcount()

Call directly accessible RemotePlayer::getHotbarItemcount() from server api

mods.cpp/h: little performance improvement in getModsInPath (+ codestyle) (#7108)

* mods.cpp/h: little performance improvement in getModsInPath

Fix bad markdown in lua_api.txt

Fix missing warningstream (or similar problem) (#7034)

Use the --color command line parameter instead of a setting for coloured logs

This fixes the missing warningstream bug, g_settings->get mustn't be used there.
Also, the decision about en- or disabling log colours fits better to the command line parameters than minetest settings.

Allow for getting world name and path separately on the command line (#6555)

Change to --worldlist instead of --world list.

Gets rid of --worldpath parameter added as part of this pull request,
instead moving the listing function to a command --worldlist that
accepts either name, path, or both and prints out the corresponding
information.

Gennotify: Add 'minetest.get_decoration_id' API

Returns the decoration ID for the provided decoration name string.
For use with gennotify, to know the decoration IDs for use in
'minetest.set_gen_notify'.

Generate Notifier: Clear events once after all 'on generated' functions

MapgenValleys: Fixed submarine valleys shape

c_converter: Function template for numeric fields, add v3s16 default (#7090)

Slippery: Simplify, make more efficient (#7086)

Use already existing collision results for the nearest colliding node
Fix slippery effect in free_move mode

Cleanup in flat lighting (#7051)

Fix liquid post effect colour behaviour in third person view

Place schematic (on vmanip): Enable use of 'place center' flags

For 'place schematic' and 'place schematic on vmanip' APIs.
Fix 'place center' code to properly centre schematics.
Fix some comments.

CollisionMoveSimple: Collide with 'ignore' nodes

SAO limits: Allow SAOs to exist outside the set 'mapgen limit'

Fix CMake error (#7074) by bumping the required CMake version past 3.8.1 (#7075)

is_area_protected: Rename from intersects_protection (#7073)

* is_area_protected: Rename from intersects_protection
Return first protected position
Clarify docs: Mods may overwrite the function

CMakeLists use the source_group command to improve look and feel when generating Visual Studio projects.
The TREE argument of source_group appears in CMake 3.8 therefore check for version.

CMakeLists pass windows sources (manifest and resource file) to executable targets separately from common_SRCS

Registration dialog: Larger text field with scrollbars (#7047)

Center text, text area doubled now

Find nodes in area (under air): Raise volume limit and document it

Settingtypes.txt: Fix valleys dungeon ymax error

Vertical biome blend: Tune blend patterns

Dungeons: Add Y limits in all mapgens

Preserve the upper limit used in mgvalleys.

Delete world dialog: Move buttons to avoid double click deletion

Move confirmation delete button to never overlap initial delete button,
to avoid world deletion by accidental double click.

/shutdown can't do countdown when using reconnect and/or shutdown message (#7055)

Delay was converted from the param string and not the delay value, thus never using the actual given delay value when used in combination with other string values in the param, in this case reconnect and the shutdown messsage.

Add a refresh button to the serverlist (#6957)

* add refresh button

* Make search and refresh buttons smaller

* Change to image button

Mitigate formspec exploits by verifying that the formspec was shown to the user by the server. (#6878)

This doesn't check the fields in anyway whatsoever so it should only be seen as a way to mitigate exploits, a last line of defense to make it harder to exploit bugs in mods, not as a reason to not do all the usually checks.

Lua API documentation: Fix spelling (#7046)

Check argument types inside MetaDataRef Lua API (#7045)

Fix "Ignoring CONTENT_IGNORE redefinition" warning (#4393)

minetest.override_item still passes to core

Remove legacy chat network code. (#6954)

Render copyright notice: Fix name spelling (#7042)

Add `on_auth_fail` callback (#7039)

Called when a client fails to supply the correct password for the account it's attempting to login as.

Dungeons: Avoid generation in multiple liquid nodes and 'airlike'

Previously only 'mapgen water source' and 'mapgen river water source'
were checked for. Games can use multiple liquid nodes defined for biomes,
many of which will not be aliased to those 2 mapgen aliases, causing
floating dungeons to generate in some liquids.

Now we check for liquid drawtype instead, so can remove liquid nodes
from dungeonparams.

Also check for 'airlike' drawtype instead of 'CONTENT_AIR' to avoid
generation in 'airlike' nodes in some rare situations. This will also be
needed for when we add definable biome air nodes.