Use the NetworkPacket api to read full version string instead of u8 pointers. Also store the reserved byte to use API properly

Fixes #2509

Fix a little regression on SendActiveObjectMessages which send all TOCLIENT_ACTIVE_OBJECT_MESSAGES as reliable

Minor bug fix (lag between damage flash and hearts updating)

Fix various damage related bugs (client-side)

E.g. spawning or teleporting into lava to become invinsible

Minor cleanup: game.cpp

* Sort includes
* Remove unnecessary includes
* Ensure the parameter name for GameRunData is consistent for class member functions

Revert "Server::step throw is never catched. Replace it with an errorstream + assert"

This reverts commit 5f8e48c63baf12930430d5d8fc07495021599e61.

Bump android version code

Server::step throw is never catched. Replace it with an errorstream + assert
This throw can be trigger by LuaError exception or ConnectionBindFailed exception in the following functions:
* EmergeThread::Thread()
* ScriptApiEnv::environment_Step()
* ScriptApiEnv::player_event()
* ServerThread::Thread()

Android: Fix auto-entry of server address and port in mainmenu

Fixes #2497.

Move client packet generators to dedicated functions for:
* TOSERVER_INIT_LEGACY
* TOSERVER_DELETEDBLOCKS
* TOSERVER_GOTBLOCKS
* TOSERVER_REMOVED_SOUNDS
Also use a std::vector instead of std::set for TOSERVER_REMOVED_SOUNDS

Move TOCLIENT_ACTIVE_OBJECT_REMOVE_ADD and TOCLIENT_ACTIVE_OBJECT_MESSAGES to private functions

Rename packethandler/{client,server}.cpp to {client,server}packethandler.cpp
* Requested by @Zeno-
* Approved by @nerzhul

If player is dead, permit it to respawn, even if damages are not enabled
* Mods can do setHP function to do damages and override the enable_damage value sometimes

SendPlayerHPOrDie everytime at client connection, damage enabled or not, this will fix the player which stay dead at connection

Finer progress bar updates when initializing nodes

The bar is only drawn when the user will notice a change, which prevents time overheads that this commit would cause, resulting from useless draws.

Android Makefile update backported from stable-0.4 with minor changes

mg_decoration: Raise highest allowed deco top to max edge of voxelmanip

Remove max_y from functions, simplify code

Android: Backport changes from stable-0.4

Android: fix package name in JAVA sources

Fix package name in AndroidManifest.xml

AndroidManifest.xml version bump

Disable double-click -> ESC translation for main menu

client.cpp: typo fix and break; fix to make some compilers happy

Replace NetworkPacket pointers to references

Prepare Protocol v25 init & authentication.
* TOSERVER_INIT and TOCLIENT_INIT renamed to _LEGACY
* TOSERVER_PASSWORD merged from dev-0.5, can use protocol v24 and v25
* TOCLIENT_ACCESS_DENIED merged from dev-0.5, can use protocol v24 and v25, with normalized strings an a custom id for custom errors
* new TOSERVER_INIT packet only send MT version, supported compressions, protocols and serialization, this permit to rework everything later without break the _INIT packet
* new TOSERVER_AUTH packet which auth the client
* new TOCLIENT_HELLO packet which send server serialization version atm
* new TOCLIENT_AUTH_ACCEPTED which is send when TOCLIENT_AUTH was okay. After this packet, the client load datas from servers, like after TOCLIENT_INIT_LEGACY packet

Handle the newly added TOCLIENT_ACCESS_DENIED and TOCLIENT_DELETE_PARTICLESPAWNER
* Rename the handlers from _Legacy to regular, because here we can use same handlers
* Fix some packet names and pseudo handlers

Rename some packet and handlers to <packet>_Legacy name for compat layer between new network changes and old network clients

Use the new Player::isDead function when it's the case

Update Indonesian Language

handleCommand_Breath mustn't update breath is player is dead. A dead player doesn't breath.
Add Player::isDead function to check player is dead

Don't handle PlayerPos packet coming from client if player is dead

Mgv5/mgv7: Add desert temples if desert stone detected in mapchunk

Unescape tooltip texts

Previously, tooltips didn't unescape escaped formspec texts.
Item descriptions that trigger tooltips like inside inventories or for item_image_button
don't need to get unescaped, as they don't get set using formspec, but from lua.

Mgv6: Fix uninitialised heightmap used by cavegen

lua_api/l_mapgen: Fix overlapping areas of minetest.generate_ores/decorations

Silence Clang warning

Ensure that heightmap is initialized before use

Without this, cavegen will use values in the heightmap before they are initialized.

Stop formspecs closing with double-click in empty area

Behaviour on Android remains the same until other alternatives can be devised (if possible)

See #2425 (and the many other -- now closed -- issues reporting this feature as a bug)

Fix memleak pointed by issue #2439.
Also change bzero to memset. bzero doesn't work on windows

Fix unitialized data when creating TOSERVER_INIT packet

Make the dummy backend only look up blocks once

Update Android Makefile

ARMv7 default build
Update curl
Update OpenSSL

Heightmaps: Fix uninitialised values in mgv5/mgv6. findGroundLevel: Return -MAP_GENERATION_LIMIT if surface not found

