Age | Commit message (Collapse) | Author |
|
Sign width was only updated when the text was changed. This seems to work for player-placed
signs as there is always a rename operation, however AIs can create a sign with text in one
go, in which case the width was never set.
|
|
cargo generation. (#7412)
|
|
finnish: 12 changes by hpiirai
greek: 40 changes by Jubilee
luxembourgish: 2 changes by Phreeze
dutch: 12 changes by JanWillem
french: 4 changes by romazoon
norwegian (bokmal): 12 changes by Leifbk
hungarian: 14 changes by Brumi
portuguese: 51 changes by JayCity, 11 changes by vesgo
|
|
finnish: 13 changes by hpiirai
korean: 1 change by telk5093
|
|
|
|
|
|
|
|
|
|
|
|
|
|
a refit if required.
|
|
|
|
preventing industry creation.
As the information is always available from the tree ground type, unconditionally
update the map array for tree tiles.
|
|
|
|
|
|
This is a C++11 feature that allows the compiler to check that a virtual
member declaration overrides a base-class member with the same signature.
Also src/blitter/32bpp_anim_sse4.hpp +38 is no longer erroneously marked
as virtual despite being a template.
|
|
finnish: 13 changes by hpiirai
danish: 2 changes by nielsmh
|
|
|
|
luxembourgish: 31 changes by Phreeze
croatian: 2 changes by VoyagerOne
|
|
|
|
|
|
invalidating hangar orders that happen to share IDs.
This was caused because hangars are referred to by station ID, which is not unique with respect to depot IDs.
|
|
|
|
dutch: 28 changes by JanWillem
russian: 1 change by Lone_Wolf
|
|
from settings.
|
|
This is fixed by handling the new Window::OnTooltip() event instead of OnHover()
|
|
This avoids windows from needing to know or care about tooltip delay settings.
|
|
display or flickered.
Measurement tooltip was auto-closed as the hover/right-click test for tooltips was not
satisfied in this case. This is fixed by keeping the tooltip visible and instead explicitly
closing the tooltip when the PlaceObject is cancelled/completed.
|
|
This switch has been a pain for years. Often disabling broke
compilation, as no developer compiles OpenTTD without, neither do
any of our official binaries.
Additionaly, it has grown so hugely in our codebase, that it
clearly shows that the current solution was a poor one. 350+
instances of "#ifdef ENABLE_NETWORK" were in the code, of which
only ~30 in the networking code itself. The rest were all around
the code to do the right thing, from GUI to NewGRF.
A more proper solution would be to stub all the functions, and
make sure the rest of the code can simply assume network is
available. This was also partially done, and most variables were
correct if networking was disabled. Despite that, often the #ifdefs
were still used.
With the recent removal of DOS, there is also no platform anymore
which we support where networking isn't working out-of-the-box.
All in all, it is time to remove the ENABLE_NETWORK switch. No
replacement is planned, but if you feel we really need this option,
we welcome any Pull Request which implements this in a way that
doesn't crawl through the code like this diff shows we used to.
|
|
In 10 years there was no active development on DOS. Although it
turned out to still work, the FPS was very bad. There is little
interest in the current community to look into this.
Further more, we like to switch to c++11 functions for threads,
which are not implemented by DJGPP, the only current compiler
for DOS.
Additionally, DOS is the only platform which does not support
networking. It is the reason we have tons of #ifdefs to support
disabling networking.
By removing DOS support, we can both use c++11 functions for threads,
and remove all the code related to disabling network. Sadly, this
means we have to see DOS go.
Of course, if you feel up for the task, simply revert this commit,
and implement stub c++11 functions for threads and stub functions
for networking. We are more than happy to accept such Pull Request.
|
|
korean: 7 changes by telk5093
|
|
GetAircraftFlightLevel<Aircraft> is only used in static functions
inside aircraft_cmd.cpp. With GCC, Clang and MSVC this is not an
issue, but on ICC fails linking, because it doesn't find this
version of this template. Possibly these two pieces of information
are linked.
Explicit defining the function fixes the issue.
|
|
(#7382)
Company performance rating calculation does not take into account the companies' money when those exceeds INT_MAX.
|
|
dutch: 1 change by JanWillem
russian: 1 change by Lone_Wolf
|
|
|
|
|
|
norwegian (bokmal): 1 change by Leifbk
|
|
clamping to min or max value. (#7361)
|
|
romanian: 8 changes by alexmerlin1985
dutch: 2 changes by JanWillem
|
|
|
|
cargo type.
It is only an error if the invalid result is actually used. This will be silently ignored at the moment.
It is still an error if a duplicate cargo type is returned.
|
|
prevent incorrect modification.
|
|
industry tiles.
Currently this can only be triggered by NewGRF house tiles querying for cargo acceptance history
of nearby stations (var 0x64) with a tile offset, and providing an offset that happens to point
to an industry tile. This serves no useful purpose.
|
|
|
|
|
|
|
|
|
|
|
|
shown (#7368)
|
|
finnish: 110 changes by hpiirai
ukrainian: 149 changes by nsergiy
|