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

Footsteps without view bobbing (#5645)

* Remove redundant view_bobbing setting

Also fixes bug where disabling view_bobbing disables footstep sounds.

* Removes redundant view_bobbing setting

Setting view_bobbing amount to 0 is now the only way to
turn view_bobbing on and off. Also fixed a bug where footstep
sounds would not play when view_bobbing was disabled.

Fix various points reported by cppcheck (#5656)

* Fix various performance issues reported by cppcheck + code style (CI)

* Make CI happy with code style on master
* guiFileSelectMenu: remove useless includes
* some performance fixes pointed by cppcheck
* remove some useless casts
* TextDest: remove unused setFormSpec function

* Fix various iterator post-increment reported by cppcheck

Gitlab-CI: keep build workspace only 1 hour

Fix lint since recent player to db merge

Player data to Database (#5475)

* Player data to Database

Add player data into databases (SQLite3 & PG only)

PostgreSQL & SQLite: better POO Design for databases

Add --migrate-players argument to server + deprecation warning

* Remove players directory if empty

LINT fix

Revert "CONTRIBUTING: disallow signed git commits"

This reverts commit 7a6502a7a5007e4bed8c1dd10af39d4fa2b296a4.

Signed commits are supported by PPA and weblate now

Pass clang-format on various cpp/header files (#5559)

Network:Remove old opcodes and fix documentation. (#5573)

Mgflat, Mgv7: Fix noise crash on world exit.

Fix crash caused by destructor 'delete' on noise objects that are not
created due to mapgen options.
Crash was caused by commit 57eaf62c697cec91890d9cb28d10385d293d2d3f

Add /fixlight chat command

Mgflat, Mgv7: Only create noise objects if needed

Sneak glitch: Set default to false

The 'sneak glitch' physics override now controls whether a player can
use the new move code replications of the old sneak side-effects:
sneak ladders and 2 node sneak jump. This completes our intention to
replicate the old sneak side-effects in new code and provide them as
an option that is disabled by default.

Add on_flood() callback.

This callback is called if a liquid definitely floods a non-air
node on the map. The callback arguments are (pos, oldnode, newnode)
and can return a `bool` value indicating whether flooding the
node should be cancelled (`return true` will prevent the node
from flooding).

Documentation is added, the callback function was tested with a
modified minetest_game.

Note that `return true` will likely cause the node's `on_flood()`
callback to be called every second until the node gets removed,
so care must be taken to prevent many callbacks from using this
return value. The current default liquid update interval is 1.0
seconds, which isn't unmanageable.

The larger aim of this patch is to remove the lava cooling ABM,
which is a significant cost to idle servers that have lava on their
map. This callback will be much more efficient.

Fix a memory leak (#5636)

Fixing warning pointed by @sfan5 and clang and cleanup guiFileSelectMenu

* Also pass clang-format on guiFileSelectMenu.h and remove it from whitelist

Revert a const ref on update texture

if mod is a reference to a class member a variable swap breaks.
We should find a way to keep this const ref if possible.
Added a comment about this in header

lua: remove core.cause_error call (#5637)

it was used in minimal to trigger core crash, not very useful

Fix various performance issues reported by cppcheck + code style (CI) (#5635)

* Make CI happy with code style on master
* guiFileSelectMenu: remove useless includes
* some performance fixes pointed by cppcheck
* remove some useless casts
* TextDest: remove unused setFormSpec function

Fix #5617 - respect message and reconnect parameters when shutting down immediately (#5621)

Fix after soft node overlays

This removes a segmentation fault and makes node meshes well colorized.

Soft node overlay (#5186)

This commit adds node overlays, which are tiles that are drawn on top of
other tiles.