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.

Item entity: Delete in 'ignore' nodes

Falling.lua: Delete falling node entities on contact with 'ignore'

Prevents falling node entities entering the ignore at a world edge and
resting on unloaded nodes 16 nodes below, unreachable, undiggable and
still being processed by 'on step' because they don't revert to nodes.

map.cpp: Initialize NodeNeighbor, set NeighborType to u8 & cleanup

Cleanup:
* Drop unused Map::transforming_liquid_size()
* NodeNeighbor must use const ref for v3s16
* Add a missing default in a switch case

Node definition manager refactor (#7016)

* Rename IWritableNodeDefManager to NodeDefManager
* Make INodeDefManager functions const
* Use "const *NodeDefManager" instead of "*INodeDefManager"
* Remove unused INodeDefManager class
* Merge NodeDefManager and CNodeDefManager
* Document NodeDefManager

Position entity nametags relative to selection-box (#7031)

Fix 'Change Keys' menu crash caused by 501f936

Allow dumping userdata (#7012)

Switch F3 to use 'enable_fog' setting

Include alpha channel reference in MaterialTypeParam

Move `setlocale` from Lua to C++.

Add main_menu_style setting

Add minetest.is_player (#7013)

* Add minetest.is_player

* First use for is_player

Lua_api.txt: Document 'wielditem' visual in object properties

Item entity: Prevent motion in ignore nodes

Add kill chat command (#6992)

Replace minetest.* with core.* in 1 file

Make hud_get return aligment, offset and size. (#7006)

* Make hud_get return aligment and offset.

* Return size aswell.

Remove unused light updating code

Also remove the unit test that tests the removed algorithms.

Refine movement anticheat again (#7004)

* Account for walking speed in vertical dir
* Avoid undefined behaviour due to division-by-zero

Fix liquid bottoms not being rendered

Add Android drivers to the video_driver drop-down menu

Adds the Android video drivers ogles1 and ogles2 to the video_drivers drop-down menu

Add minetest.bulk_set_node call + optimize Environment::set_node call (#6958)

* Add minetest.bulk_set_node call + experimental mod unittest

* Optimize set_node function to prevent triple lookup on contentfeatures

Do only one lookup for old, and try to merge old and new lookup if node is same than previous node

* Add benchmark function + optimize vector population to have real results

Make chat command + privilege help slightly more accurate (#6964)

* Make chat command help slightly more accurate

* Slightly more accurate privilege help

* Simplify command/priv help

* More command/priv help tweaks

Apply physics overrides correctly during anticheat calculations (#6970)

[CSM] HUD Fix not updating server HUDs caused by 4f688d5

Fix issues with earlier CSM HUD commit (#6940)

The CSM HUD PR caused some strange behavior including aborts due to parts of it using some slightly hacky code, the event refactor changing how events are processed and a minor oversight.

Main menu: Change tabs to 'Start Game' and 'Join Game' (#6955)

Disable fall damage when "immortal" group set (#6946)

Add missing languages from menu (#6953)

The following languages exist in Minetest PO folder were missing from the selection menu:
- dv (Dhivehi)
- ms (Malay)
- sl (Slovenian)
- sv (Swedish)
- sw (Swahili)

Intersects_protection(): Move from Minetest Game to builtin (#6952)

A useful function that applies 'core.is_protected()' to a 3D lattice of
points evenly spaced throughout a defined volume, with a parameter for
the maximum spacing of points.

CSM: Remove screenshot API

Reverted from commit 19960e26c672c6337f8c6ffbe27f2c6bca49750c
(* [CSM] add screenshot api lua)

[CSM] Don't Load the package library (#6944)

Already removed by a latter step in CSM init so this just saves on pointless work.

Change include from "cmake_config.h" to "config.h"

Fixes #6894

[CSM] Remove `on_connect` callback (#6941)

Fixes #6939

[CSM] Add functions to create particles and particlespawners. (#6072)

Add minetest issue template (#6936)

* Add minetest issue template

This permits end user to have a basic template permitting to do issue triage and for users to know what we expect.

Game refactor [4/X]: keycache is now owned by InputHandler

* Make InputHandler own the key cache
* Add a helper function InputHandler::cancelPressed to avoid multiple similar calls in game.cpp
* Move RandomInputHandler::step definition into cpp file

Game refactor [3/X]: Move keycache to inputhandler

Game refactor [2/X]:  Various moves (profilergraph, nodePlacementPrediction, create_formspec_menu)

* Move profilergraph to dedicated files
* Move nodePlacementPrediction to Game class
* Rename create_formspec_menu to GUIFormSpecMenu::create

Game/Input refactor [1/X]: make RealInputHandler handle joystick inputs with standard input

Joystick input is a RealInputHandler only usage, make it intelligent and handle the joystick with keyboard direct.
This permits to remove many getters in game which should be owned by RealInputHandler

[CSM] Add basic HUD manipulation. (#6067)

* [CSM] Add basic HUD manipulation.

Workaround for on_connect not working right now.

Biomes: Add 'get heat', 'get humidity', 'get biome data' APIs

'get biome data' returns biome id, heat and humidity.
Clean up nearby lines in lua_api.txt.

Load a texturepack from the 'textures' subfolder of a game

Lua_api.txt: Various improvements

Improve documentation for formspec 'position' and 'anchor' elements.
Add missing documentation of 'animation' and 'glow' for particle spawners.

Mapgen folder: Update and improve copyright information of files

Lua_api.txt: Improve bullet point indentation consistency

Registration confirmation dialog: Fix grammar

Fixes commit 792752997c5ae2aaa4f54d0a2e2af2a96d7d1e9f.

Lua_api.txt: Add chat command params info

Add confirmation on new player registration (#6849)

* Attempt to add registration confirmation

Using SRP auth mechanism, if server sent AUTH_MECHANISM_FIRST_SRP that means the player isn't exist.
Also tell player about the server and chosen username.
Local game has localhost as IP address of the server.
Add RenderingEngine::draw_menu_scene() to draw GUI and clouds background.
aborted -> connection_aborted

* Rewrite information message text

Client::promptConfirmRegister() -> Client::promptConfirmRegistration()

Revert "Add an active object step time budget #6721"

This reverts commit 9c669016d1578a5c62f932c6ccb7a2b4b1e21f0a.
See #6907

Add a build step to test non freetype builds (#6908)

* Add a build step to test non freetype builds

Forget to fix non freetype build in StaticText

Lua_api.txt: Fix, improve and add to Object Properties documentation

Correct 'automatic rotate' to be a number instead of a bool.

Lua_api.txt: Improve and complete ABM documentation

Document 'active object count (wider)'.

Settingtypes.txt: Correct value of 'max block send distance'

Fix ambient occlusion and dark lines at mapblock borders