Bot Spell Casting Fixes

[bad_captain]

Thanks for the idea. I will work on adding this on, but in its current state, it is no worse than it was before (I guess for clients, it should still find the one with the lowest hpr, instead of just the first one who needs to be healed, and is still better than before).

I guess I just need to know if I should change it before it would be considered for being committed.

[c0ncrete]

I haven't looked through your code too much, but I'm sure there is significant improvement.

It might be easiest to just make mob->IsEgnaged() a virtual function and overload it in the client object. I can't think of anything that would break. The only time IsEngaged() is used in reference to a client is in the bot code as far as I'm aware. That way, you wouldn't have to mess with all of the group iterations in the bot code.

[realityincarnate]

If you're going to use AggroCount to track whether the client is engaged (and using that with a virtual IsEngaged() seems like a good idea to me), you'll also want to change the IncrementAggroCount() and DecrementAggroCount() functions around a little bit. Currently, they only actually change AggroCount if the rest regen rules are in effect.