Android: statically load iconv library

Fixes #3291

Thanks to @arpruss for reporting the bug, and suggesting the fix.

Also, remove trailing whitespaces.

Run updatepo.sh

Better gettext support for protocol version mismatch messages

Previously, xgettext failed to resolve the dynamic call.
Thanks to @JakubVanek for pointing this out.

Small logging refactor and additional options

-> Get rid of Logger::logToSystem and use normal downstream output system for android instead

-> Give the downstream output system more information: enrich the log function of ILogOutput
with information and add ICombinedLogOutput for easier use.

-> Make Logger::getLevelLabel() static and public so that it can be used by downstream log output.

-> Add g_ and m_ prefixes where required

Improve Lua settings menu

* Add key settings to setting table and ignore them later
  This way they are added to the auto-generated minetest.conf.example
* Add flags type
* Add input validation for int, float and flags
* Break in-game graphic settings into multiple sections
* Parse settingtpes.txt in mods and games
* Improve description for a lot of settings
* Fix typos and wording in settingtypes.txt
* Convert language setting to an enum

Fix compilation under MSVC and remove unnecessary conditional function prototype

Thanks to SmallJoker for pointing this out.

Fix on_rightclick() being called directly after placing node

fixes https://github.com/minetest/minetest_game/issues/537

Improve rollback database indexing

Index more columns in the action table of the rollback DB to improve the performance of /rollback_check

Flush rollback log more often

Flushes the buffer of rollback actions that wait to get saved in two more situations:

1. Flushes in the destructor of the rollback. This makes the server not
forget the last < 500 rollback entries when it shuts down.

2. Flushes the rollback when /rollback_check is invoked. This is neccessary
as otherwise it leads to confusion if users want to test the rollback functionality
by placing a node and then executing the check on it, or if the actions were
very recent out of other reasons.

Fix some threading things and add additional thread unittests

- Fix thread name reset on start()
- Fully reset thread state on kill()
- Add unittests to check for correct object states under various circumstances

Decoration API: Add flag for placement on liquid surface

Add findLiquidSurface() function to mapgen.cpp
Update lua_api.txt

Add more ways to pass data to check_player_privs

The callback can now be invoked with either the player object or name as
the first parameter, and with either a table or a list of strings, like
this:

    minetest.check_player_privs(player_name, { shout = true, fly = true })
    minetest.check_player_privs(player_name, "shout", "fly")
    minetest.check_player_privs(player, { shout = true, fly = true })
    minetest.check_player_privs(player, "shout", "fly")

Translated using Weblate (Portuguese (Brazil))

Currently translated at 72.6% (202 of 278 strings)

Translated using Weblate (Japanese)

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (German)

Currently translated at 100.0% (278 of 278 strings)

init_log_streams: check if log_filename is empty.

Fixes #3262.

ABMs: Make catch-up behaviour optional

Default is true for backwards compatibility
Update lua_api.txt

Settings tab: double click opens/closes trees

Settings tab: don't autoscroll when toggling "Show technical names"

Remove wstrgettext

Everywhere where wstrgettext was used, its output was converted back
to utf8. As wstrgettext internally converts the return value
from utf8 to wstring, it has been a waste. Remove the function, and
use strgettext instead.

Display sane output for empty descriptions

According to its man page, the function gettext(3)
"may return a nonempty string" when msgid is "".

This commit fixes a bug where the comment ""
for some settings caused gettext to return a
"nonempty string", in this case header info of the
po file.

Fix GUITable selection issues with trees

- setOpenedTrees(): this internal function was calling setSelected()
  to update m_selected. Since setSelected() calls autoScroll(),
  this caused the scrollbar to scroll back to the selected row
  in some cases when that shouldn't be done.

  For example, clicking the "+" to open a tree caused autoscroll.

  Fix this by making setOpenedTrees() modify m_selected directly.

- setDynamicData(): set scrollbar position after calling
  setSelected(), not before. This avoids setSelected()'s autoscroll
  messing up the scrollbar position again.

- setSelected(): If an invisible row is selected, open all parents
  of the selected row in order to make the selected row visible.

  This fixes the issue where all the trees are closed again whenever
  you return from the setting edit dialog to the settings tab.

Re-add "file" type for --add-location for xgettext call

Passing line numbers in the comments is bad.

References:

* Commit 94961b3364f76d5861913af321e9be6200d080b3
* Previous commit

New settings tab contain all possible settings

Settings are automatically parsed from builtin/settingtypes.txt
The edit dialog automatically adjust based on the type of setting

Fix == to =

Fix missing include on AIX

Refactor Thread class to improve readability and portability

- Fix some incompatibilities with obscure platforms (AIX and WinCE)
- Clean up Thread class interface
- Add m_ prefix to private member variables
- Simplify platform-dependent logic, reducing preprocessor
  conditional clauses and improving readibility
- Add Thread class documentation

Refactor thread utility interface

