Loïc Guilloux
6ff0858b38
Fix #9669 , cbe00ec: Only try valid reverse directions ( #9672 )
3 years ago
glx22
cbe00ec651
Fix: Try all possible reverse directions when a ship reaches a dead end
3 years ago
Loïc Guilloux
11dece205c
Revert 7ca1793: Using Trackdir keyed node is not required, Exitdir keyed node still have the correct trackdir ( #9576 )
3 years ago
Patric Stout
f87fe395a7
Fix: pathfinders always tried to avoid docking tiles (even if nothing was on them) ( #9522 )
...
When coming across any docking tile (for example, all tiles around
an oilrig are docking tiles), it always at least added a penalty
of 3 times a normal tile, even when there are no ships on them.
In result, the pathfinder got suggested to always go around docking
tiles. This was most likely not the intention of the change made in
31db4f8d5e
.
3 years ago
glx22
49b66ea504
Codechange: Remove FOR_EACH_SET_TRACK
3 years ago
Rubidium
281a65b3e1
Cleanup: simplify some boolean expressions
3 years ago
rubidium42
55a11710a6
Codechange: convert printf DEBUG statements to fmt Debug statements
3 years ago
Patric Stout
28e90769f7
Codechange: use "[[maybe_unused]]" instead of a wide variety of other ways we had
...
While at it, replace OTTD_ASSERT with WITH_ASSERT, as this
is always set if assert() is valid. No matter if NDEBUG is set
or not.
3 years ago
Vít Šefl
0125ba82e8
Fix: Do not send vehicles towards incomplete PF nodes
...
YAPF could end up in a situation where it sets the best intermediate node
to a node whose construction is never finalized (i.e. it is never added to
the open list). The content of the node would be overwritten in the next
round, potentially sending the vehicle to an unwanted location.
3 years ago
rubidium42
9197de39e4
Cleanup: remove unused copy-constructor without copy-assignment
3 years ago
Vít Šefl
33d99d27f4
Fix: Encountering two-way red signals could prune unrelated branches.
...
The intermediate node branch is now only pruned if the node is on the
path leading to the two-way red signal.
3 years ago
rubidium42
44ca7d9377
Change: Use gender-neutral pronouns
3 years ago
Patric Stout
665a3928e2
Remove: performance measurements in YAPF
...
YAPF was constantly measuring its performance, but only at
certain debug-levels this information was shown.
Now after years, I sincerely wonder if anyone still knows about this
feature and who still use it. Especially with the new framerate window,
this detailed performance is not as meaningful anymore as it once
was.
3 years ago
Patric Stout
e162aff7a3
Cleanup: remove weird left-over comment in yapf.hpp
3 years ago
Charles Pigott
e8a94dc8bb
Cleanup: Delete remaining Blob code
4 years ago
Charles Pigott
e8022a589d
Codechange: Replace CBlobT usage with std::vector
4 years ago
Charles Pigott
f481c9fc2c
Codechange: Replace CStrA with std::string
4 years ago
Charles Pigott
591ea9862d
Codechange: Suppress warnings when asserts are disabled ( #8916 )
4 years ago
Patric Stout
d4583fa64c
Fix #8123 : trams on half-tiles couldn't find depots ( #8738 )
...
Basically, follow_track.hpp contains a fix for half-tiles, but
this wasn't duplicated for when trying to find a depot and in
a few other places. This makes sure all places act the same.
4 years ago
Charles Pigott
80fb1c74f0
Change: Make pathfinder account for maximum order speed, if set
4 years ago
Charles Pigott
1d6a0c7b52
Fix: [YAPF] Road pathfinder did not account for length of tunnel/bridge in path cost
...
Copy calculations from the rail pathfinder
4 years ago
Charles Pigott
c461999b2b
Fix #8594 : [NRT] Road pathfinder did not account for roadtype speed limits
4 years ago
Charles Pigott
9b800a96ed
Codechange: Remove min/max functions in favour of STL variants ( #8502 )
4 years ago
Patric Stout
56d54cf60e
Add: introduce CMake for project management
...
CMake works on all our supported platforms, like MSVC, Mingw, GCC,
Clang, and many more. It allows for a single way of doing things,
so no longer we need shell scripts and vbs scripts to work on all
our supported platforms.
Additionally, CMake allows to generate project files for like MSVC,
KDevelop, etc.
This heavily reduces the lines of code we need to support multiple
platforms from a project perspective.
Addtiionally, this heavily improves our detection of libraries, etc.
4 years ago
Quipyowert2
acb3d10832
Codechange: Format unsigned integers with %u instead of %i or %d.
5 years ago
Jonathan G Rennison
1a88fb5c91
Fix #7592 : Do not cache road vehicle path within 8 tiles of destination with multiple entrances
...
Ported from jgrpp commit 79d5be7e26
5 years ago
S. D. Cloudt
13cc8a0cee
Cleanup: Removed SVN headers
5 years ago
Charles Pigott
71a3e83468
Fix: GCC9's warnings about deprecated implicit assignment operators
5 years ago
JMcKiern
04f659e768
Fix: Some typos found using codespell
5 years ago
peter1138
0db31ae27f
Change: Avoid caching end of ship path, to allow penalties to apply.
5 years ago
peter1138
31db4f8d5e
Add: Penalty for occupied docking points.
5 years ago
peter1138
f538179878
Feature: Multi-tile docks and docking points.
5 years ago
peter1138
c02ef3e456
Feature: Add NotRoadTypes (NRT)
5 years ago
peter1138
bcdb28249c
Codechange: Use RAILTYPES_NONE instead of INVALID_RAILTYPES when not using rail.
...
INVALID_RAILTYPES, if it was accidentally tested, would match any railtype.
5 years ago
Charles Pigott
9f3928658b
Codechange: Remove Track{dir,}{Bits,}Byte types
5 years ago
Henry Wilson
7c8e7c6b6e
Codechange: Use null pointer literal instead of the NULL macro
6 years ago
Charles Pigott
fe448a2616
Remove: OPF
6 years ago
Michael Lutz
cc5f175615
Feature: Railtype flags to allow/disallow 90 degree curves. ( #7352 )
6 years ago
PeterN
6c6971fb43
Add: Road vehicle path cache. ( #7261 )
6 years ago
Peter Nelson
7ca1793ec4
Change: Automatically use Trackdir node key as needed if ship curve penalties differ.
6 years ago
Peter Nelson
b8a0107ad1
Change: Add configurable curve penalty for ships.
6 years ago
Peter Nelson
3f327116db
Change: Don't apply forbid 90 deg turn settings to ships.
6 years ago
J0an Josep
029c48cf4a
Cleanup: [NPF] Remove unused parameter.
6 years ago
J0an Josep
d814dd3434
Cleanup: [NPF] Assert that a track has been chosen.
6 years ago
J0an Josep
aa63517c92
Fix #7060 : [NPF] Do not check whether ignored first tiles are end nodes.
6 years ago
J0an Josep
19be1f4ace
Codechange: [NPF] Add some consts.
6 years ago
J0an Josep
96c5e5e73a
Cleanup: [NPF] Remove unused parameters.
6 years ago
J0an Josep
a5836874ef
Codechange: Use track functions.
6 years ago
Peter Nelson
81330b8d6e
Change: Add path cache for ships.
6 years ago
J0an Josep
effb7da5b4
Doc: Fix spelling in comments.
6 years ago