Merge lp://staging/~widelands-dev/widelands/bug-1786163 into lp://staging/widelands
Proposed by
TiborB
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Merged at revision: | 8788 | ||||
| Proposed branch: | lp://staging/~widelands-dev/widelands/bug-1786163 | ||||
| Merge into: | lp://staging/widelands | ||||
| Diff against target: |
19 lines (+9/-0) 1 file modified
src/ai/defaultai_seafaring.cc (+9/-0) |
||||
| To merge this branch: | bzr merge lp://staging/~widelands-dev/widelands/bug-1786163 | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| GunChleoc | Approve | ||
|
Review via email:
|
|||
Commit message
When AI finds a second ship in expedition - it cancels the expedition. AI is written to control only one expedition at once.
Description of the change
AI should not start second expedition, but it happened. And this led to crash. I introduced very simple workaround here - just cancel such second expedition. Still it is mystery why this had happened. I introduced a log about this so player will get some more info about what is going on...
To post a comment you must log in.
Revision history for this message
| bunnybot (widelandsofficial) wrote | # |
Revision history for this message
| GunChleoc (gunchleoc) wrote | # |
Revision history for this message
| bunnybot (widelandsofficial) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Code LGTM :)