Skip to content

Conversation

ArseniyShestakov
Copy link
Member

At first I'm want to make merging working as in H3 which is ready as well as few fixes for siege / battle handling. There is bigger plan to implement more merging options since H3 algorithm don't merge stacks of same type and you may end up with smaller army if you split stacks too much.

Merging algorithm is replicate one from original game.
@ArseniyShestakov
Copy link
Member Author

So I didn't found appropriate way to make sure battleFinished is called for CGHeroInstance since it's map object removed in endBattle. However everything else must work properly now and behaviour is replicate H3.

PS: Fixed issues: 169, 2252

@ArseniyShestakov ArseniyShestakov changed the title [WIP] Feature/armies merging Feature/armies merging Feb 22, 2016
@ArseniyShestakov
Copy link
Member Author

So I decide to test this one more and then merge without adding extra bloat. Advanced options will be in separate pull request since they require refactoring of current army merging code.

ArseniyShestakov added a commit that referenced this pull request Feb 22, 2016
@ArseniyShestakov ArseniyShestakov merged commit 7f1ddc4 into develop Feb 22, 2016
@ArseniyShestakov ArseniyShestakov deleted the feature/armiesMerging branch February 22, 2016 17:23
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

Successfully merging this pull request may close these issues.

1 participant