Inventory: Properly revert client predictions (#8945)

Caused by incremental inventory sending
Previously everything was overwritten by serializing the entire inventory

Improve undersampling settings

The setting now accepts values between 1 and 8 in the
Advanced Settings menu.
Values 0 and 1 now behave the same way (setting it to 1 won't
disable MSAA anymore), so there's no need to expose 0 as a value.

This closes #8939.

Inventory: Undo prediction on drop

Fix the bgcolor formspec element (#8716)

Fix formspec version backup in prepends losing data

Dungeons: Clean up parameters, improve structure variety (#8918)

While preserving the general character of dungeon structure.
Slightly increase the range of standard room horizontal size, while
preserving the average horizontal size.
Return to classic maximum large room size of 16x16x16.
Make 1 in 4 dungeons have a 1 in 8 chance for each room being 'large',
making multiple large rooms possible for the first time.
Make 1 in 8 dungeons allow diagonal corridors, to make these a little
more common.
Make corridor width vary from 1 to 2, but forced to 2 if diagonal
corridors are allowed, to make them passable.
Add some comments.

Built-in formspecs: Force version 1

client.h: Add missing const's

LocalPlayer: Fix code-style

client.cpp: Fix code-style in Client::sendPlayerPos

Fix LocalPlayer::isDead always returning false if player is immortal

Send ActiveObjects once right after Init2

Formspecs: Introduce formspec_version to mods

CSM: Fix itemstack:get_meta() 'metadata' indexing error

Load CSM environment after the restrictions are known

Safety-guards for CSM callbacks to abort on a bad implementation
Only run callbacks when the mods are loaded (and with it: builtin)

Duplication checks inside constructors

InventoryManager: Fix virtual function overload warning

Send cumulated inventory changes only each step (#8856)

Applies to player and detached inventories

Update minetest.conf.example

Update translation strings

Update from Weblate

Dungeons: Remove non-functional 'projecting dungeons' setting (#8897)

Lua API documentation: Various fixes (#8914)

Remove unusable 'minetest.setting_*' from client_lua_api.txt
lua_api.txt:
- Define the 'mod.conf' format
- More precise 'settingtypes.txt' format reference
- Document special sound files 'player_*damage'
- Group, sort and add 'not_in_creative_inventory' special group
- Define the 'Settings' format
- Warning about incorrect byte saving in 'StorageRef'
- Note about non-persistent player definitions fields
- Better 'leveldiff' and 'level' group description

lua_api.txt: Improve privilege definition (#8912)

Move debug.txt after it grows too big (#8904)

Before opening the file for writing, its file size is tested. If it exceeds 50 MB, it is moved to debut.txt.1, otherwise the log is appended to the old messages. An old debut.txt.1 is removed if it already exists.

label[]: Fix cut-off translated text

Fix Irrlicht 1.9 support

doc/client_lua_api: Clarify how client side api and mods start (#8822)

Force item entities out of solid nodes (simpler) (#8885)

Fix Inventory::moveItemSomewhere()

Inventory: Handle removeItem leftovers (#8884)

'All Settings': Don't use checkboxes for 'no...' mapgen flags (#7847)

Various fixes to lua_api.txt (#8878)

particles.cpp: Fix code-style

Mainly spacing and pointer placement

clientobject.h: Fix code-style

Fix rotation of attached particlespawner

Co-authored-by: ANAND <ClobberXD@gmail.com>

Document world format version 28 (private nodemeta) (#8857)

Document luaentity pitch and roll in world format (#8864)

Clarify docs regarding pointed_thing and get_pointed_thing_position (#8766)

Various settingtypes.txt improvements (#8867)

Improve 'readable' setting names for waving water settings.
Change 'near plane' to 'near clipping plane'.
Document physics settings.

mgv6_spflags:
Change 'new biome system' to 'snowbiomes flag' for clarity.
Add missing 'noflat' flag to default flags.

Inventory: Fix missing setModified calls

Previously inv:remove_item and inv:set_list did not trigger an inventory update

Mapgen v6: Various mudflow code improvements and bugfixes  (#8805)

Calculate 'i' inside 'y' loop to avoid 'i' and 'y' becoming
out of step due to the uses of 'continue'.
Simplify calculation of 'p2d' when coordinates are inverted.
Remove some unnecessary and unreliable 'index out of
voxelmanip' checks.
Move calculation of '&em' out of loops.
For 'Loop further down until not air' code, use y coordinate
to detect being out of voxelmanip, instead of checking index
which did not detect that.
Add and improve comments.
Indent the 'for (;; y--) {' loop.
Improve format and fix codestyle issues.

fontengine: Fix non-gettext builds

Inventory: Send dirty lists where appropriate (#8742)

This change reduces the amount of sent data towards clients. Inventory lists that are already known to the player are skipped, saving quite some data over time.

Raises protocol version to 38 to ensure correct backwards-compatible code.

Add ItemStack:get_description() to get tooltip (#8847)

Improve occlusion culling in corridors with additional check

Restore approximate occlusion check

While less precise, it worked better which is what matters in the end.

Make Mapgen::spreadLight use a queue (#8838)

Fix default hand definition not using wieldhand.png

Occlusion: Check for light_propagates and do mapblock bounds checks

Occlusion: Begin cleanup

Revert "Don't send position update packet if player is dead"

This reverts commit fb6f1fdcbe3ee2321cc33eb00f2c6da86744d00a.

Add player sprite textures back into the engine as a fallback (#8823)

'player.png' and 'player_back.png' are hardcoded into the engine as
the default appearence of the player object, so the textures should
be present in the engine.
This results in a functional fallback sprite player for when a game
has no player appearence related code.

lua_api.txt: Add link to Minetest Modding Book (#8826)

remove_detached_inventory: Fix segfault during mod load

Don't send position update packet if player is dead

Disable autoforward if player is dead

Restore intended functionality to minimap markers (#8819)

Find LuaJIT headers on vcpkg

luajit headers were moved into a subdirectory to avoid conflicts with plain lua

ClientInterface: Use recursive mutex to prevent freeze in on_newplayer() (#8808)

Remove unused function in ReliablePacketBuffer

Minor refactor of IncomingSplitBuffer

Drop m_list_size from ReliablePacketBuffer

It's not required and, worse, can lead to bugs.

Mapgen v6: Fix mudflow iteration and iterate twice (#8795)

In MapgenV6::flowMud(), the previous implementation of coordinate
inversion caused the 2 inverted mudflow iterations (out of the 3
iterations) to not loop over the area, so only 1 non-inverted
iteration occurred.

Fix this bug but only iterate mudflow twice, as mapgen v6 has only
had 1 iteration for many years. There is now a good balance of 1
non-inverted iteration and 1 inverted iteration.

network: Stricter handling of split packets

network: Fix crash in ReliablePacketBuffer on mismatching packets

In the error condition the exception would be thrown before m_list_size
is decremented, causing a nullptr dereference in e.g. popFirst().

Fix undeclared global warning because of _ (#8798)

Add function `minetest.read_schematic`

Fix unnecessary exception use in 3 more methods (#8791)

* Fix unnecessary exception use in Server::SendBlocks

The code in this method calls getBlockNoCreate and then
messes around with try...catch to skip blocks which are not
in the memory. Additionally, it repeatedly calls
m_env.getMap() inside this loop. Speed the code up by
extracting the m_env.getMap() out of the loop and getting
rid of the try...catch.

* Fix unnecessary exception use in Server::SendBlock

Another unnecessary try...catch is slowing down
Server::SendBlock. Remove that to speed it up and get a nice
side effect of simplifying the code in question.

* Fix unnecessary exception use in MMVManip::initialEmerge

Remove another unneeded exception usage from
MMVManip::initialEmerge to speed that code up and simplify
it but be careful to not remove the braces as there is a
TimeTaker in use there.

Fix unnecessary exception use in Map::getSectorXXX (#8792)

The Map::getSectorNoGenerate throws an exception but no other
code is really dependent on that. Fix the odd instance of
misuse in ClientMap::emergeSector and remove the exception
throwing version of the method along with the "NoEx" suffixes
in the names of these methods.

Better F6 profiler (#8750)

Update the profiler names to make more sense of what they actually represent
Move the profiler code from header to its source file
Use monospace font to align lines
Format the statistics line to align better with surrounding values
Refresh the profiler each 3 seconds (roughly)

Fix unnecessary exception use in Map::isNodeUnderground

The isNodeUnderground calls getBlockNoCreate which calls
getBlockNoCreateNoEx and throws InvalidPositionException
if the returned value is nullptr, which isNodeUnderground
then catches to return "false". Remove the try..catch in
isNodeUnderground by calling getBlockNoCreateNoEx instead
of getBlockNoCreate and checking the returned value for
nullptr.

Fix compare between pointer and 0 in unittests

Pointers shall be set to nullptr, not 0, according to the
coding standards. By implication they shall be compared with
nullptr, not 0, too. Fix this code to match that.

Clear old item groups when they are overridden. (#8753)

This fixes overridden items keeping their old groups in the group to
items mapping even after their groups have been changed in lua.
It also prevents a more widespread issue where overriding an item
will add its content ID *twice* to the mapping, resulting in odd
behaviour in features such as ABMs.

Add support for set_formspec_prepend in main menu (#8611)

Handle multiple deserialization of a block's entities

This fix consists of two parts:
- Clear the list of stored entities. This has no side-effects.
- Catch the case where active entities exist and print a message.
  Clearing the active entitiy list has side-effects that should
  be handled. (those entities are known to the environment and
  to clients).

  As avoiding those side-effects is more complex, and as this
  problem is not expected to occur (with PR #4847 merged), there
  is no real incentive to implement this ATM.

This issue was a contributing factor to bug #4217. With the other
contributing factor removed (PR #4847), this commit makes sure this
factor does not go unnoticed if it ever happens again.

Update Gradle to 5.5.1 (#8771)

Fix breakage of non-GLES2 setups (#8774)

The commit 526a9e4b66abaf83eb6b1aaa3e93375acd87b830 breaks
the non-GLES2 setups because the code that is intended to
handle that is behind "elseif()" which is interpreted as
"elseif(false)" and thus the code never gets executed. Fix
that by changing the offending line to else().

Additionally, to avoid breaking the server only build
(which shall not have a dependency on GL/GLU/GLES at all),
enclose the entire block code in if(BUILD_CLIENT).

Fix some issues with minetest.clear_craft (#8712)

* Fix some issues with minetest.clear_craft

- Fix memory leak
- Fix crafts with an output count not being cleared when clearing by
  input.
- Fix recipe list being reversed when clearing by input.

* Add CraftInput::empty()

Merge pull request #8776 from osjc/FixGetNode

Finish getNode cleanup

Add player knockback on punch to builtin

Implement adding velocity to player from Lua

The intended usecase is knockback, but there's potential for more.

Builtin UI: Move box element to ensure correct draw order

Allow customizing chat message format (#8529)

guiVolumeChange: Fix child not being removed

Avoid crash caused by, and improve, 'findSpawnPos()'  (#8728)

Avoid an unsuitable spawn position (which if outside mapgen limits can
cause a crash) if the main 0-3999 loop reaches its end. Fallback to a
spawn at 0,0,0.
Check the mapgen-returned 'spawn_level' value for being outside limits.
When 'air_count' reaches 2, move back down 1 to spawn in the lower
empty node.
If the spawn position is disallowed by 'objectpos_over_limit()', 'break'
from loop instead of 'continue' because positions above are probably
also over limit.
Reset 'air_count' to 0 if an obstruction is found, to make 'air_count'
consecutive empty nodes.
Allow spawn in 'airlike' drawtype nodes such as mod-added vacuum,
alien atmospheres, fog etc.
Add clarifying comments and improve codestyle.

Game: Fix double BS multiplication

Fix usage of wrong variable in builtin chat command handling (#8762)

This was introduced in commit 8e75785 and resulted in chat commands not
returning their output text.

Client::Interact: Use InteractAction enum instead of numeric constants

This replaces the magic numbers used as interaction modes both client-side and server-side, primarily for the sake of ease-of-readability.

Unify wield item handling (#8677)

This moves the wield item functions to Player and the tool utils for range calculation
Also 'local_inventory' was removed due to redundancy in Client

guiConfirmRegistration: Set focus to text field (#8761)

Hide chat when console is open (#8656)

Fix inventory_overlay for nodes without inventory_image (#8433)

Optimize usage of TOSERVER_GOTBLOCKS packet

Do not add group values of zero to group lists. (#8751)

This fixes an issue where when the engine looked up groups (for example,
in ABM node names), NodeDefManager's m_group_to_items would contain nodes
with a group value of zero, resulting in nodes with flammable = 0 being
burned by a fire mod with a group:flammable checking ABM.

It brings consistency to the behaviour described in the api
documentation, where zero and nil groups should be the same.

Clean up and fix freetype=false crashes (#8641)

A IGUIFont of type bitmap/vector cannot be converted to CGUITTFont
Fixes various segfaults in gameplay
Shorter font cache code, cleaned up (?)

Add luacheck to check builtin (#7895)

Mapgen Flat: Fix and improve getSpawnLevelAtPoint() (#8756)

Previously, this wrongly returned ground level (a position containing
a solid node) as spawn level.
Return ground level + 2 (+ 2 to spawn above biome 'dust' nodes).
Improve codestyle and make more consistent with generateTerrain().

Update curl dependency in buildbot

Fix binary-string confusion in client network code

Clean up FindOpenGLES2.cmake