Save and remove player by pointer

Why doing things simple ? Use pointer instead of strings to save players and remove them.
Saving players by name does a lookup to find pointer we already have ! Idem with removePlayer
Also remove unused removePlayer(peer_id), it's never called

Mapnode: Add rotateAlongYAxisFull supporting 24 facedirs

Replace "self program" with "this program" in fstk boilerplates

... and remove trailing whitespace.

Fix another typo in doc/world_format.txt

lua_api.txt: fix typo

Thanks to @kaadmy (NeD) for pointing this out.

Various style cleanups + unused code removal

-> Don't pass pointer to whole IGameDef to NodeMetadata constructors
and deserializers, but only to IItemDefManager, which is needed
-> Remove the unused content_mapnode_get_new_name() method
-> Fix style for MapBlock::deSerialize and MapBlock::deSerialize_pre22,
improving accuracy of error messages a bit
-> Fix style at other serialisation methods too
-> Improve accuracy of some comments

Little optimization on getAdded/Removed activeobjects per player loop.

Use std::queue instead of std::set, we don't need such a heavy container.
Don't convert position to int to convert it back to float in the next function.

Mgv5/6/7: Re-add #include profiler.h as commented-out option

Blob ore: Fix partial blobs

Document current node metadata format

Document the node metadata changes of the commit (serialisation version >=23):

704782c95b8a4194a9383da55d93f37fd0f7278f "WIP node metadata, node timers"

Send proper block to old clients for swap_node calls

The legacy code added in commit

d879a539cd19ddd1ee34afec2512fb2238de2822 - "Add minetest.swap_node"

for sending the whole mapblock to older clients on the case of a node
modification with swap_node, had the problem that the block chosen to be
sent to the client was referenced with node coordinates and not with
block coordinates, resulting in getting the wrong block sent to the client.

Ore: Add puff ore type

Fix object position border checking

Borders have to be converted into BS format in order to be accurately comparable to
object positions.

Disallow placing entities outside safe boundaries

Entity positions are serialized as F1000. Disallow placing
entities outside safe borders with the minetest.add_entity
call.

Note that this patch only enforces those boundaries for
placing entities, moving entities that move outside boundaries
aren't affected.

Thanks to @nanepiwo for pointing this out.

Firelike drawtype: Improve code

Remove unusable fine rotation by param2
Remove unused and redundant code
Fix code style issues

Minor tweaks handle_settings_buttons

Fix "make install" and add underscore to doc file name

Fix regression since commit:

915807f8db1f3721ad9ffc00a4863ad940010c45 "Rename doc/mapformat.txt and update doc to match SRP changes"

And add an underscore to doc file name to make it more readable.

Serialisation: documentation fixes, clarifying renames and whitespace fixes

1. Do two renames:
* SER_FMT_CLIENT_VER_LOWEST -> SER_FMT_VER_LOWEST_WRITE
* SER_FMT_VER_LOWEST -> SER_FMT_VER_LOWEST_READ
Now the two define values are consistently named with the _WRITE defines
SER_FMT_VER_{HIGHEST,LOWEST}_WRITE, and to better point out what the two
serialisation versions actually are for.

2. wrap some lines in doc/worldformat.txt, and point out that the node
timers are serialized at a later point, as this can cause confusion about
what now happens (if one doesn't strictly read the if block's conditions).

3. some whitespace fixes in NodeTimerList::serialize, and one new comment.

Rename doc/mapformat.txt and update doc to match SRP changes

The documentation file contains not just information about the
map itself, but also about further files inside the world's
directory.

Documentation didn't reflect recent SRP addition, now it does.

Ore: Add ore sheet column height range selection

Modders are now able to select the range of ore column height,
and the midpoint at which they 'grow' starting from.
This commit adds three new parameters for the 'sheet' ore type:
column_height_min, column_height_max, and column_midpoint_factor.
clust_size is now deprecated for this ore type.

Run updatepo.sh

After this, it should hopefully not record line numbers anymore,
so the diffs of updatepo.sh runs are smaller. Well, this is theory,
lets see how it will turn out to be in practice.

Translated using Weblate (Esperanto)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Hungarian)

Currently translated at 91.1% (246 of 270 strings)

Translated using Weblate (Russian)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Russian)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Indonesian)

Currently translated at 98.1% (265 of 270 strings)

Penerjemahan

Translated using Weblate (Dutch)

