A symbol clash breaks building ICU on macOS, and although it isn't
necessary, it might as well be possible.

Bug introduced via commit ab711e6

)

This fixes a bug that can reproduced with these steps:
* Start openttd in fullscreen mode
* Turn off fullscreen mode
* Try to resize the window. The window can't be resized.

NetworkTCPSocketHandler::SendPacket is not thread safe and may not
be used concurrently from multiple threads without suitable locking

Remove static buffer form of NetworkAddress::GetAddressAsString.
This is used in multiple threads concurrently, and is not thread-safe.

Replace it with a form returning std::string.

conected and aborted flags are used concurrently from multiple threads.

…ndgame screen

…st road/tramtype

…TD#8263)

…8273)

Based on report in <https://www.tt-forums.net/viewtopic.php?f=31&t=87374>

This changes the error when you attempt to convert a road owned by a town to another road, specifying that it's owned by the town rather than simply being incompatible. As the original poster of the issue pointed out, these seemed to be the only occurences of the incompatible road string, so now it's unused, but they would be left untouched in case of future use or since changing it to a different error would do the work of removing it then. If requested, it likely wouldn't be too difficult to remove the string entirely.

…penTTD#8240)

This could be caused by an engine being renamed, and the old
name being cached from a previous sort.

See: OpenTTD#7838

This could be caused by a group being renamed, and the old
name being cached from a previous sort.

See: OpenTTD#7838

…tead of using desync-prone town caches

…ing the savegame version