- Add "thr_" prefix to thread utility functions
- Compare threadid_ts in a portable manner, where possible

Fix crash regression when invsize formspec gets used

The invsize formspec element is outdated. Even though,
it is still supported, only a deprecation warning is shown,
introduced by commit [1]. The lua context passed to the
log_deprecated method added by commit [1] is NULL for the
invsize deprecation warning, as its run on the client and not
the server.

Commit [1] has removed checks for NULL inside the log_deprecated
method, resulting in a crash when a formspec with an invsize
element is parsed. This commit puts the check back.

Fixes #3260.

Referenced commits:

[1]: b5acec0a3c5701c53854ff7afdf4008863e6e8df "Add proper lua api deprecated handling"

[2]: 7b8d372947aae232ddf598155e972bb4dda157a "Use warningstream for deprecated field messages and refactor log_deprecated"

Fix enforcing of nametag hiding

Commit

d2ca662569427d36642660314668e416bf68f3c8 "Enforce hiding nametag"

didn't fix the issue for "client" instances, where the nametag update
was received before the object was added to the scene. This resulted
in the grey shadow on the nametag that commit tried to fix.

Thanks to @neoascetic for pointing out that there still is a shadow.

Compress textures and fonts

Used PNGOUT, OptiPNG and DeflOpt.
Removes ~350 KB without any loss in quality.

Add BufReader and vector-based serialization methods

Clean up gettext initialization

Use warningstream for deprecated field messages and refactor log_deprecated

Remove explicit syslog printing for uncaught exceptions on Android

All log operations are now added to the syslog implicitly.
Also, pass along mutable string to argument vector for main().

Rename macros with two leading underscores

These names are reserved for the compiler/library implementations.

Always use errorstream for DEBUG_EXCEPTION_HANDLER

Lower log level for benign socket errors

Use warningstream for log messages with WARNING

Remove DTIME macro and its uses, too

Refactor logging

- Add warning log level
- Change debug_log_level setting to enumeration string
- Map Irrlicht log events to MT log events
- Encapsulate log_* functions and global variables into a class, Logger
- Unify dstream with standard logging mechanism
- Unify core.debug() with standard core.log() script API

Fix how address is logged when a wrong password is supplied

- SRP: print the address only once, not twice
- Legacy: previously the address was not printed at all
- Make both messages structurally the same, to facilitate log analyzers

Localize digprop_err function

Mgfractal: Independant offset and slice params for mandelbrot and julia

Player now spawns on julia set due to julia offset
Add commented-out '#include profiler.h' for timetaker use
Use v3fs to reduce number of parameters
Tune tunnel width to match mgv7

Fractal mapgen: Fix mysterious bug

Clear list rings when loading a new formspec

Fixes a bug where the old list ring remained when a new formspec
was displayed over the old one. This created the list-ring of the new formspec
to be partly ignored.

Thanks to @VanessaE to report the bug, and @DonBatman to produce the code that
exposed it.

Mapgen: Use mapgen-specific names for constants in headers

Update copyright years in all mapgens
Add myself to copyright notices in mgv5 and mgv7

Fractal mapgen: Add seabed and large pseudorandom caves

Android/Mapgen: Add fractal mapgen file to android makefiles

Correct SRP documentation

Previous statements were wrong.

Add viewing range GUI setting

Mapgen: Add 4D fractal mapgen

Mgv5: getGroundLevelAtPoint searches a larger range

Translated using Weblate (Spanish)

Currently translated at 99.6% (277 of 278 strings)

Translated using Weblate (German)

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (German)

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (German)

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (German)

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (Spanish)

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (Ukrainian)

Currently translated at 81.6% (227 of 278 strings)

Translated using Weblate (Ukrainian)

Currently translated at 78.0% (217 of 278 strings)

Translated using Weblate (Spanish)

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (Norwegian Bokmål)

Currently translated at 3.5% (10 of 278 strings)

Translated using Weblate (Hungarian)

Currently translated at 92.8% (258 of 278 strings)

Translated using Weblate (Spanish)

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (Spanish)

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (Ukrainian)

Currently translated at 66.5% (185 of 278 strings)

Maybe it is better to describe allowed symbols instead of writing regex?

Translated using Weblate (Ukrainian)

Currently translated at 46.4% (129 of 278 strings)

Translated using Weblate (German)

Currently translated at 99.2% (276 of 278 strings)

Translated using Weblate (Chinese (Taiwan))

Currently translated at 100.0% (278 of 278 strings)

Translated using Weblate (German)

Currently translated at 98.9% (275 of 278 strings)

Translated using Weblate (German)

Currently translated at 98.9% (275 of 278 strings)

Translated using Weblate (German)

Currently translated at 98.9% (275 of 278 strings)

Translated using Weblate (German)

Currently translated at 98.9% (275 of 278 strings)

Translated using Weblate (Chinese (Taiwan))

Currently translated at 0.0% (0 of 278 strings)

