Thread:FixSRwiki/@comment-46261745-20200829124347/@comment-26544499-20200830134654

Well... that's a lot to look over. Thanks!

I've tried out your speed mod. It's a bit unwieldy, but I guess that it might be a step in the right direction. I think that one big message box would've done the job better than several smaller ones. I'm also not quite sure where it's factoring in move-based delay or why the first box always seems to say that the character's class-based delay is 15. But hey, it works!

My impression is that add1 is the only thing populating the m_anTarget1 array and that add1 is only called for the two Snipe moves. The array itself is only used by the check function that the select method calls. Basically, as I read it, this stuff only matters for Ninja Snipe and Footsoldier Snipe. Even then, I'm half sure that there's a 30% chance that the unit with the Snipe moves picks a random target instead of a random Ninja/Foot.

...And that's assuming that there's no silly glitches. For example, I suspect that this code might cause the 70% that targets the appropriate class to never even try to target the front row. I'd be quite interested to see what would happen if we took a force of six Ninjas to fight a force of 6 Archers with low Atk, many actions, and Bow Attack + Ninja Snipe. My prediction is that 70% of the time when they are using Ninja Snipe, they'd attack the back row.