Currently translated at 97.0% (262 of 270 strings)

Translated using Weblate (Indonesian)

Currently translated at 97.7% (264 of 270 strings)

This is a combination of 9 consecutive commits by the same author.
Their messages in chronological order are:

Translated using Weblate (Indonesian)

Currently translated at 79.2% (214 of 270 strings)

Translated using Weblate (Indonesian)

Currently translated at 80.0% (216 of 270 strings)

Ini merupakan istilah dan tidak perlu diterjemahkan

Translated using Weblate (Indonesian)

Currently translated at 89.6% (242 of 270 strings)

Penerjemahan

Translated using Weblate (Indonesian)

Currently translated at 94.8% (256 of 270 strings)

Penerjemahan

Translated using Weblate (Indonesian)

Currently translated at 95.9% (259 of 270 strings)

Mencari persamaan makna

Translated using Weblate (Indonesian)

Currently translated at 96.2% (260 of 270 strings)

Penerjemahan

Translated using Weblate (Indonesian)

Currently translated at 96.6% (261 of 270 strings)

Mencari persamaan makna

Translated using Weblate (Indonesian)

Currently translated at 97.0% (262 of 270 strings)

Penerjemahan

Translated using Weblate (Indonesian)

Currently translated at 97.7% (264 of 270 strings)

Istilah tidak diterjemahkan

Translated using Weblate (Esperanto)

Currently translated at 73.7% (199 of 270 strings)

Translated using Weblate (Dutch)

Currently translated at 90.7% (245 of 270 strings)

Translated using Weblate (German)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Esperanto)

Currently translated at 44.8% (121 of 270 strings)

Translated using Weblate (Esperanto)

Currently translated at 100% (0 of 0 strings)

Created new translation.

Translated using Weblate (Czech)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Spanish)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Spanish)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Czech)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Czech)

Currently translated at 95.5% (258 of 270 strings)

Translated using Weblate (Italian)

Currently translated at 82.9% (224 of 270 strings)

Translated using Weblate (Italian)

Currently translated at 80.7% (218 of 270 strings)

Translated using Weblate (Lojban)

Currently translated at 38.5% (104 of 270 strings)

Translated using Weblate (German)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Lojban)

Currently translated at 100% (0 of 0 strings)

Created new translation.

Translated using Weblate (Japanese)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Japanese)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (Japanese)

Currently translated at 100.0% (270 of 270 strings)

Translated using Weblate (German)

Currently translated at 100.0% (270 of 270 strings)

networkprotocol.h: remove "u16 command" from doc

Its obvious that "u16 command" is inside every packet, therefore this
commit removes all mentions of the command, if non-array like notation
is used. We already didn't add "u16 command" to new packets or removed
it at packet changes, so now we remove it from existing packets.

Improve locale directory detection

Use in-place locale directory if that exists, and
static one (RUN_IN_PLACE or CUSTOM_LOCALEDIR) doesn't exist.
Report to errorstream if neither static nor in-place locale
dirs exist, and report successfully found paths to infostreem.

Fixes two bugs:

-> Regression of commit [1] where if we use RUN_IN_PLACE=false,
but don't make install, locales aren't found. One might
think this is no regression, as its no bug, but all other
paths (mainmenu, etc.) are detected properly.
-> Regression of commit [1] where locales don't work on windows.

References:
[1]: Commit 645e2086734e3d2d1ec95f50faa39f0f24304761 "Use CUSTOM_LOCALEDIR if specified" by @ShadowNinja

Update leveldb git url

Google code was shut down, project moved to github.

Change m_client_event_queue's type to std::queue

As indicated in its name, m_client_event_queue should be a queue.
Change std::list to std::queue to improve the queue's performance.

Add more information about how get_node_light works.

Shaders: use triple-frequency waving for leaves and plants

Mgv5/mgv7 biomes: Reduce heat and humidity noise spreads to former value of 750

Use CUSTOM_LOCALEDIR if specified

Fix occasionally failing travis builds

We require sudo right now, tell this travis, so that they don't try to
run it on their container based infrastructure.

Fix building on OSX, broken since "Clean up threading"

Commit

e4bff8be94c0db4f94e63ad448d0eeb869ccdbbd - Clean up threading

by @ShadowNinja has broken the OSX build.

Including things inside a namespace isn't good.

Also fixes #3124.

Change my email

Rename minetest-icon.svg to minetest.svg - Fixes #1564

