Codechange: Use IsValidCargoID/IsValidCargoType.

IsValidCargoType() is used only for unmapped IDs.

Codechange: Rename IsCargo..Valid to IsValidCargo..

This better fits our naming pattern for this type of function.

Update: Translations from eints
galician: 41 changes by pvillaverde
esperanto: 95 changes by legoscia
portuguese (brazilian): 18 changes by ericandradex

Fix: work around GCC compiler bug #105120 in fmt

The fmt code pushes a pragma option, and later pops is. The intrinsics code
interacts with it via the __OPTIMIZE__ macro. This has been set by the pragma
option push, but not unset/reset to the original with the pop.
Since the pragma is only used for the GCC compiler (not Clang, not MSVC, not
ICC) and in debug mode, just remove the whole pragma handling for it.

Codechange: fmt (and std::format) do explicitly not support enums out-of-the-box

That it works for the version we have packaged it pure coincidence, as that is
one of the few versions that due to a bug allow it. So add the appropriate
template specialisations to support it out-of-the-box within OpenTTD.

Update: Translations from eints
esperanto: 52 changes by legoscia

Change: Allow dedicated server to use threaded saves. (#10787)

Co-authored-by: Jonathan G Rennison <j.g.rennison@gmail.com>

Fix: survey result on crash only worked on Linux (#10855)

Every OS-specific crashlog handler has their own MakeCrashLog
in some form. In result, only Linux was calling the generic one.

Update: Translations from eints
catalan: 12 changes by J0anJosep
esperanto: 678 changes by legoscia

Codechange: Use display list instead of exclude list for file window. (#10845)

This simplifies retrieving the correct data for each row when data is
filtered. The background FileList is left intact so that savegame data
does not have to be rescanned when the filter is changed, and sorting
still remains the task of the background FileList.

Update: Translations from eints
catalan: 14 changes by J0anJosep
dutch: 14 changes by Afoklala

Codechange: Use std::array for GRF(File|Config) parameters.

This simplifies comparison, copying and assignment operations.

Update: Translations from eints
russian: 2 changes by Ln-Wolf

Codechange: Use unique_ptr for fios company properties.

This is the data used to show company names in the saveload window.

Codechange: Use unique_ptr for text layout font mapping.

This must stay a pointer as the value passed to other structures.

Codechange: Swap SocketList map key/value around.

This map is used store socket and address together, and, other than
checking that the address does not already have a socket, the data layout
does not seem particularly important.

However, as address is the key, technically it should not be modified,
and address may self-modify itself during comparisons.

Fix: Don't rely on static initialization to set up sprite font caches.

The order of static initialization is undefined, so this can cause initalization
before relevant caches are initializations.

Update: Translations from eints
romanian: 11 changes by bnegrut
turkish: 14 changes by densxd

Update: Translations from eints
english (au): 14 changes by krysclarke
english (us): 14 changes by 2TallTyler
vietnamese: 14 changes by KhoiCanDev
korean: 19 changes by telk5093
italian: 14 changes by Rivarossi
russian: 14 changes by Ln-Wolf
finnish: 14 changes by hpiirai
danish: 22 changes by bscargo
portuguese: 14 changes by azulcosta
polish: 14 changes by pAter-exe

Fix #10811: Crash getting row from non-resizable widget. (#10833)

GetScrolled*FromWidget took line height from the widget's resize_y value,
however not all widgets are resizable, resulting in a division-by-zero.

Allow passing line height explicitly in cases where a widget is not
resizable.

Codechange: Use std::any_of() (#10830)

When the result of std::find_if is compared only with end()
then '!= end()' is replaced with any_of().

Just... there's only one.

Feature: opt-in survey when exiting a game

On first start-up, the game will ask if you want to participate
in our automated survey. You have to opt-in, and can easily opt-out
(via the Options) at any time.

When opt-in, whenever you exit a game, a JSON blob will be send
to the survey server hosted by OpenTTD. This JSON blob contains
information that gives a global picture of the game just played:
- What settings were used
- How many humans vs AIs
- How long the game has been played
- Basic information about the OS / CPU

All this information is kept very generic, so there is no
chance we send private information to our survey server.
Nothing in the JSON blob could identify you as a person; it
mostly tells about the game played. At any time you can see
what the JSON blob includes, by pressing the "Preview Survey
Results" button in-game.

Fix: multiplication result converted to larger type

Technically unlikely to happen, though uint16 * uint16 get promoted to int and
then stored as uint64; similarly uint * uint16 remains uint and gets stored as
uint64. In both cases the value can get truncated before the change to uint64.

Update: Translations from eints
vietnamese: 4 changes by KhoiCanDev
hungarian: 74 changes by PstasDev
turkish: 4 changes by densxd

Codechange: Pass reference instead of pointer to GUI*Lists. (#10822)

Pointer-avoidance.

Codechange: Use std::move for bridge list. (#10821)

This follows the pattern for dropdown lists, and avoids new/delete and pointers.

Fix #10819, Fix #10811: GetVehicleFromDepotWndPt used widget- instead of window-relative positions. (#10820)

Pass window-relative positions instead, and adjust for relative positions where needed within the function itself. This simplifies calling code.

Fix: Don't use a loop to test if classid is valid. (#10818)

Additionally the Object class test was broken.

Codechange: rework NewGRFProfiler to use ticks instead of calendar-days (#10815)

We are planning to allow things like freezing the calendar, which
makes this variable a bit problemetic. So instead, suggest to the
user how many ticks there are in a calendar day, and let them figure
out how many ticks they want.

Additionally, use a TimeoutTimer for this, instead of an end-date
variable which is checked in an IntervalTimer.

Update: Translations from eints
finnish: 4 changes by hpiirai

Codechange: Make NEW_STATION an actual item in join station list.

Add NEW_STATION to the nearby station list to indicate that a new
station should be built. This removes special-casing for a non-existant
list item and keeps the list count and scrollbar count the same.

Fix: NewGRF Profile didn't stop if there were no events yet (#10816)

This meant you could have the following situation:
- You start a profile on a GRF with no events, for N days.
- The days pass, the profile should stop. It doesn't.
- The profile will never stop, even if the GRF start generating events.
- There is no real way to discover this, so .. byebye memory? :)

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