Translated using Weblate (Chinese (Taiwan))

Currently translated at 100% (0 of 0 strings)

Created new translation.

Translated using Weblate (Esperanto)

Currently translated at 99.2% (276 of 278 strings)

Add new ContentParamType2 "CPT2_DEGROTATE"

This might break some mods, but it is important for all uses of the param2 to
be documented.

This doesn't need a serialisation version or network protocol version change,
as old clients will still work on new servers, and it is bearable to have
new clients getting non rotated plants on old servers.

Allow setting chunksize in core.set_mapgen_params

Hide mapgens from main menu not intended for end users

Add emerge completion callback mechanism

Major refactor of emerge.cpp and Map::init/finishBlockMake

doc: Update node callback documentation

Remove redundant code in player interact handler

Define and use limit constants for Irrlicht fixed-width types

Added minetest.wallmounted_to_dir

Fix MinGW 32-bit build

Add get_biome_id(biome_name) callback

It returns the index used in mg->biomemap for a given biome name.
The biomemap is useless without this unless you re-register all existing biomes,
which could cause problems for anyone else trying to use biomemap.
With this, you can quickly create a lookup table of ids and names.

Add environment variable MINETEST_WORLD_PATH

Also add PATH_DELIM for Windows compatibility.

Mapnode: Replace rotateAlongYAxis with improved version

Get facedir by using lowest 5 bits of param2 and limiting to 23
More robust, frees up higher param2 bits for other uses
Change lookup table and table index to u8

Fix some SRP issues

-> Remove memory allocation bugs
-> Merge changes from upstream, enabling customizeable memory allocation

Some map border related fixes

1. Check for entity addition success in spawn_item implementation
2. Check for success in item_drop callback, so that the player
doesn't lose the item if they are outside bounds and try to drop it.
3. When existing player joins game, check that their position is inside
map bounds. If not, set their position to the return value of findSpawnPos().
4. Make findSpawnPos() respect the border

2 fixes a lua crash if a player drops an item outside map bounds.
3 fixes an assertion crash if a player leaves when being outside map bounds,
and then rejoins.

Don't serialize StaticObjectList with > 65535 objects

Because the count is serialized as u16, this would cause overflow.

If minetest later deserialized a mapblock with an incorrect
static object count, it would be unable to find the NameIdMapping
(which comes after the StaticObjectList) and abort with an error
such as "Invalid block data in database: unsupported NameIdMapping
version" (issue #2610).

Decorations: Remove error message 'chunksize not divisable by sidelen'

Sidelen larger than 16 is essential for low density decorations
With sidelen > 16 chunksize may not be divisable by sidelen if
chunksize is changed, in this situation setting sidelen = chunksize
is desirable and should not create error messages.

Abort at uncatched exceptions

Change a remaining assert(0) call to FATAL_ERROR(msg).
There was a regression since commit

ced6d20295a8263757d57c02a07ffcb66688a163 "For usages of assert() that are meant to persist in Release builds (when NDEBUG is defined), replace those usages with persistent alternatives"

where when an "uncatched" exception is thrown inside a "side thread",
the program doesn't abort anymore.

This led to the problem @netinetwalker experienced where the emergethread
got an unhandled exception for loading a mapblock while redis was loading,
(see #3196) and then jmped outside its loop to work down its queue.
This resulted in the server not doing any emerges anymore.

Fix redis erroring on non found blocks

Thanks to @netinetwalker for spotting the error, proposing a fix, and testing it.

Error due to @est31's merging changes to PR #3202 to add more error reporting for invalid reply types, commit:

524a7656e3e5cd671b05c13e2ad69cb84bad0423 "redis: throw error if block request failed"

Now we branch out on the valid reply type "not found".

FindJson: use PATH_SUFFIXES jsoncpp to find incdir

For example, on Fedora systems jsoncpp headers is installed in `/usr/include/jsoncpp`.

Fix falling through nodes on world load (fixes #2784)

On world load the collision code can not see node boxes, since the
nodes have not been loaded. Thus it collided only at the next full
node. However, standing on a slab on world load leaded to sinking into
it until the world finished loading. Then one maybe fell further, if
the node below was not walkable.

Now, with this commit, when no node around the player has been loaded
it simply does not move the player.

Clarify radii and distance types in documentation

Because not all circles are round:
* circles using an euclidean metric are what we usually call "round"
* circles using a maximum metric look like euclidean rectangles with equal adjacent sides (squares)
* circles using a manhattan metric look like an euclidean right angled rhombus (squares, but 45° rotated to the former one)

[ci skip]

Update .gitignore to ignore symlinks to non-static Minetest directories and sort into an editor section

[ci skip]

redis: throw error if block request failed

Fixes #3196. Before, we didn't throw an error, and the engine thought the
block isn't occupied. But in fact it might be that redis is still loading,
and the block does exist in the database. The result was a cheesy map.

Add /emergeblocks command and core.emerge_area() Lua API