-Complies with the Linux standard freedesktop icon naming specs: http://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html#guidelines
-Ensures compatibility with icon themes
-Edit minetest.desktop, CMakeLists.txt and README.md to reflect this change

Areastore: fix "attempt to index a number value"

Before, calling get_areas_in_area for an areastore with both
include_borders and include_data would result in a lua error,
if there was at least one area as result:
attempt to index a number value in function 'get_areas_in_area'

Small fixes of minetest.has_feature

Add two missing directories to doxygen

Don't add line number to comment when running updatepo.sh

Otherwise, this updates all .po files for all line numbers that changed
since the last time updatepo.sh was run. We still add the filename, but
this information shouldn't change too often.

xgettext beginning with 0.18.4 supports this feature.

Warn when building without cURL

Enable server build when no builds are enabled

Make example config more readable

Minor tweaks __builtin:falling_node

Hide minimap if it has been disabled by server

gettext.cpp: Fix syntax error when using MSVC

Also remove trailing whitespaces from the file

Credits tab: fix accidental merger of two contributors

Fixes #3125

Make ClientInterface::statenames consistent with the state enum again

Fixes minetest.get_player_information segfault due
to out of bounds access problems, when compiled as debug build.

l_mainmenu.h: remove unused l_get_dirlist function

The commit
8f9af57314f71aae1cc77e13f9996e13015d776d "Add core.get_dir_list" by @ShadowNinja
has removed the implementation of the l_get_dirlist function and all its usages
from the l_mainmenu.cpp file, but hasn't removed it from the header file.

The reason why this hasn't been detected earlier is that C++ has this interesting
feature to still make it possible to create instances of classes whose never used
private methods are declared but not defined.

Ores: change ore chance in clusters to better respect clust_num_ores for dense clusters

Dungeongen: Remove floating frames

Preserves the rare unbroken protruding dungeons
Fix random range for first room roomplace
Fix checked volume for first room 'fits' bool
and check for 'untouchable' flag instead of 'inside'
Remove 'enable floating dungeons' setting

Push error handler afresh each time lua_pcall is used

Fixes "double fault" / "error in error handling" messages
(issue #1423) and instead shows a complete backtrace.

Use numeric indices and raw table access with LUA_REGISTRYINDEX

Change i++ to ++i

Add fgettext_ne as keyword to updatepo.sh

Use gettext to "None" of texture pack list

Add setting for mods to copy to Android app

Fix comment positioning in minetest.conf.example

Fix Android build of threading sources

Clean up threading

  * Rename everything.
    * Strip J prefix.
    * Change UpperCamelCase functions to lowerCamelCase.
  * Remove global (!) semaphore count mutex on OSX.
  * Remove semaphore count getter (unused, unsafe, depended on internal
    API functions on Windows, and used a hack on OSX).
  * Add `Atomic<type>`.
  * Make `Thread` handle thread names.
  * Add support for C++11 multi-threading.
  * Combine pthread and win32 sources.
  * Remove `ThreadStarted` (unused, unneeded).
  * Move some includes from the headers to the sources.
  * Move all of `Event` into its header (allows inlining with no new includes).
  * Make `Event` use `Semaphore` (except on Windows).
  * Move some porting functions into `Thread`.
  * Integrate logging with `Thread`.
  * Add threading test.

Fix wrong android versionName

Before, android versionName was set to "...13", which made the f-droid build fail.

Continue with 0.4.13-dev

Bump version to 0.4.13

tileable flags are needed also without shaders because of filters

Remove use of engine sent texture tiling flags - theyre no longer needed

Remove some emails from credits tab

Update credits tab

Fix indianred and indigo of color-string

Add Japanese translation of desktop file

Client: disable mmdb modstore

The service isn't online, and on the long term, mmdb will likekly be
replaced with other services from the modding community.

Android: bypass broken wide_to_utf8 with wide_to_narrow (again)

This bypass had to be re-enabled as some users reported issues,
even after the iconv build fix.

While utf8_to_wide works well, wide_to_utf8 is quite broken
on android, for some reason, and some devices (unrelated from build
configuration).

Fix inventory replace bug

SAPI: Disable unlockable time profiling

SEnv: Remove static_exists from ActiveObjects in deleted blocks

Remove unused functions.

Defaultsettings: Increase client_mapblock_limit to 5000

Rollback: Fail on bad precondition instead of causing assertion error