rP22225 hurriedly fixed a crash, but made the JPC cache fail - though we are not using it and it might fail for other reasons still.
Details
- Reviewers
Itms - Group Reviewers
Restricted Owners Package (Owns No Changed Paths) - Commits
- rP22301: JPS - make the JPC cache usable again (rP22225 fix)
Review code
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- temp
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 7579 Build 12355: Vulcan Build Jenkins Build 12354: arc lint + arc unit
Event Timeline
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/1364/display/redirect
source/simulation2/helpers/LongPathfinder.cpp | ||
---|---|---|
722–725 | Use the explicit type, see comment on the commit :) |
What Stan said, unless the complete type is actually very long (not on desktop, I can't check).
I liked the extra line between lines 722 and 723 but at that point I'm just being annoying ?
Apart from that this is good for me.
no more auto (tbh here I feel like it would have been legitimate, but I'm definitely more on the "auto" side than most of you guys).
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/1478/display/redirect
source/simulation2/helpers/LongPathfinder.cpp | ||
---|---|---|
722 | Add a space between the two closing >. Sub-par compilers (can't remember which ones) sometimes mistake that for a >> operator. |
source/simulation2/helpers/LongPathfinder.cpp | ||
---|---|---|
722 | That's... Retarded... |
source/simulation2/helpers/LongPathfinder.cpp | ||
---|---|---|
722 | (2019 and this is still an issue -_-) Are these compilers actually supported nowadays? Does VS2013 exhibit this? |
source/simulation2/helpers/LongPathfinder.cpp | ||
---|---|---|
722 | I actually think splitting them helps with readability, so I'd push for doing it anyway. But yes IIRC the issue still exists with 2013 and is fixed in 2015. |
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/1487/display/redirect