Misc. minor fixes

Little coding style fix on porting.h

Settings: pass name to callbacks by reference

Spare some copies.

Update clouds enable_3d_clouds when setting changed

Windows: Fix some warnings.

Biome API decorations: 'spawnby' searches a 3D neighbourhood

The neighbours checked are the 8 nodes horizontally surrounding the decoration base
and the 8 nodes horizontally surrounding the ground node below the decoration

Add new leaves style - simple (glasslike drawtype)

Use UTF-8 instead of narrow

Use wide_to_utf8 and utf8_to_wide instead of wide_to_narrow and narrow_to_wide at almost all places.
Only exceptions: test functions for narrow conversion, and chat, which is done in a separate commit.

Clean-up Minimap code

- Fixed race conditions
- Fixed null dereference
- Fixed out-of-bounds array access
- MinimapMapblock is now allocated and added to update queue only when enabled
- Removed dependency on LocalPlayer
- Fixed code style
- Simplified expressions and program logic
- Cleaned minimap object interfaces

Mgv7: Auto-set lowest mountain generation level

Lowest level of base terrain determines mountain generation in mapchunk
Change some positional function arguments from int to s16

Client: better m_proto_ver initialisation

Previously, m_proto_ver was set to the serialisation version
inside the legacy init packet.

Now, if the server doesn't send a protocol version (protocols < 25),
we set m_proto_ver to some value < 25 and > 0.

Use minetest logging facilities for irrlicht log output

Mgv7: Lower base of mountain generation to -112 and define constant

Fix endless loop since grandparent commit

Fix attempt to start a world when no world is selected/created

Craftdef: Use numbers instead of iterators

Use numbers instead of iterators to traverse various vectors.

Fix missing check for 0 in craft replacements

Fix code style from recent commits and add misc. optimizations

Set server_announce to world.mt and respect modes when changing subgame

Shaders fixes and cleanup relief mapping code.

Fix world.mt not written when selecting mode

Inventory manager style cleanup and further checks

Disable mesh cache by default

Cavegen: Cleanup code. Define constant for MGV7_LAVA_DEPTH

Add android tools environment to gitignore

Previously, if you did `make envpaths` (as described in README.android), the new file got listed by git as new file.

Make m_blocks_cache private

@RealBadAngel has removed all outside access of the cache.

Defer minimap update when shape changed

Close keybind settings menu with esc

Fix single click world select

Fix bugs in mainmenu

Biome API: Increase heat and humidity noise spreads to 1000

Add UpdateThread and use it for minimap and mesh threads

Fix *BSD build with GNU iconv

Add Lua errors to error dialog

Bugfix: variable type mismatch

Bugfix: minimap was updated only with loading new blocks, allow forced updates with changing player pos

Minimap update

Remove busy polling inside minimap thread

Mgv5/mgv7 caves: Remove sand found in underground tunnels

Add missing check for max_stone_y to mgv5 cavegen
Tunnels now carve through sand below water_level

Add minimap feature

Remove compile warning

Thanks @TeTpaAka for pointing this out.

Return to ignore

Use frame instead of words 'no screenshot available'

Typo fix on previous @est31 commit

Small TOCLIENT_HELLO logging fix

Fix grammar error and missing number conversion for version.
Also add deployed protocol to log line.

Add MoveSomewhere inventory action

Improve shift+click experience

Use actionstream instead of errorstream when failing to emerge player

Fix string conversion error message

Fix bug when craft input isn't replaced

Update Chinese translation

Fix some issues with animations, and allow non-looped animations to be defined

Mgv7: Edit noise parameters. Fewer octaves, larger spreads.

Remove textures vertical offset. Fix for area enabling parallax.

Android: use loop inside makefile to spare repetition

Small CAO improvements

-> remove the old parent as @kahrl suggested
-> use indices no iterator as @kwolekr suggested

Mapgen objects: Enable heatmap and humidmap for all biome api mapgens

More correct wrap_rows implementation

Generic CAO cleanups and renames for clarification

