There is no need to explicitly call MoveTo() in PerformTradeandmovetonextmarket
Details
Details
- Reviewers
- None
- Commits
- rP24676: Fix market waypoints.
- Trac Tickets
- #5923
Create a trade route with waypoints.
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
Build has FAILED
builderr-debug-macos.txt fatal error: file '/Users/wfg/Jenkins/workspace/macos-differential/build/workspaces/gcc/../../../libraries/source/spidermonkey/include-unix-debug/mozilla/LinkedList.h' has been modified since the precompiled header 'obj/engine_Debug/precompiled.h.gch' was built note: please rebuild precompiled header 'obj/engine_Debug/precompiled.h.gch' fatal error: file '/Users/wfg/Jenkins/workspace/macos-differential/build/workspaces/gcc/../../../libraries/source/spidermonkey/include-unix-debug/mozilla/LinkedList.h' has been modified since the precompiled header 'obj/engine_Debug/precompiled.h.gch' was built note: please rebuild precompiled header 'obj/engine_Debug/precompiled.h.gch' 1 error generated. make[1]: *** [obj/engine_Debug/CConsole.o] Error 1 make[1]: *** Waiting for unfinished jobs.... fatal error: file '/Users/wfg/Jenkins/workspace/macos-differential/build/workspaces/gcc/../../../libraries/source/spidermonkey/include-unix-debug/mozilla/LinkedList.h' has been modif
Link to build: https://jenkins.wildfiregames.com/job/macos-differential/2923/display/redirect
See console output for more information: https://jenkins.wildfiregames.com/job/macos-differential/2923/display/redirectconsole
binaries/data/mods/public/simulation/components/UnitAI.js | ||
---|---|---|
5783–5789 ↗ | (On Diff #15436) |
Comment Actions
Tried setting three waypoints? first to somewhere, second to the other market, third to somewhere else?
(That gives quite strange behaviour.)
Comment Actions
As discussed on IRC this was already weird behaviour in A23, and I won't fix it right now because the code looks rather complicated. Thanks for reporting & testing, I'll merge this tomorrow.