Codechange: Use GetScrolled(Row/Item)FromWidget in more places.

In many instances the clicked row position is 'manually' calculated
instead of using the GetScrolledRowFromWidget helper function, with
variations on checks. Replace with the two helpers where possible.

Codechange: Add and use GetScrolledItemFromWidget to get a list item.

This function returns an iterator, either to the selected item or the
container's end.

This makes handling the result more robust as indices are not used.

Revert 8fa61533: "Fix 30ae072: when a road type is hidden, towns may not build them even when that flag is set"

This reverts commit 8fa61533f041b16497282107afa600e919ba7154.

Update: Translations from eints
italian: 4 changes by Rivarossi
catalan: 4 changes by J0anJosep
dutch: 9 changes by Afoklala

Cleanup: widget pos_x/y are already int. (#10809)

Remove old casts left over from when pos_x/y were unsigned.

Update: Translations from eints
english (au): 4 changes by krysclarke
english (us): 4 changes by 2TallTyler
russian: 4 changes by Ln-Wolf
portuguese: 4 changes by azulcosta
polish: 8 changes by pAter-exe

Change: [Actions] Use -fuse-ld=lld and Ninja for MinGW (#10785)

And some miscellaneous things while touching the file

Update: Translations from eints
portuguese (brazilian): 30 changes by ericandradex

Fix: disable "redundant move" warnings for GCC (#10803)

GCC warns about redundant std::move while clang warns when they are missing, so we silence the less harmful one

Change: Units-system can convert from N to kN, don't preconvert.

This allows force to passed as is and avoid premature rounding.

The AI function "GetMaxTractiveEffort" still needs to return kN to avoid breaking the API.

Change: Standardise unit conversions and allow decimal places.

Previously the decimal_places member was mostly ignored except for
specific conversions. {DECIMAL} with 0 is the same as {COMMA} so there
is no downside to allowing any conversion to have decimals.

Change: Perform unit convert with a double-precision constant.

Unit conversion is only performed for display purposes, this does not
affect lock-step mechanics.

This replaces the old multiply and shift algorithm which relies on
choosing a multipler and shift combination that gets close. Some of these
multiply/shift combinations were quite inaccurate. We can just
use (close-to) real-world numbers instead.

Update: Translations from eints
catalan: 30 changes by J0anJosep
french: 25 changes by glx22
polish: 5 changes by pAter-exe

Fix: WWT_TEXT with SetTextStyle did not work. (#10797)

FontSize was passed to incorrect parameter of DrawString function.

Fix: Support more than 256 stations/waypoints/roadstops per class. (#10793)

It was already possible to define more than 256 per class, but not possible
to use them as the index used in GUI and passed through commands was limited
to a byte.

Codechange: C++-ify the Layouter and related functions

They all now access a std::string_view, instead of a "const char *"
or std::string (in some cases).

Additionally, GetCharAtPosition and friends now return an index
instead of a "const char *", as it makes for a more clear interface.

Fix: [ICU] crash when trying to break a non-breaking run (#10791)

Clusters from harfbuzz are indexed from the start of the buffer,
not from the start of the run analyzed. This confuses other parts
of the code that do assume they are from the start of the run.

Change: include fmt.h C++ headers in stdafx.h

This to prevent compilation issues between runs with and without precompiled
headers. Also remove the headers from the rest of the code base as they are
not needed there anymore, although they do relatively little harm.

Codechange: Rely on Scrollbar::SetPosition to clamp.

Manually clamping scrollbar bounds before calling `SetPosition()` is doubling up work that the function already does.

Codechange: Scrollbar methods now accept size_t.

This clears up a lot of casts from size_t to int.

Update: Translations from eints
vietnamese: 5 changes by KhoiCanDev

Codechange: Clean up build industry window. (#10779)

* Remove left-over code that treated an invalid list selection as 'fund
many', which is actually implemented as a separate button.
* Manual list management replaced with std::vector.
* Enabled state is only needed for the current selection.
* Selected index is not required only selected type.

Codechange: Scrollbar::UpdatePosition() will tell if the position changed. (#10777)

So we don't need to check this manually.

Update: Translations from eints
english (au): 5 changes by krysclarke
english (us): 5 changes by 2TallTyler
korean: 5 changes by telk5093
italian: 5 changes by Rivarossi
russian: 5 changes by Ln-Wolf
finnish: 5 changes by hpiirai
turkish: 5 changes by densxd
portuguese: 5 changes by azulcosta

Change: Replace ScriptLog data array with std::deque. (#10770)

Due to cyclic header dependency this requires moving the data types used
by ScriptLog out of the ScriptLog class.

Update: Translations from eints
japanese: 10 changes by fmang
vietnamese: 14 changes by KhoiCanDev
dutch: 14 changes by Afoklala
polish: 1 change by pAter-exe

Fix: Make all settingsgen 'warnings' fatal. (#10766)

Compilation should stop If settingsgen fails to complete properly.

Fix c6c3d0e6: restore string-based settings in network-private settings (#10765)

By accident the SDTC_SSTR got replaced with SDTC_VAR, which breaks
many of the settings in this file.

Add: Station property 1C/1D to set name/classname.

These are an alternative to the original "generic" C4xx/C5xx method,
which can only assign strings to IDs up to 256.

Change: Read Action 3 IDs as extended-bytes for all features.

This can be done because previous the value 0xFF (which indicates an
extended byte) was reserved for this purpose. Other features which may
not have mentioned reserving 0xFF do not allow this many IDs anyway.

This makes Action 3 consistent across all features. The allowable limits
for each feature do not change.