Merge lp://qastaging/~widelands-dev/widelands/bug-1827033-shipping-algorithm into lp://qastaging/widelands
Proposed by
Benedikt Straub
Status: | Merged |
---|---|
Merged at revision: | 9127 |
Proposed branch: | lp://qastaging/~widelands-dev/widelands/bug-1827033-shipping-algorithm |
Merge into: | lp://qastaging/widelands |
Diff against target: |
68 lines (+18/-8) 2 files modified
src/economy/portdock.cc (+16/-7) src/logic/map_objects/tribes/ship.cc (+2/-1) |
To merge this branch: | bzr merge lp://qastaging/~widelands-dev/widelands/bug-1827033-shipping-algorithm |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
GunChleoc | Approve | ||
Review via email:
|
Commit message
Fix an assert fail because of nullptr destinations in the shipping algorithm
Description of the change
I have a feeling that this only masks the real issue. Portdock code first removes items that don´t have a portdock as destination and implicitly re-adds them at once. This causes *nullptr to be passed as an argument, resulting in the assert fail.
This branch simply prevents this situation by skipping something that doesn´t work anyway, so it shouldn´t have undesired side-effects. Someone who knows (or wrote) that code should check why the unallowed elements are adressed to a portdock when they should not be and therefore re-add themselves...
To post a comment you must log in.
Continuous integration builds have changed state:
Travis build 4896. State: passed. Details: https:/ /travis- ci.org/ widelands/ widelands/ builds/ 528437763. /ci.appveyor. com/project/ widelands- dev/widelands/ build/_ widelands_ dev_widelands_ bug_1827033_ shipping_ algorithm- 4677.
Appveyor build 4677. State: success. Details: https:/