little bump here
A small note about the 101 chance:
I always thought that 101 chance would mean "hits always"
But it is possible that the 101 chance is an entirely independend chance calculation, and in these cases we just need to use the other values for calculation
Some more notes about targeting:
spell
http://www.wowhead.com/spell=4064 has no unit-targeting, but is AoE-Dest-Targeting, so this calculation would not fit into CheckCast.
Also it would be required to ensure that the code is safe (not checked if getUnitTarget returns a valid pointer)