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.

Gitlab-CI enhancements (#5629)

Fix various performance issues reported by cppcheck (#5628)

* Also remove 1 non declared but defined functions

Run generate_from_settingtypes.lua

Light update for map blocks

This is not really different from the light update of a voxel
manipulator. This update does not assume that the lighting was correct
before, therefore it is useful for correction.

Also expose this function to the Lua API for light correction, and
allow voxel manipulators not to update the light.

Split light update into two parts

The common part can be reused.

Fix various copy instead of const ref reported by cppcheck (part 3) (#5616)

* Also remove 2 non declared but defined functions
* Make some functions around const ref changes const

Fix various copy instead of const ref reported by cppcheck (#5615)

* Also remove InventoryList::peekItem unused function
* Fix some post increment to preincrement reported by cppcheck

Conf.example: Move some lines to minetest.conf.example.extra

Some information in conf.example cannot be generated from
settingtypes.txt, moving it to a new file makes generating
conf.example while preserving that information easier.

Regenerate conf.example from settingtypes.txt.

Pointed thing to face pos: Fix crash if opening door with slab or stair

Avoids crash caused when 'pointed thing -under' and '-above' are not
face-neighbours, for example in the case of pointing to the top half
of a door.

Tools: Fix tool digging speed limit

Fix various variables passed by copy instead of const ref (#5610)

Pointed by cppcheck

Fix broken lint since 04cc9de8f2fbcb11f133c88f02fc11504b3ea6f3

Reorder TileSpec. (#5591)

Put accessed members that are needed by updateFastFaceRow()
all in the same cacheline.

Fix always using the xbox layout (reported by coverity).

Don't make TAB exit game if bound to inventory.

I play with the TAB key bound to the inventory. However, the
code here assumes that TAB means "close formspec" in all contexts,
including the main menu. This causes my game to exit when I attempt
to TAB in between USERNAME and PASSWORD fields.

We know when m_client != NULL that the game is a client game and
not in the main menu, and then it's OK to use the INVENTORY bound
key to exit the formspec, since it's not the main menu.

Fix MSVC build broken by 34d32ce
`round` -> `myround`
Remove superflous `floor` calls

MeshUpdateQueue: Add a MapBlock cache that minimizes the amount of MapBlock copying done in the main thread

Cache size is configurable by the meshgen_block_cache_size (default 20 MB).

New profiler stats:
- MeshUpdateQueue MapBlock cache hit %
- MeshUpdateQueue MapBlock cache size kB

Removes one type of stutter that was seen on the client when received MapBlocks
were being handled. (the "MeshMakeData::fill" stutter)

Kind of related to at least #5239

Originally preceded by these commits, now includes them:
- Move the mesh generator thread into src/mesh_generator_thread.{cpp,h}
- mesh_generator_thread.cpp: Update code style
- MeshUpdateThread: Modify interface to house a different implementation: Actual functionality will be changed by next commits.
- MeshMakeData: Add fillBlockData() interface (so that caller can fill in stuff from eg. a MapBlock cache)

Include container.h in util/thread.h. util/thread.h doesn't compile without it

Sneak: Add option for old move code

Temporary option for the old move code for specific old sneak behaviour.
Enabled by setting the added 'new move' physics override to false.
By default 'new move' is true.

Remove an unused variable in Android Build

Plug two minor Leaks (#5603)

* Resource leak: CHECK_FILE_ERR returns, without freeing chunk_name.

Found with static analysis.

* Resource leak: leaks `page` on error path.

Found with static analysis.

Android progressbar fix (#5601)

* Fix progressbar for Android

Fixes #5599
Fixed #5403

* draw_load_screen: use texturesource
  this permits to unify texture loading code
* scale progress bar

* Add gl version check for GL_OES_texture_npot. This fixed the texture on loading screen

* Remove two sanity checks pointed by @celeron55

* sfan5 comments + android ratio fixes

Disable android leveldb by default (#5596)

1) Now leveldb is brick (thanks google)
2) By default, use SQLite3, this work perfectly and NOBODY not use LevelDB on Android :)

Implement delayed server shutdown with cancelation (#4664)

Minimap: Do a double-typecast to fix compiling with MSVC

Partial damage cheat fix: node damages server side (#4981)

* Damage cheat fix: server side

* Lava/Node damages overtime server side
* lava hurt interval is only for old protocol

NetworkPacket: don't copy push std::string and std::wstring

Fix wrong channel type in Client/Server CommandFactories
This is a u8 not a u16

ClientIface::sendToAll: honor packet configuration (#5590)

Fix android build

[CSM] Add function to set minimap shape (#5569)

* [CSM] Add function to set minimap shape

Also deprecates `toggle_shape`.

* Oh fish, I messed that one up!

* Fix Style

* Sorry, I missed something

I still had the `luamethod` call in there!

* Add getters

* Remove extra line

* Remove useless variable

Please review again @nerzhul . Thanks!

* Satisfy nerzhul

Only use palette if param_type2 is correct

Add documentation for map block format 27 (#5576)

[CSM] Fix localplayer documentation (#5557)

Fix inability to change metadata using stack:to_table() (#5547)

Fix fips_cipher_abort warning and other duplicate macros

[CSM] Allow escaping chatcommands and add missing calls to gettext. (#5565)

Replace occurrence of luaL_reg in l_localplayer (#5566)

Related to commit 41c5483. Replace a final occurrence of luaL_reg in
src/script/lua_api/l_localplayer.cpp

Sneak glitch: Detect ledge for 2-node climb-up

Re-creates the old sneak-jump behaviour in new code.
Enabled by the 'sneak glitch' physics override.
When a ledge is detected the jump speed modifier is set to the larger
of 'physics override jump' and 1.3 to allow a 2-node climb-up.

An unexpected side-effect is the simple sneak ladder working smoothly.

[CSM] Use more gettext (#5553)

[CSM] Move `.list_players` and `.disconnect` to builtin. (#5550)

Clang-format fix for Android

Clang-format styles fixes since previous commit

Clang format: fix LINT on old PR which doesn't have LINT enabled

Hard-coded undersampling.

Adds uniform undersampling for the 3D rendered scene. GUI elements
are not undersampled, resulting in better playability for users
with low-performance platforms with readable fonts and formspecs.

The undersampling setting can be set to 0 (disabled), 2, 3, 4 pixels
which translates into a resolution reduction of x4, x9 or x16, and
is significant.

LocalPlayer api lua

Hardware coloring for itemstacks

Adds the possibility to colorize item stacks based on their metadata.

In the item/node definition you can specify palette (an image file)
and color (fallback color if the item has no palette or metadata).
Then you can add palette_index to the metadata.

Dropped itemstacks with different colors do not merge.

Move chat command handling code from C++ to Lua (#5528)

Document that write_json will error on unserializable types. (#5539)

Previously it was erroneously documented that it would save them as null.

Prepare for Linux daily build packages (using Gitlab.com CI) (#5542)

* Prepare for Linux daily build packages (using Gitlab.com CI)

Replace luaL_reg with luaL_Reg as recent LuaJIT dropped the Lua 5.0 compat (#5541)

We are bundling Lua5.1 which has same macro

code style fix on src/script/cpp_api/s_client.h

[CSM] Add event on_connect player API lua (#5540)

* Add event on_connect player API lua

Fix cmake library default build problem since moving to lib/

Also make Lua library check a cmake module

Pass clang-format on 14 trivial header files fixes

Also remove them from whitelist

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