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

Multi-tile Units does not handle damage correctly #23

Open
perara opened this issue Sep 11, 2021 · 0 comments
Open

Multi-tile Units does not handle damage correctly #23

perara opened this issue Sep 11, 2021 · 0 comments

Comments

@perara
Copy link
Member

perara commented Sep 11, 2021

deep-rts/src/Tile.cpp

Lines 262 to 270 in 148df8d

void Tile::update(Unit& unit) const {
// TODO - does not handle multi-tile blocks (for example Town-Hall)
// TODO - maybe this is OK?
if(hasOccupant() && damageModifier > 0){
unit.afflictDamage(std::ceil(damageModifier));
}
}

In multi-tile units, the damage is only applied to the "primary tile". Units currently only reference1 tile, but perhaps there should be a list of "secondary tiles"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant