Tidy up dlg_config_world.lua (#5351)

Move code to pkgmgr

Fix GameUI flag value regression introduced by a78659ed0529226ad937f5e241ad72ba93702959

Added more strong unittests on that part to prevent future regression

Print error when HOME is not set (#7376)

In some configurations, such as when using the runit supervisor and its
tool chpst, the HOME variable might be unset. This resulted in an
unclear error message that was hard to pin down.

Fix more GCC 8.1 warnings                                                           1   master 

Fix 3 warnings reported by GCC 8.1 of the following type

```src/client/gameui.cpp:191:43: warning: « void* memset(void*, int, size_t) » effacement d'un objet du type non trivial « struct GameUI::Flags »; use assignment or value-initialization instead [-Wclass-memaccess]
  memset(&m_flags, 0, sizeof(GameUI::Flags));
```

Fix a -Wcatch-value warning reported by GCC 8.1

```
src/translation.cpp:43:16: warning: interception du type polymorphique « class std::out_of_range » par valeur [-Wcatch-value=]
  } catch (std::out_of_range) {
```·

Remove a useless struct keyword for ObjectProperties

Formspec verification: Fix show_formspec inside callbacks (#7374)

Vein ore: Fix bug caused by changing perlinmap Y size (#7371)

Because vein ore uses 3D noise (all the other ores use 2D noise) the
perlinmap Y size can be different in different mapchunks when close
to the ore Y limits.
Previously this caused bugs in the vein structure because changes in
perlinmap Y size did not recreate the noise objects.

Delete and recreate the noise objects with the new Y size if Y size
has changed.

Schematic decorations: Fix placement bug when centred and rotated (#7365)

Previously, the centering caused by the 'place center x/z' flags did
not take rotation into account. So schematics with unequal X and Z
dimensions were incorrectly placed. The bug was hidden for schematics
equal in X and Z dimensions.

Dungeons: Fix duplication of y limit parameters (#7359)

Fix missing ignore textures (#7326)

Small usage changes for air and ignore items (#7305)

* Remove “you hacker you!” from node description
* Prevent placement of ignore in builtin
* Prevent giving of "unknown" explicitly

Add screenshots to online content browser

Player marker on both minimaps (#7350)

Add updating to online content browser

Pointed_thing_to_face_pos: Avoid crash when player is inside a node (#7342)

Avoid crash in some situations when player is inside a node, causing
'above' to equal 'under'.
In this situation return 'under' which is the node position very close
to the face position that would normally be returned.

Say position of locale dir more clearly in docs (#7338)

Minimal: Fix HP change crash (#7344)

Use server's zoom fov for distant world loading.

Contributing.md: Various additions and edits (#7309)

Add a new point to encourage discussion before coding. Move the 'ask
before significant coding' suggestion into this.
Edit and extend commit message guidelines. Refer to core dev commit
messages as good examples.
Remove 'short' and 'briefly' from the description guideline to try to
improve description quality.
Larger text for '### A pull-request is considered ..'.
Add a second roadmap link to celeron55's blog roadmap which is the 1st
part of the forum roadmap.
Add full stops to lists.
Consistent text size for '## Donations'.

Don't show Android edit dialog when tapping read-only field (#7337)

* Don't show Android edit dialog when tapping read-only field

From Lua API, "If the name is empty the textarea is readonly."

Fix segfault in player migration and crash in log_deprecated

Makes log_deprecated work when triggered from no function

Vertical biome blend: Tune PRNG seed for finer detail (#7329)

Make sure color returns to normal after a damage flash (#7332)

Update contentdb_url to content.minetest.net

Camera: Improve subpixel movement (#7319)

Run detach callbacks on player leave

Correct docs regarding non-nil detaching children

Fix crash guiConfirmRegistration quit menu (#7313)

Fix luajit include not being found

Mgv7: Code cleanup (#7299)

Fix builtin inventory list crash when size = 0 (#7297)

Cavegen: Allow small RandomWalk caves to generate beyond mapchunk border

Fix invisible chat error messages (#7289)

nodedef: Delete two unused method definitions (#7288)

Global new() or grab() to be managed in constuctor only (#7235 partial) (#7236)

* g_extrusion_mesh_cache new() and grab() to be managed in constuctor only (#7235 partial)

This global was getting grab()-ed by each call to getItemMesh, incrementing its reference count. What was to be the final drop() in the destructor ended up with > 0 reference count, so memory not freed by Irrlicht.

Allow damage for attached objects, add attach/detach callbacks (#6786)

* Allow right-clicking on attached LuaEntities

MetaDataRef: Add contains() and get() (#7214)

Mapgen caves: Re-order generation to fix cavern bug

Previously, caverns confused tunnel generation causing biome top and filler
nodes to appear in caverns.
Split 'generateCaves()' into 2 functions to separate tunnel and large
randomwalk cave generation.
In each mapgen re-order cave generation to generate tunnels before caverns.

Biome-defined cave liquids: Use faster biome calculation

Biome-defined dungeon nodes: Use faster biome calculation

Fix description wrapping in browse online content

Dungeons: Mostly fix missing stair nodes

Mgvalleys: Update settingtypes.txt

Mgvalleys: Code cleanup

Split some long lines.
Edit comments.
Remove unnecessary comments and unnecessary commented-out code.
Use std::fmax/fmin instead of MYMAX/MYMIN.
Remove scope-limiting braces.
Consistently define literals as floats, except in noise parameters.
Cleanup literals in noise parameters.
Remove unnecessary 'near_cavern' line.
Reduce max spawn y to be consistent with other mapgens.

Move ASCII art to std::cerr, to remove it from logs

Formspecs: Allow setting alpha value for the box[] element

Mgvalleys: Use shared tunnel / cavern code instead of internal

Caverns first appeared in mgvalleys and were later added to other
mapgens as shared code. Now this shared code can replace mgvalley's
internal cavern code.
Also use shared tunnel code instead of internal code.

Changes to mapgen that will affect existing worlds (mgvalleys is not
stable):

Single lava and water sources not added in tunnels.
Previous caverns are unchanged but an equal number of new ones are
added at a distance from the previous.
From y = -256 to 192 nodes below, where the caverns taper at their
upper limit, they will have a slightly different shape as the taper
is now linear.

Cave liquids: Use a more precise point for calculating biome

Use the centre point of the route being carved for a more precise match
between cave liquids and biome.

Fix i386 bit build at OpenBSD (#7259)

Fix Android build, broken since 87ad4d8

Node resolver: Make error on fallback optional, disable for mapgen aliases

Add online content repository

Replaces mods and texture pack tabs with a single content tab

Builtin auth handler: Speed up file writing (#7252)

Fix wrong channel number representation in logs (#7205)

PlayerSettings struct for player movement code (#7243)

Instead of calling g_settings->getBool("flag") multiple times
during each movement step, the current settings are cached
in a new player object member. Updated via registered callbacks.

Android: Modify touch screen GUI's buttons (#7240)

* Android: Add zoom, minimap, and toggle chat button
Zoom button is put above jump button.
Minimap and toggle chat button are put in settings bar.
* Jump button is rotated down button
* Move three buttons on the right screen higher

Cavegen: Fix variable typo that broke mgvalleys large cave distribution (#7249)

Fix elusive 5 year old bug that caused mgvalleys large caves to be flat and
limited to mapchunk borders.
Error was fixed 2 years ago in 'CavesV6' but not in 'CavesRandomWalk'.

Add CMakeDoxy* to .gitignore

FOV: Raise lower limit to avoid zoom-loading of distant world (#7234)

In the client, raise lower limit from 30 to 45 degrees, to avoid server
seeing this as a zoom and loading world beyond the server-set limit.
Add minimum in settingtypes.txt and enforce lower limit when set using
minetest.conf.

In the server, distrust the client-sent FOV if below the heuristic zoom
threshold and use the player object property 'zoom_fov' to check it, to
protect against hacked clients.

Android: Replace movement buttons with joystick (#7126)

* Android: Replace movement buttons with joystick

Replace movement control buttons (arrows at bottom left screen) with virtual joystick.
Joystick has 8 directions (same as keyboard). Basically, just map it to keyboard input.
Joystick applies only on left 1/3 of screen.
Joystick's position can be fixed by enabling fixed_virtual_joystick setting.
Three new images:
(1) placeholder joystick,
(2) joystick container (background), and
(3) joystick cursor.
Remove unused images: movement control buttons (*_arrow.png).
New data type: touch_gui_joystick_move_id

Joystick's fixed position is spaced one button size from bottom and from left of screen.
Remove unused variable: m_joystick_downlocation

Fix segfault caused by wrong wgettext()

Fix for translating empty strings

Fix for incorrect translation of empty strings

In the key change menu, when a button key not have name an empty string is passed to gettext.
The empty string is reserved for gettext to return de header of the .po file an this is shoved in the button

upright_sprite: Fix texture position for players
Fixes #6471

Sun colour: Undo colour change. Remove double assignment (#7227)

Commit 5070ca2111dd5b3f333a942059ef2c592de8dc24 changed sun colour by
fixing what seemed to be a code mistake.
Return to the standard colour but comment-out the first assignment of the
double-assignment for performance.
Add a comment to explain.

Biome API / dungeons: Add biome-defined dungeon nodes

Add new biome fields 'node_dungeon', 'node_dungeon_alt', 'node_dungeon_stair'.
If 'node_dungeon' is not defined dungeons fall back to classic behaviour.

Remove messy and imprecise dungeon material code from 'generateBiomes()'.
Code deciding dungeon materials is now in 'generateDungeons()' and uses the
biome at mapchunk centre for more precision.

Remove hardcoded 'MG_STONE' types as long intended.

Fix SAOM (#7200)

* Bugfix

Add player:get_meta(), deprecate player attributes (#7202)

* Add player:get_meta(), deprecate player attributes

Mgcarpathian: Fix spawn level calculation (#7212)

Mgcarpathian: Remove insignificant 'base' noise variation (#7209)

Was only +-1 node over a scale of thousands of nodes.
Replace with 'base_level' parameter value.

Zoom adjustDist(): Improve variable name (#7208)

Biome API / cavegen: Add definable cave liquid for a biome (#7192)

Add 'node_cave_liquid' as a new field in biome registration.
If field is absent cave liquids fall back to classic behaviour.

[clang-tidy] Promote some performance-* as a coding error (#7194)

* Promote performance-type-promotion-in-math-fn as a coding error
* Promote performance-faster-string-find too (which is not problematic currently)
* Same for performance-implicit-cast-in-loop
* Fix remaining tidy points

Huge LBM lookup performance improvement on mapblock loading (#7195)

* Huge LBM lookup performance improvement on mapblock loading

Fix 5 issues reported by PVS studio

* src/sky.cpp  146     warn    V519 The 'suncolor_f.r' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 142, 146.
* src/sky.cpp  147     warn    V519 The 'suncolor_f.g' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 143, 147.
* src/sky.cpp  148     warn    V519 The 'suncolor_f.b' variable is assigned values twice successively. Perhaps this is a mistake. Check lines: 144, 148.
* src/threading/thread.cpp     63      err     V730 Not all members of a class are initialized inside the constructor. Consider inspecting: m_thread_obj.
* src/server.cpp       3243    err     V595 The 'log' pointer was utilized before it was verified against nullptr. Check lines: 3243, 3258.

Fix last performance-type-promotion-in-math-fn problems

Optimize a little bit isBlockInSight, adjustDist & collisions (#7193)

* Use constexpr + unroll some calculations to cache definitively some calculations
* Unroll some calls in collision code & use a constref instead of a copy in one occurence

Fix last clang-tidy reported problems for performance-type-promotion-in-math-fn

Based on https://travis-ci.org/minetest/minetest/jobs/361810382 output

Also fix 2 missing copyright notices

Fix more clang-tidy reported problems for performance-type-promotion-in-math-fn

Based on https://travis-ci.org/minetest/minetest/jobs/361714253 output

Fix various clang-tidy reported performance-type-promotion-in-math-fn

Selected ItemStack: Reduce black magic (#6520)

* Selected ItemStack: Reduce black magic
Better button variable value naming

Optimize entity-entity collision (#6587)

* Add IrrLicht type aliases
* Add hash for IrrLicht vector
* Add object map

Fix many issues reported by clang-tidy (#7189)

* Fix many issues reported by clang-tidy

We have many issues in code related to some performance to float <-> double.
Clang-tidy reported it in performance-type-promotion-in-math-fn

I fixed many of them. It's not ready for a promote to blocking

Also fix some value which should be const-ref

'fix' LINT, use InventoryLocation==

Run callback in IDropAction, refactor function arguments

Add player inventory callbacks

Formspecs: Use mouse wheel to pick up and deposit single items

LINT: add clang-tidy step (#6295)

* Implement new travis clang-tidy build step

* This step enable some rules and enforce one rule as error
* This permits to have some C++ quality rules based on clang & clang contributor guidelines

* Fix clang-tidy reported problems on push_back -> emplace_back

ObjectRef: Add add_velocity() (#3208)

Allow changing the velocity of objects relatively to their current velocity

event.h: Fix import GCC warning caused by ce87310

Client eventmanager refactor (#7179)

* Drop EventManager from GameDef & do some client cleanups

* EventManager is only used by Client. Don't expose it on Server & GameDef for nothing
* Drop Client::event() in favor of direct calls to getEventManager
* Cleanup some event put from new + put to put(new)
* MtEvent: add Type(u8) enum
* This will enhance event performance & ensure stricter type
* Drop MtEvent::checkIs (unused)

* clang-tidy reported fixes

* Code style

* Move event_manager.h to the client directory as it's only used by client

Add EventManager unittests + switch to unordered_map as order is not important here

Drop a unused function

Mgcarpathian: Mapgen loop optimisations. fabs() -> std::fabs()

* Mgcarpathian: ZYX -> ZXY mapgen loop optimisation.

* 'pow(n, 3)' to 'n * n * n' type optimisations.

* fabs() -> std::fabs().

core.rotate_node: Do not trigger after_place_node (#6900)

Sound: fix static initialization order dependency by not having one (#7168)

Ensure singleton is reset

Optional alpha channel support for entities

Fix a warning introduced by c5b4e541749c50805519ce040d98a0a8e5e0ec03

Fix a warning reported by clang

minetest/src/script/cpp_api/s_player.h:27:1: warning: struct 'PlayerHPChangeReason' was previously declared as a class [-Wmismatched-tags]
struct PlayerHPChangeReason;

Fix various Client class functions not marked as override (virtual)

Also switch some of them which are reported by clang-tidy

Load dependencies and description from mod.conf

Add reasons to on_dieplayer and on_hpchange

Add formspec theming using prepended strings