Fix signed/unsigned conversion warning

There was no bug here (as I checked for negativeness),
however it's good to get rid of warnings.

Cavegen: Fix non-constant array initializer error for MSVC

Settings: Update documentation (#5534)

Now documented (sorted a-z):
enable_console
enable_particles
creative_mode
hud_scaling
inventory_image_hack
keymap_console
keymap_zoom
shader_path
view_bobbing

Clang-format: fix some header files and remove them from whitelist

Add the REQUIRE_LUAJIT option (#5526)

Windows: Skip cmd for release builds (#5416)

Fix "error: ‘playerSAO’ was not declared"

Cause: 351cc2e79a7d78f7ec97ff9b33e4f0bad4042b19

A rebase issue missed a rename from playerSAO to playersao.

Tested.

Fix spacing

Fix problems when overriding the hand:
- If the hand can dig a node the item wielded can't, allow to dig it anyway.
- Fix the API callbacks from setting the hand instead of the wielded item.

Add Joystick type detection and Xbox controller support

* Add joystick type detection (with joystick_type setting to override it)
* Fix multiple joysticks from interfering with each other by only reading from one (add joystick_id setting)
* Add support for Xbox controllers

Fix multiple death messages (#5305)

Fix multiple death messages (#3565) and damage server logs after death.

Show media download progress in percent on progress screen (#5498)

LINT: Switch whitelist check from egrep to awk

Bonus: make CI happy with the last rules fix

[CSM] Add function to get the server protocol version. (#5529)

Clang-format continuation indent fixes + .gitignore additions

* Proper support for continuation indents in clang format
* make src/wieldmesh.h proper and remove it from whitelist
* Add CLion default build directories in .gitignore

Clang format brace style fix

Fix clang-format binary selection

Also fix spaces to tabs

Move LINT process in dedicated shell & fix
Move lint to dedicated shell permit to use it from your shell easily to check what is wrong
Also fix recent regressions in code style

Clang format: only show errors on non whitelisted files

Translated using Weblate (Indonesian)

Currently translated at 69.4% (638 of 918 strings)

Translated using Weblate (Russian)

Currently translated at 59.2% (544 of 918 strings)

Translated using Weblate (Italian)

Currently translated at 96.0% (882 of 918 strings)

Translated using Weblate (Portuguese (Brazil))

Currently translated at 99.4% (913 of 918 strings)

Se isso se referir a Instrument builtin aa tradução é embutido

Translated using Weblate (Turkish)

Currently translated at 87.0% (799 of 918 strings)

Translated using Weblate (Korean)

Currently translated at 67.8% (623 of 918 strings)

Translated using Weblate (Malay)

Currently translated at 37.0% (340 of 918 strings)

Translated using Weblate (Korean)

Currently translated at 66.4% (610 of 918 strings)

Translated using Weblate (Chinese (China))

Currently translated at 73.6% (676 of 918 strings)

Translated using Weblate (Malay)

Currently translated at 24.8% (228 of 918 strings)

Translated using Weblate (Serbian (cyrillic))

Currently translated at 34.9% (321 of 918 strings)

Translated using Weblate (Chinese (China))

Currently translated at 73.0% (671 of 918 strings)

Translated using Weblate (Polish)

Currently translated at 70.2% (645 of 918 strings)

Translated using Weblate (Malay)

Currently translated at 17.4% (160 of 918 strings)

Translated using Weblate (Chinese (China))

Currently translated at 73.0% (671 of 918 strings)

Added translation using Weblate (Malay)

Translated using Weblate (Chinese (China))

Currently translated at 72.9% (670 of 918 strings)

Translated using Weblate (Belarusian)

Currently translated at 100.0% (918 of 918 strings)

Translated using Weblate (Norwegian Bokmål)

Currently translated at 3.9% (36 of 918 strings)

Translated using Weblate (Belarusian)

Currently translated at 100.0% (918 of 918 strings)

Translated using Weblate (Belarusian)

Currently translated at 13.5% (124 of 918 strings)

Translated using Weblate (Polish)

Currently translated at 61.2% (562 of 918 strings)

Translated using Weblate (Indonesian)

Currently translated at 69.4% (638 of 918 strings)

Translated using Weblate (Lithuanian)

Currently translated at 27.1% (249 of 918 strings)

Translated using Weblate (Portuguese (Brazil))

Currently translated at 99.3% (912 of 918 strings)

[CSM] Add support for positional audio. (#5516)

Fixes parts of #5389.

Expose vector helpers to CSM

Memleak fix: LocalPlayer object was not deleted

Delete LocalPlayer when ClientEnvironment (object owner) is destroyed

Mapgen documentation: Add descriptions to noise parameters

Shorten 'readable names'.
Add a new advanced settings menu section for Biome API noises.
Various minor edits and improvements.

Mapgen: Remove '#include treegen.h' from non-mgv6 mapgens

Only mgv6 uses the tree functions from treegen.cpp.

MapgenBasic: Add lava source as commonly used content (#5512)

Future mapgens are likely to use this for magma and volcanos.
Remove the getting of lava source content id in mgvalleys.

Client handlers: Remove useless stringstream usage in two handlers (#5510)

[CSM] Add local node meta reference. (#5508)

Cavegen/Mgv5/Mgv7: Add optional giant caverns

Add to MapgenBasic for use by multiple mapgens.
Add to mgv5 and mgv7, enabled by default.

Similar to mgvalleys caverns but half the scale.
Parameters for upper y limit, distance caverns taper to full size, and
noise threshold (full cavern size).
As with mgvalleys caverns are generated first and classic caves are
disabled in any mapchunk containing a cavern, to avoid excessive
spreading volumes of liquids.
This also avoids floating blobs of liquid where a large classic cave
has overgenerated out into a neighbouring previously-generated mapchunk.

Update embedded jsoncpp from unk version to 0.10.6 + move libs to lib/ instead of src/ (#5473)

* Update embedded jsoncpp from unk version to 0.10.6

0.10.6 is last release without c++11

* Make jsoncpp more compliant with its amalgamate

Jsoncpp cpp file should be upper, make the library like it does in amalgamate

* Reorganization: move minetest embedded libs outside of source tree to /lib

* Fix a dead grep in LINT

minetest.after(): simplify further, pause in singleplayer (#5500)

Using the `dtime` value entirely, this will stop the clock
if the game is paused in singleplayer. Since most of the
clocks were fixed a long time ago, this should again be
safe to use.

First commit for fine pointed (#5485)

Add missing source to android build (#5496)

Clang-format: trivial fix on some headers (#5495)

Fix some headers style and remove them from whitelist

[CSM] Add function and chat command to disconnect from server. (#5487)

Signlike, glasslike drawtypes: Fix inverted textures

mapgen: Fix segfault when selecting invalid mapgen (#5491)

Fix clang-format Columns Width

Also fix l_client.cpp/h and remove them from whitelist

Wieldmesh: Natural orientation depending on pitch (#5490)

Windows: Set window icon (#5486)

Paramtype2: Add missing type CPT2_GLASSLIKE_LIQUID_LEVEL

Add the missing paramtype2 for param2 controlling the liquid level
inside the glasslike_framed drawtype.
Add missing documentation of the feature to lua_api.txt.
Update and improve comments for drawtype enumerations in nodedef.h.

l_minimap: don't show minimap if configuration doesn't allow it

Add CPP11 header to define nullptr & constexpr (#5471)

This header permit to use nullptr & constexpr keywords in portable code segments and benefit from nullptr & constexpr when using C++11 and greater

Sneak: Fix sneaking on free-floating lower-half slabs

Sneak: Fix stair lower step bug and ladder in 1x1 hole bugs

Add functions to strip color information. (#5472)

Fix a type bug in colorize function

This bug was introduced by dd2f1d7551bfd70357ad4f3fb180704194272147 and reported by @kilbith

[CSM] Fix errors in and improve the CSM documentation. (#5467)

clang-format: add a whitelist (#5459)

If file is in the whitelist, softfail, else hard failure

Some files are not in whitelist and marked as normal:
* src/content_mapnode.h
* src/cguittfont/xCGUITTFont.cpp
* src/gameparams.h
* src/profiler.cpp

PostgreSQL: fix delete block
Typo fix: we are doing select on delete block.

This fix issue #5426

Map generation limit: Make per-world

The setting limits map generation but affects nothing else.
Add 'mapgen_limit' to global mapgen parameters.
Move 'blockpos_over_mapgen_limit()' to the only place it is called
from: map.cpp.
Allow teleportation to any part of the world even if over the set
mapgen limit.
Simplify the reading of this limit in mgvalleys.
Remove the 'map_generation_limit' setting.

Pathfinder: Send errors to `warningstream`.

Avoids spamming the chat about several errors.

Fix mismatch struct/class for GameUIFlags

clang-format: fix counter increment & output (#5456)

clang-format: fix bash syntax on increment

clang-format: limit diff to 50 files to prevent exceding the maximum travis output (4MB) (#5455)

Sneak: Replicate sneak ladder in new code, is optional

Enabled using the existing 'sneak_glitch' physics override.

Change command prefix to "." and add "help" command.

Add mesh generation delay

Add clang format & skip build if no source file modified (#5433)

* [BUILD] Add clang format + build skipping

* Add clang-format tool to check codestyle.
Warning: it check the whole modified file, not the diff part, it's why it's lazy. Please also look if rules are perfect, i take the Linux codestyle from LLVM site

Fix issue #5415

* Skip building project if no file is modified
* Fix a wrong brace to trigger LINT

* Make lint step outside of unix build scope

* Add AccessModifierOffset: -8

* Typo fix & needs compile fix

* Fix header priorities

Tile.cpp: Fix MSVC build broken by 072bbba

Typo fix

Add multiline support to colorize. (#5444)

Some performance optimizations (#5424)

* Some performance optimizations

This is globally removing some memory useless copy

* use a const ref return on std::string Settings::get to prevent data copy on getters which doesn't need to copy it
 * pass some stack created strings to static const as they are not modified anywhere
 * Camera: return nametags per const ref instead of a list pointer, we only need to read it
 * INodeDefManager: getAll should be a result ref writer instead of a return copy
 * INodeDefManager: getAlias should return a const std::string ref
 * Minimap: unroll a Scolor creation in blitMinimapPixersToImageRadar to prvent many variable construct/destruct which are unneeded (we rewrite the content in the loop)
 * CNodeDefManager::updateAliases: prevent a idef getall copy
 * Profiler: constness
 * rollback_interface: create real_name later, and use const ref
 * MapBlockMesh updateFastFaceRow: unroll TileSpec next_tile, which has a cost of 1.8% CPU due to variable allocation/destruction,
 * MapBlockMesh updateFastFaceRow: copy next_tile to tile only if it's a different tilespec
 * MapBlockMesh updateFastFaceRow: use memcpy to copy next_lights to lights to do it in a single cpu operation

[CSM] Add function to get player names in range (#5435)

* [CSM] Add function to get currently connected player names

Firelike drawtype: Add missing docs for visual_scale in lua_api.txt (#5434)

Content_mapblock.cpp: Refactor

Refactored client event if-else cascade to switch statement

Map generation limit: Rewrite

The previous implementation applied the setting to blockpos_over_limit(),
objectpos_over_limit() and in createSector(), causing many bugs near the
world edge.

First revert the previous implementation.
Rename blockpos_over_limit() to blockpos_over_max_limit() for clarity.
Add a new function to mapblock.h called blockpos_over_mapgen_limit() that
checks against the map_generation_limit setting, and call this only from
the code that decides where mapgen stops.
Use MAX_MAP_GENERATION_LIMIT in objectpos_over_limit() to reduce the
chance of bugs, there is no need to use map_generation_limit here.

Fix /shutdown or ^C no longer causing server exit

Broken since b8484ef24e8e1ec90a2967372808ab2bea538c7c

Refactor Game class (part 2) (#5422)

* showPauseMenu is now part of game
* remove many flags parameters passed to game functions, use the member.
* rename VolatileRunFlags to GameUIFlags (this will permit to share structure with client and CSM
* updatePointedThing: remove pointer ref, we already have the pointer in rundata
* move some attributes outside of VolatileRunFlags after renaming, to game class
* rename statustext to m_statustext
* make some const variables static
* All those changes permit to reduce a little bit function class cost and will permit to interface CSM with some interesting Game flags
* Expose GameUIFlags to client
* Client now have GameUIFlags parameter and setters for other classes
* Fix minimap show/hide in Lua because we now have access to the real flag

Block access to the `io` library

Update server min protocol version to v24 (#5411)

* Update server min protocol version to v24

It's based on @sfan5 stats. See https://kitsunemimi.pw/tmp/serverlist_stats_2017-03-17.txt

v24 was bumped 25/08/14 and 0.4.11 was released 25/12/14

* Drop protocol v23 and lesser code

Server list cleanup

This removes the hacky server_dedicated pseudo-setting.

Show itemstring as tooltip if no item description

TOGGLE_CHAT keymap should togglechat not HUD

Fixes #5413

Reduce memory & function cost of Game class functions (#5406)

GameRunData is passed on many game functions, or one of its attributes whereas it's a member of the class. Remove it from functions arguments and call object directly from concerned functions.
This will reduce a little bit the Game class loop usage & very little bit the memory usage (due to non creation of pointer/references)