Optimize Profiler::avg()

Don't use luaL_checkstring to read node names, it's only for arguments

This caused confusing error messages like "argument #4 to set_node is not a string."

Add Turkish comment translation to minetest.desktop

Add more Android build files to .gitignore

Respect game mapgen flags and save world noise params

Fix RUN_IN_PLACE broken due to invalid usage of assert

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

Improve Clouds::render mathematics

Fix Android text bug (no text displaying)

Dungeongen: Optionally set ignore to be untouchable to disable floating dungeons

Don't start a server for map migration

Clean up database API and save the local map on an interval

Biome API: Enable decorations placed on water

Add schematic decoration force placement flag

Improve group-based connection between raillike nodes

l_get_modnames: Compare using std::sort instead of a custom function which does same work

collisionMoveSimple: use std::vector instead of std::list, this improve the performances

ProfilerGraph: Use std::list instead of std::vector

Fix mapgen using unitialised height map values

Use std::queue for HTTPFetchRequest and std::vector for log_output instead of std::list

Fix issue #2441: crash on respawn, since a conversion std::list to std::vector on Environment.cpp
* Also change some std::list to std::vector for ClientMap::renderMap
* Remove disabled code in ClientMap::renderMap, disabled since a long time

Replace std::list to std::vector into tile.cpp (m_texture_trash) and move tile.hpp to src/client/

Server.cpp Use std::list instead of std::vector for playSound, fillMediaCache, sendRequestedMedia, sendMediaAnnouncement and related functions

Server::AsyncRunStep: buffered_messages now uses std::vector instead of std::list. * sendRemoveNode and sendAddNodes are also converted as a side effect

Fix some rendering glitches

- Fixes particle draworder
- Fixes nodehighlighting

Performance improvement: Use std::list instead of std::vector for request_media, Server::getModNames, Environment::m_simple_objects.
* Also remove unused Server::m_modspaths

Remove Queue class which uses std::list and use native std::queue

Fix memory leak in MapgenV6

Cavegen, mgv5: Cleanup code

Conf.example: Update mgv5 mapgen params
Mgv7: Lava caves only below -256

Use std::vector instead of std::list in StaticObjectList and MutexedMap::getValues()

ABMHandler and player_collisions use sequential read/write. Switch from std::list to std::vector
* Also remove dead code Map::unloadUnusedData which is dead since a long time

ClientInterface::getClientIDs doesn't need a std::list. Use a std::vector for better perfs

We always know playerSAO when calling SendInventory. Using it instead of searching it via peer_id

Radius parameter for /deleteblocks here

Send Position packet on event, don't check it at each AsyncRunStep.
* This permit to cleanup the player checking loop

ASyncRunStep doesn't need to lock when do setTimeOfDaySpeed.
* setTimeOfDaySpeed already lock a mutex when modify the value, we don't need to lock all environment.
* add a fine grain lock for getTimeOfDay and setTimeOfDay to solve environment multithread modifications on this value

Send Inventory packet on event, don't check it at each AsyncRunStep.
* Call UpdateCrafting into SendInventory because this functions is only called before SendInventory
* Use Player* instead of peer_id for UpdateCrafting because SendInventory already has the Player* pointer, then don't loop for searching Player* per peer_id
* m_env_mutex don't need to be used with this modification because it's already locked before the calls

Remove unused m_wielded_item_not_sent

Send Breath packet on event, don't check it at each AsyncRunStep

Send Player HP when setHP (or a setHP caller) is called instead of looping and testing the state change.

Fix occlusion

Fix narrow_to_wide_c (ANDROID)

* Ensure converted string is NUL terminated
* Restore logic to that used prior to 9e2a9b5

Mgv6: Add heightmap. Do not make large caves that are entirely above ground

Add /setpassword and /clearpassword logging

Add Turkish language

Update minetest.po

Revert "Fix particles draworder" (issue #2388)

This reverts commit c09d026f0561ee3c6db821a7e193716f724a0e4a.

Change find_surface_nodes_in_area to find_nodes_in_area_under_air

Add find_surface_nodes_in_area LUA call which permit to only get the nodes which touch air. This permit to massively improve performance for mods like plantlife

Wrap lines longer than 80 characters in lua_api.txt ...and minor formating changes

Fix minor memory leak (Android)

Biome API: Re-calculate biome at every surface in a mapchunk column

Add camera smoothing and cinematic mode (F8)

Rename --do-unittests to --run-unittests as @Zeno- and @sfan5 requested

Optimise MapBlockMesh related functions

Directely or indirectly optimises the following functions:

* MapBlockMesh::MapBlockMesh
* MapBlockMesh::getTileInfo
* MapBlockMesh::makeFastFace
* MapBlockMesh::getSmoothLightCombined

Fix wrapDegrees family of functions

wrapDegrees() (renamed to modulo360f)
wrapDegrees_0_360
wrapDegrees_180

Minor errors were present in previous versions; see issue #2328

Add src/network/CMakeFiles/ and src/client/CMakeFiles to gitignore

Disallow object:remove() if the object is a player

Rebased by Zeno- (conflict in lua_api.txt)