* Use enum for GENERIC_CMD_*
* Rename m_attachements to attachement_parent_ids (public member and clearer name)
* Rename GENERIC_CMD_SET_ATTACHMENT to GENERIC_CMD_ATTACH_TO
* USHRT_MAX + 1 buffer sizes to prevent overflows as @kahrl suggested
* Remove unneccessary m_id from GenericCAO (shadowing protected superclass member for no reason) as @kahrl suggested

Make attached objects visible in 3rd person view

Fix wrap_rows at inner byte of multibyte sequence

Also fix UTF-8 inner byte bounds and make unittest for case this fixes.

Biome API: Add noise defined biome blend

Mgv6: Don't create air gap in tundra at y = 48 in custom high terrain

Don't print an error for valid listring[] formspec

Make wrap_rows not wrap inside utf-8 multibyte sequences

Also count multibyte sequences as "one" character.
Adds unittest for the bug reporter's case.
Fixes #2796.

Fail iconv call gracefully

No freezing when inbuf_size doesn't decrease over time.

Simpler patch for saplings trying to grow on unknown nodes

GUIFormSpecMenu::OnEvent code style update and small refactor

* Make method (more) consistent with current code stlye
* Move index into loop constructor after @rubenwardy's suggestion
* Cache inv_s->getList(s.listname), which removes a possibly bad scenario
of inv_s being null.
* Properly check for validity

Update credits tab

Add list-rings

Adds list-rings, a method to implement item sending between inventories via shift-click.
Nice insider feature: a ring consisting of a single inventory list serves as nice clean-up method.
Also adds them to minimal game, and the standard inventory.
Craft output slots are not supported.

Remove unneccessary space for tab completion

Don't highlight entries in credits

Don't crash when saplings try to grow on unknown nodes

Biome API: Enable biome generation to lower world limit

Decorations: Remove lower limit of water level for placement

Ignore punches with no damage in the actionstream

Remove reference to deprecated privilege

Automated whitespace error fix for last commit

Improved parallax mapping. Generate heightmaps on the fly.

Add UTF and other utility unit tests

Android: bypass broken wide_to_utf8 with wide_to_narrow

While utf8_to_wide works well, wide_to_utf8 is quite broken
on android, for some reason.

Use utf-8 in formspecs

Add utf-8 conversion utilities and re-add intlGUIEditBox

Add minetest.register_on_player_hpchange

Add return list of individual counts to find_node_in_area

dofile error reporting for syntax errors

According to doc, dofile() raises an error when parsing failed due to syntax errors.
Fixes #2775

Minimal: Add snow biome and jungleleaves nodes. Add mapgen aliases

Treegen: Remove alias checks from jungletree generation for speed

Reposition irrlicht timestamp in makefile

More reliable serverlist behaviour

-> remember the previously loaded serverlist, and use if new one failed
-> show a "loading" serverlist

Minimal: Edit mapgen aliases. Use blob ore for clay, update other ores. Update simple biomes. Cleanup code

Decrease minetest.after globalstep lag

* abort if theres no active timer
* only reduce the timer.time of all timers when its necessary
* move updating timers_to_add into a seperate function

Mgv5/mgv7: Trigger biome recalculation at underwater surfaces

Fix uninitialized variable error

If you run minetest with valgrind, you'll quickly notice uninitialized jump
depend error messages that point to s_base.cpp:131. This commit fixes those.

Nicer time setting logging

Now logs

ACTION[ServerThread]: player sets time to 6:03

instead of

ACTION[ServerThread]: player sets time to 6:3

Fix wrong replace from previous commit

Localize inside whole misc_helpers.lua

Make split method static

Mapblock: nodecount refactor

Spare direct multoplication, use constant MapBlock::nodecount instead of
local nodecount variables.

Also use strides at one place instead of multiplications.

Only retrieve always_fly_fast setting once

(Android) Only simulate holding down fast key if fast_move is toggled to true

Make get_biome_list() error message more helpful

Fix typo in variable name