this might fix #5661, needs testing (#5775)

[CSM] Fix crash when the minimap is disabled. Caused by e25a38e

Fix input regression introduced by a4a377ecad6f1732cc54e7ac329fdef9949f9bf7

Fix #5776

Fix mem leak in mesh cache (#5781)

Various code cleanup & little performance improvement on HTTP download (#5772)

* Disable or remove unused enum members/functions
* Tiny code style fixes
* Make some functions const
* Replace ClientMediaDownloader std::unordered_map with std::map

Do not create dummy normalmaps (#4180)

fixes #1811

Fix textarea bug: clicking right of the end of a line caused the cursor to be in one of the next lines (#3474)

Fix alpha for liquid nodes (#5494)

Mainmenu: Fix issues while trying to enable all mods (#5770)

When minimap is disabled in configuration, really disable it (#5771)

* When minimap is disabled in configuration, really disable it

Fix a warning pointed by GCC 7.1

lua_tonumber overflow in snprintf (12 bytes and only 10 can be written)

Builtin: Fix subgame mod selection (#5367)

Particles: Do not send single particles to distant clients (#5760)

Previously, every individual particle on a server is sent to, and rendered by
(even if not actually visible), every client regardless of distance. This
significantly reduces client FPS and creates unnecessary network traffic.
Maximum distance is set by 'max block send distance' as this determines how far
a client is able to see.

Append -debug to version string (#5765)

CSM: Document forgotten functions

Caverns: Remove unnecessary liquid excavation

Also disable CavesRandomWalk at a safer distance from caverns.

Excavating liquids in cavern code is unnecessary as in practice we are already
successfully disabling the generation of liquid caves that could intersect
with caverns and cause excessive amounts of spreading liquids in caverns.

However to be safer this commit now disables liquid caves at a larger distance
from caverns, to compensate for liquid caves being able to generate up to a
mapblock beyond a mapchunk border.

Not excavating liquids in cavern code also allows a feature i am working on in
experimental new core mapgens, but also allows for more flexibility in future.

Fix a memleak pointed by @Zeno- in MeshUpdateQueue

This leak was introduced when added cache to MeshUpdateQueue and only occurs when leaving world.

Fix uninitalised variable in event.cpp (#5764)

Add option to use neither node highlighting nor outlining

No cursor lock when window inactive (#5754)

Android dependencies updates (#5755)

* irrlicht ogles 5122 -> 5145
* openssl 1.0.2j -> 1.0.2k
* curl 7.52 -> 7.54
* sqlite 3.15.2 -> 3.18.0

Limit properly the sound setting at updateSound runtime step (#5753)

* Limit properly the sound setting at updateSound runtime step

Fix #5026

* Add a comment

Move KeyList & InputHandler from game.h to client/inputhandler.h (#5752)

* Move KeyList & InputHandler from game.h to client/inputhandler.h

We have a header for inputs, move inputhandler class & related keylist object to it

Also introduce a cpp file for MyEventReceiver::OnEvent function in inputhandler.h because a so huge function doesn't needs to be inlined

* Pass clang-format on inputhandler.{cpp,h} (compatible)

Android build fix

Minor pause/escape menu improvements (#5751)

- Split movement keys into separate entries.
- Increase buffer size due to the above change.
- Change capitalisation.
- Remove code-style violating white spaces.

Replace inconsistent mentions of core.* with minetest.* (#5749)

Rework escape/pause menu (#5719)

* Rework escape/pause menu

- Remove build information
- Use current controls instead of default controls
- Add information about the current server in place of the build information
- Add text saying the game is paused to if in singleplayer mode.
rework pause/escape menu

* improve consistency + display server_name

Menu: Fix starting a server if the server address is an empty string. (#5742)

Minetest uses an empty string internal to signal that a server should be started.

Private nodemeta (#5702)

* Private node metadata that isn't sent to the client

minetest.deserialize: Throw error when argument not string (#5738)

Android: Fix crash while loading main menu (#5736)

Broken by 43d1f37

Add cancel button to password change menu. (#5720)

* Add cancel button to password change menu.

Move Pilzadam to previous developers. (#5735)

Revert custom player collision box and step height commits

These caused inability to pass through 2 node high spaces or step up onto slabs
or steps when a new client connected to an older server.

read_schematic_replacements: ensure fields are strings (#5726)

* read_schematic_replacements: ensure fields are strings

add a type check before reading strings on read_schematic_replacements deserializer

* throw LuaError instead of asserting the whole client

Custom step height: Fix implementation

Recent commit 45ab62d6a3d90ab3b97aec88251a766cb5dd1899 had a coding error that
made climbing out of water difficult due to an incorrect value of the step height
when not 'touching ground'.
It also incorrectly multiplied the custom stepheight by BS, resulting in being
able to step-up 2 nodes if set to the default of 0.6, or even 0.3.
Also the implementation was wrong because it customised the step height when
not 'touching ground', this step height is for a slight rise when catching the
edge of a node during a jump, and should always remain at 0.2 * BS.

Shaders: Remove unused water surface shader

Also remove hardcoded MTGame node.

The 'water surface shader' was duplicated shader code in preparation for
intended new water surface shaders. For development purposes the MTGame node
'default:water_source' had it's top tile assigned to 'water surface shader'.
Due to shader duplication this commit does not cause any change to shader
behaviour.

Player attrs: permits to remove an attribute by setting value to nil (#5716)

* Player attrs: permits to remove an attribute by setting value to nil

When doing player:set_attribute("attr", nil) remove attribute

Also remove a useless check on C++ API part (already done by checkplayer)

Fix #5709

LINT fix since d0678948165768472fc940c03e78cba787f49ea5

Player properties: Set correct default collisionbox

Recent commit b6f4a9c7e1a4f0bac66fd6f6ff844425ac775975 removed a hardcoded
player collisionbox which resulted on falling back to an incorrect default.
This stopped players walking through 2-node high spaces and made the player
slightly wider.

Improve docs for custom player collisionbox feature and reformat nearby lines.

Replace occurrence of luaL_reg in l_camera.cpp

Related to commit 41c5483. Replace an occurrence of luaL_reg in
src/script/lua_api/l_camera.cpp (added by commit de028fc).

Fix myfloor(0.0)

Clean up numeric.h and split FacePositionCache from it

I also optiized FacePositionCache a bit: I removed a map
lookup and vector copy from both branches of getFacePosition.

Make the player collisionbox settable

Use a settings object for the main settings

This unifies the settings APIs.

This also unifies the sync and async registration APIs, since the async
registration API did not support adding non-functions to the API table.

Fix codestyle since CSM Camera API

Add configurable key bindings for hotbar scrolling, and for changing volume.

Use stepheight from CAO instead of hardcoded value

[CSM] add `on_item_use` (#5544)

Make players respect makes_footstep_sound in the object properties

[CSM] Add camera API (#5609)

* [CSM] Add camera API
roper rebase & squash

* Address nerzhul's review

Clouds API: Fix yellow clouds at dawn and dusk (#5707)

Add missing server sending rule for bd921a7916f0fafc493b1c4d0eeb5e2bb1d6a7c2

Fix Android build (#5706)

* Fix Android build

Save minetest screen width/height options when modified (#5683)

* Save minetest screen width/height options when modified

* Add autosave_screensize setting (default true)

* Fix @SmallJoker comments

Add option to also check the center to `find_node_near` (#5255)

* Add option to also check the center to `find_node_near`

Translated using Weblate (Esperanto)

Currently translated at 31.9% (293 of 918 strings)

Translated using Weblate (Italian)

Currently translated at 96.0% (882 of 918 strings)

Translated using Weblate (Portuguese)

Currently translated at 66.9% (615 of 918 strings)

Translated using Weblate (Belarusian)

Currently translated at 100.0% (918 of 918 strings)

Translated using Weblate (Spanish)

Currently translated at 58.0% (533 of 918 strings)

Translated using Weblate (Spanish)

Currently translated at 57.9% (532 of 918 strings)

Added translation using Weblate (Swedish)

Translated using Weblate (Japanese)

Currently translated at 49.6% (456 of 918 strings)

Translated using Weblate (Japanese)

Currently translated at 49.4% (454 of 918 strings)

Translated using Weblate (Russian)

Currently translated at 60.1% (552 of 918 strings)

Translated using Weblate (Turkish)

Currently translated at 93.6% (860 of 918 strings)

Translated using Weblate (Malay)

Currently translated at 54.9% (504 of 918 strings)

Remove CMAKE -ffast-math flag from OSX/FreeBSD

Fixes issue: #4274

I have tested on MacOS 10.12.4

Requires testing on:
FreeBSD, Windows and Linux which I do not have access to.

LINT fix since cloud API merge

Directional coloured fog: Fix order of SColor components

'video::SColor pointcolor' was initialised with order RGBA instead of ARGB.
No change in behaviour as 'm_bgcolor' has alpha 255.

Clouds: Fix reddish clouds. Add missing alpha update

Fix accidental swap of red and blue components that caused reddish clouds
Add missing update of alpha in remoteplayer.cpp

Add function to get server info.

Fading soungs: Fix client crash on older servers

Set sky API: Add bool for clouds in front of custom skybox

Default true.
Add 'm_clouds_enabled' bool to sky.h, set from new bool in 'set sky' API.
Make 'getCloudsVisible()' depend on 'm_clouds_enabled' instead of
'm_visible' (whether normal sky is visible).

Sneak: Improve and fix various things

Remove useless `got_teleported`.
Fix jitter when walking against the sneak limits.
Fix damage evading on sneak ladders.

Sound API: Add fading sounds

Add clouds API

Optimize updateFastFaceRow processing by removing some TileSpec copy (#5678)

* Optimize updateFastFaceRow processing by removing some TileSpec copy

It permit to decrease this function from 54% runtime to 45% and reduce copy from 14% runtime to 12.5%
getTileInfo also reduced from 27% to 23%

* makeFastFace should use a const ref too

this trigger a const pointer need in the underlying function

Also fix some code style and prevent calculating 4 times the same position at a point

* Reduce a comparison cost for lights in updateFastFaceRow

Fix visual slide issue with set_detach, fixes #5620

content_cao: fix getPlayerControl structure copy on each step (#5677)

Also fix some codestyle issues around it.

Client & ClientEnvirnment: don't create fake events (#5676)

Instead of create fake events on the stack on each loop call (Game::run), verify is queue is empty or not and handle event directly if there is.

This prevents fake ClientEvent creation & memory allocations

Same fix is also applied on ClientEnvironment, & rename getClientEvent to getClientEnvEvent to match ClientEnvEvent object

Fix Travis/unittest broken since b662a45

Remove legacy unused define DIGGING_PARTICLES_AMOUNT

[CSM] Add event on_place_node API lua (#5548)

* [CSM] Add event on_place_node API lua

Fix Lint broken by b662a4577d692329b9ca83525e6039f2ddcd1ac1

Reorder TileLayer. (#5638)

Despite the split of TileSpec into TileDef and TileLayer, the
TileLayer struct is still 66 bytes large, and doesn't fit in
a single cacheline.

I'm moving the color member to cacheline 2, in the hope that it
is less used and the compiler loads all the hot members in a single
cacheline instead. Only color sits now in cacheline 2, all the
other members are in cacheline 1.

Note: is_color is probably rarely set, most nodes will likely
not use hardware coloring, but this may change in the future.
Ideally, this class is shrunk to 64 bytes.

[CSM] add screenshot api lua (#5674)

* [CSM] add screenshot api lua

Clean up getTime helpers

This increases size of the getTime return values to 64 bits.
It also removes the TimeGetter classes since the getTime functions
are now very precise.

Fix click-digging torches (#5652)

Torches are dug instantly again.
When the digging time is 0, a delay of 0.15 seconds is added between digging nodes. If the left mouse button is released, the delay is set to 0, thus click-digging.

Allow mesh and nodeboxes to wave like plants or leaves. (#3497)

We introduce a new value for "waving" - 2:

0 - waving disabled
1 - wave like a plant
2 - wave like a leave

Plantlike nodes will only allow waving = 1, but for leaves we will
permit both 1 and 2 since current minetest_game sets it to 1 for
all leaves. This makes it somewhat backwards compatible.

For mesh and nodebox, values 1 and 2 are both valid, and the node
can wave in both fashions as desired.

I've tested this with the crops:corn plants, which are mesh nodes,
and the results are really good. The code change is trivial as
well, so I've opted to document the waving parameter in lua_api.txt
because it was missing from there.

Nodeboxes likely will not wave properly unless waving = 2. However
it's possible that waving=1 may be desired by some mod developers
for geometries I have not tried, so the code will not prohibit
either value for mesh and nodebox drawtypes.

Add lua_api.txt documentation for this feature and document both
the existing functionality and the expansion to mesh and nodebox
drawtypes.

Fix Travis builds

Search user path for sounds (#5657)

Don't permit to copy Scripting classes (Client,Server,Mainmenu)

Allow scripts to get the client protocol version in non-debug builds. (#5649)

Fix incorrect formspec-tooltip doc, add detail in 'floodable' & 'on_flood' docs (#5660)

* Fix incorrect formspec tooltip documentation

* Improve `floodable` and `on_flood` documentation.

The original documentation did not specify that liquids should not themselves be floodable. This is probably something that should be mentioned.

Fix Android build since 2818d3f2244d2146a5cdb61cd41f6561c514f97c

Fix #5655 (#5658)

Client: Don't send `TOSERVER_RECEIVED_MEDIA` since it's not used anymore
Server: Handle `TOSERVER_RECEIVED_MEDIA` using `Server::handleCommand_Deprecated`

Tiny documentation fix. (#5659)

There are plenty of lines longer than 80 characters, and spliting the function declaration across two lines can be momentarily confusing.

Rename Scripting API files for consistency