Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Work around AI orders to invalid cells. #14974

Merged
merged 1 commit into from Mar 22, 2018

Conversation

Projects
None yet
2 participants
@pchote
Copy link
Member

pchote commented Mar 22, 2018

Fixes #14923.
Fixes #14863.
Fixes #14820.

The main (but its possibly not only) cause of these crashes is the AI trying to blindly attack-move units to the location of a target actor in a different movement layer. NearestMoveableCell is not aware of layers, so will happily return a bogus location that then causes a crash.

@abcdefg30 abcdefg30 merged commit d79d447 into OpenRA:bleed Mar 22, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@abcdefg30

This comment has been minimized.

Copy link
Member

abcdefg30 commented Mar 22, 2018

@pchote pchote deleted the pchote:fix-ai-attackmove-crash branch Apr 28, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.