Fix secure io.lines

It used to drop all of the return values from the
insecure version of the function.

Fix secure io.open without mode

Make supplying empty formspec strings close the formspec (#4737)

This will only happen if the formname matches or if formname is "".

No functional changes. Credit gcu

This merge doesn't make any functional changes. It's a trivial style fix so that @gregorycu can be dual credited along with shadowninja for PR #4800

Fix superflous shader setting updates (#4800)

This improves rendering performance by ~40%

Tool break: Reduce gain of break sounds

Particles: Make attached particle spawners respect the parent's yaw

Position, velocity and acceleration vectors of particles are rotated
by the yaw of the parent object so that they are truly relative to it.
Clarify new attached particle spawner behavior in lua_api.txt.

Builtin/../item: Enable tool breaking sounds

If a tool wears out and is destroyed, it's itemstack count
goes to 0, and we can optionally play a breaking sound.

This patch implements playing a breaking sound when this
occurs. Sounds need to be added to the tool itemdef
registration as the sound name string in the .sound.breaks
member.

Introduce builtin_shared and use it to fix #4778

Fixes #4778 which was about the error:

ServerError: Lua: Runtime error from mod '' in callback item_OnPlace():
/usr/local/share/minetest/builtin/game/item.lua:278: attempt to call global 'check_attached_node' (a nil value)

The issue was a regression of commit 649448a2a91fbf3e944b2f2e739f4e2292af1df0

"Rename nodeupdate and nodeupdate_single and make them part of the official API"

Fix unexplained shader issue (glsl compiler bug??) (#4757)

Lua_api.txt: Naming convention for custom item/entity fields

Custom (non-engine) field names of items and entities are allowed.
This is now documented in lua_api.txt. Field names beginning with
an underscore are now reserved for mod use, the engine must not
introduce any fields beginning with an underscore.

Mgv7: Add optional floatlands, disabled by default

Revert "Adding particle blend, glow and animation (#4705)"

This reverts commit 93e3555eae2deaeca69ee252cfa9cc9c3e0e49ef.

Adding particle blend, glow and animation (#4705)

Rename nodeupdate and nodeupdate_single and make them part of the official API

Now, the renamed forms of nodeupdate and nodeupdate_single are part of the official API.

As nodeupdate has been used by Minetest Game and in mods despite of not
being part of the official API, we ease the transition by still supporting
it for the 0.4.15 release. After the release, the two functions can be removed.

The removal will not violate the stability promise, as that promise only
includes the official and documented API.

Also, make some formerly global functions local. They most likely haven't
been used by mods, therefore they won't get stubs with deprecation warnings,
hard erroring directly.

Install texture_packs.txt to DOCDIR

Fix crash when attached object no longer exists

Active objects that are attached to other objects are not safe
from deletion. As a result, the parent object may have a reference
to an id of a child's that no longer exists.

If at some point an attempt is made to manipulate the child,
enviromment->getActiveObject(child-id) returns NULL. Using the
NULL pointer causes the crash...

Add control information to player interacts (#4685)

Nodeupdate: Remove documentation in lua_api.txt

In preparation for nodeupdate being renamed and made official API
in future.

Halo: Highlight selected face

This is a slightly modified and cleaned up version of #3774 by RealBadAngel.
By sofar: Remove color change (just make it lighter) and some minor cleanups.

Fixed tooltips not resizing with \n (#4766)

* Fixed tooltips not resizing with \n

* Fixed it for 1.8.4 too.

* Fixed not working with Freetype disabled.

* Modified it to use Zeno-'s solution.

Fix incorrect distance computation for visible blocks (#4765)

The client would not compute the distance from the camera to
to a mapblock correctly. The result was that blocks that were in
view (i.e. not beyond the fog limit) would not be rendered.

With the improved distance computation, a range adjustment that
existed in clientiface.cpp is no longer required.

Fix mob deserialization errors in the client (#4743)

The problem was seen while using the mobf mod package.

The problem happens when the server serializes entity attachments.
Sometimes, such attachments no longer exist. The serialization code
skips those. However, the total number of attachments was serialized
earlier.  Therefore the client expects more than it gets, and logs a
serialization error.

Occlusion culling: Add comments, minor code improvements

Remove unnecessary code.
Use '/ 2.0f' because endoff is a float.

Conf.example: Document block_send_optimize_distance

Add missing documentation.

Sky: Draw bottom of the sky box in 'cloudy fog color'

To be identical to the lower half of the skybox sides.
Now needed as the skybox base is often seen due to increasingly vertical
mapgens.

Retrieve mapblocks from the server in a sphere, not a cube

Use unused range argument in 'isBlockInSight()' to limit mapblock sends
to a sphere of radius 'max block send distance'.

Occlusion culling: Fix 'end offset' distance, half this for centre point

'endoff', the maximum diagonal of a mapblock, was incorrectly calculated.
Half this value for the centre point of the mapblock.

Don't use reduced vertical limits for mapblock send and generation

Builtin/../falling: Fix bugs caused by 'ignore' nodes

Original commit by t4im, rebased and developed by paramat.
Fix CONTENT_IGNORE being replaced by falling nodes or causing large
areas of sand to collapse into itself.
Format some conditional code for clarity.
Add and clarify some comments.

Privileges: Only give singleplayer 4 basic privs

- interact
- shout
- privs
- basic_privs

Windows: Add manifest file and set "High DPI Aware" true

This resolves washy font and incorrect mouse handling on Windows if
a screen with high DPI is used.

MSVC: Generate debug information for release builds (#4674)

Replace parameters with the equivalent /Ox

Separate optional from required mod dependencies in main menu (#4721)

* Separate optional from require dep's in main menu

* Simplify modmgr mod dependency listing code

PlayerSAO saving fix (#4734)

PlayerSAO::disconnected() function was historical and remove the link between SAO and RemotePlayer session. With previous attributes linked to RemotePlayer saving was working. But now attributes are read from SAO not RemotePlayer and the current serialize function verify SAO exists to save the player attributes.

Because PlayerSAO::disconnected marks playersao for removal, only mark playerSAO for removal and let PlayerSAO::removingFromEnvironment do the correct saving behaviour and all the disconnection process instead of doing a partial removal and let the server loop doing the RemotePlayer cleanup and remove some saved attributes...

Fix memory leak in ::safeLoadFile (#4730)

Remove unused shader matrices. (#4723)

Cycle directly to nothing shown instead of showing the profiler graph again

Fix for previous commit.

Add debug priv, and allow player to display the scene as wire-frame. (#4709)

Add minetest.get_server_uptime() function to Lua API (#4702)

Add minetest.get_server_uptime() function to Lua API

Add version API

Lua voxelmanip: Add optional buffer param for 'get param2 data'

Update lua_api.txt.

Core.rotate_and_place: Remove unused call to get_look_pitch()

Optionally disable optimization that causes underwater and cave rendering glitches. (#4686)

Fix overloading problems mentioned by clang

PlayerSAO/LocalPlayer refactor: (#4612)

* Create UnitSAO, a common part between PlayerSAO & LuaEntitySAO
* Move breath to PlayerSAO & LocalPlayer
* Migrate m_yaw from (Remote)Player & LuaEntitySAO to UnitSAO
* Migrate m_yaw from Player to LocalPlayer for client
* Move some functions outside of player class to PlayerSAO/RemotePlayer or LocalPlayer depending on which class needs it
* Move pitch to LocalPlayer & PlayerSAO
* Move m_position from Player to LocalPlayer
* Move camera_barely_in_ceiling to LocalPlayer as it's used only there
* use PlayerSAO::m_base_position for Server side positions
* remove a unused variable
* ServerActiveObject::setPos now uses const ref
* use ServerEnv::loadPlayer unconditionnaly as it creates RemotePlayer only if it's not already loaded
* Move hp from Player to LocalPlayer
* Move m_hp from LuaEntitySAO to UnitSAO
* Use m_hp from PlayerSAO/UnitSAO instead of RemotePlayer

Rename testsounds/ directory to sounds/ (#1984)

Add missing keyname_to_keycode function (needed on Android)

When rewriting keycode.cpp I overlooked this function, fixes #4683 completely

Add missing remoteplayer.cpp to Android build

this should fix #4683

Add unittests to test player saving/loading (#4679)

Cleanup

Use node lighting for liquid spreading

This commit modifies the liquid transforming procedure to light and
unlight nodes instead of whole map blocks.

Improved lighting

This commit rewrites the procedure that is responsible for light
updating.

this commit
-provides iterative solutions for unlighting and light spreading
-introduces a new priority queue-like container for the iteration
-creates per-node MapBlock caching to reduce retrieving MapBlocks from
the map
-calculates with map block positions and in-block relative node
coordinates
-skips light updating if it is not necessary since the node's new light
will be the same as its old light was

Document item use callbacks (#4668)

Shaders: Remove special handling for liquids. (#4670)

Adding LuaError on attempt to assign vectors with values out of range

ParticleSpawner: fix offset being added twice

Windows: dont link to libraries that are already linked by cmake

This solves the problem whith building where build fails if the libaries have different names.

Shaders: Apply tone mapping before fog calculation.

Shaders: Harmonize Irrlicht and shader fog calculations

Lua_api.txt: Clarify 'override_meta' bool in 'set mapgen setting'

Damage flash: Reduce maximum alpha. Avoid fade overload

Flash alpha maximum is reduced from 180 to 127 to avoid player blindness
in combat. Flash alpha minimum is unchanged.
The 'damage_flash' value is now limited to max alpha, to avoid multiple
hits creating a huge value that causes flash to stay at maximum alpha
for a long period. Now alpha always starts to fade immediately after
taking damage.
Both problems can be seen in Minetest let's play videos.
Simplify and optimise some code.

Changes to static object storage limit and error message

Move static object storage force-delete message from errorstream to
warningstream.
Increase 'max objects per block' setting to 64.
Add missing spaces in warning code.

Chat commands: Trim whitespaces from input of `/privs` command

Fix crash on attaching player to entity

Rename "refresh" to "processInitData"

Builtin: Add vector.floor helper function

Revert changes to toggleNoClip and toggleFreeMove

Don't use day light sky unless noclip and free_move are enabled

Enable mod security by default

Builtin/../chatcommands: Add /grantme command

Emergeblocks: Fix occasional crash

Modification of the emergeblocks internal state was not protected
by a lock, causing a race condition.
This can be reproduced by repeatedly running emergeblocks for an
already-generated section of the map (with multiple emerge threads).

Travis: Only trigger a compile, when C/C++ related files were touched (#4284)

Make documentation about ItemStack:set_{name,count,wear} clearer

These methods do not actually fail but instead clear the item stack
and return false if a value like e.g. "" is passed.

Builtin/../falling.lua: Code optimisation

Use range-based fog instead of z-plane based.

Attached particle spawners

Don't use unordered maps for ProfilerGraph (fixes flickering)

Add a button for disabling all mods to world config. (#3900)

Add missing languages to the settings

Disable cmake message concerning WIN32 on non-windows systems

A little cleanup since previous commit

Environment cleanup

* Move client list to ServerEnvironment and use RemotePlayer members instead of Player
* ClientEnvironment only use setLocalPlayer to specify the current player
* Remove ClientEnvironment dead code on player list (in fact other players are CAO not Player objects)
* Drop LocalPlayer::getPlayer(xxx) functions which aren't used.
* Improve a little bit performance by using const ref list for ClientEnvironment::getPlayerNames() & Client::getConnectedPlayerNames()
* Drop isLocal() function from (Local)Player which is not needed anymore because of previous changes

This change permits to cleanup shared client list which is very old code.
ClientEnvironment doesn't use player list anymore, it only contains the local player, as addPlayer is only called from Client constructor client side.
Clients are only CAO on client side, this cleanup permit to remove confusion about player list.

Remove unused parameter of GUIVolumeChange

Move RemotePlayer code to its own cpp/header

RemotePlayer/LocalPlayer Player base class proper separation (code cleanup) (patch 3 of X)

* remove IGameDef from Player class, only LocalPlayer has it now
* move many attributes/functions only used by LocalPlayer from Player to LocalPlayer
* move many attributes/functions only used by RemotePlayer from Player to RemotePlayer
* make some functions const
* hudGetHotbarSelectedImage now returns const ref
* RemotePlayer getHotbarSelectedImage now returns const ref
* various code style fixes

Optimize ClientIface::getPlayerNames(): return const ref instead a copy of all names

More code cleanup (UNORDERED + RemotePlayer/LocalPlayer)

* ClientEnvironment now uses UNORDERED MAP for active objects
* Use RemotePlayer and LocalPlayer everywhere it's possible
* Minor code style fixes
* Drop Client::getBreath() unused function

Fix backwards compatibility issue introduced by close_on_enter

Ensure std::unordered_ be used on MSVC 2010 too (#4600)

Player/LocalPlayer/RemotePlayer inheritance cleanup (part 2 on X)

* Server/Client Environments now have an helper to cast Player object in the right type to use it
* Server: use RemotePlayer everywhere and remove previous added casts
* Client: use LocalPlayer where needed
* Environment: remove unused functions (getPlayers(), getRandomConnectedPlayer(), getNearestConnectedPlayer())

Speed up emerge thread by using unordered map in a few places. Looking at 25% speedup in Emerge thread on Just Test.

Ensure std::unordered_ will be used on supported MSVC compilers

Prevent attached models from disappearing during parent reload (#4128)

VoxelArea: faster iter function (#4490)

Remove some unused attributes/class functions in server.cpp/h

Player/LocalPlayer/RemotePlayer inheritance cleanup (part 1 on X)

* LocalPlayer take ownership of maxHudId as it's the only caller
* RemotePlayer take ownership of day night ratio as it's the only user
* Pass getPlayerControl as const reference to prevent object copy on each call (perf improvement in ObjectRef::l_get_player_control call)
* getPlayerSAO is now only RemotePlayer call
* get/setHotbarItemCount is now RemotePlayer owned
* Server: Use RemotePlayer instead of Player object on concerned call to properly fix the object type
* PlayerSAO now uses RemotePlayer instead of Player because it's only server side
* ObjectRef::getplayer also returns RemotePlayer as it's linked with PlayerSAO

Lua_api.txt: Add biome and nodeupdate documentation

Add missing documentation for 'register_biome' and
'clear_registered_biomes'.
Add documentation for 'nodeupdate_single'. 'nodeupdate'
is not yet documented due to a bug it causes.

Textures: Ignore unknown node in override.txt

Fix C++11 Windows build of threading code

The initial problem was that mutex_auto_lock.h tries to use std::unique_lock<std::mutex>
despite mutex.h not using C++11's std::mutex on Windows. The problem here is the mismatch
between C++11 usage conditions of the two headers. This commit moves the decision logic
to threads.h and makes sure mutex.h, mutex_auto_lock.h and event.h all use the same features.

use unordered containers where possible (patch 4 on X)

Also remove some unused parameters/functions