Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Thanks to @theultramage for reporting and pinpointing the error as always! Commiting this one for @aleos89, he fixed it for you guys.
- Loading branch information
1 parent
2f471a1
commit 4871dab
Showing
2 changed files
with
14 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4871dab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mhm, okay. So L14141 limits WL_COMET to MAX_PARTY, but the code in battle.c still has
p_sd[5]
.Furthermore, the way you split up that check is not equivalent to the old code. You are now missing the "*c >= 1 in all other cases" check. I see AB_ADORAMUS and WM_GREAT_ECHO also calling this. I too had trouble rewriting this check into something more readable.
4871dab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WM_GREAT_ECHO is a Chorus skill, so it should get flagged by:
And return:
I left out the miscellaneous case because we pretty much have all the other cases checked. If we just have:
It would be more of a fail safe check I guess.
You are correct about WL_COMET and AB_ADORAMUS though. Thanks!