This needs a bit of work before it can be committed. ThrowingAttack is a member of Client, not Mob. So, it would look something like:
Client *c=CastToClient();
c->ThrowingAttack(target);
However, I don't agree with that method. To the client, Rage Volley is a spell and it should be treated as such. Even if its effects need to be added manually that would be fine, though I would imagine that is handled by the spell data. The change in spells.cpp is not needed as whether a spell is resistible or not is handled by the spell data. The change to the ammo consumption causes zone to crash, however again the spell data for Rage Volley does not consume an item, so it wouldn't anyway (and testing it without your code proves me correct.) A minor detail, but important isCasting has to be IsCasting. Also, I agree with AndMetal, spellids should only be specified to exempt if the spelltype cannot be used for whatever reason (and I can only think of two situations in the code where this happens.) I would take a look at the spell data:
http://lucy.allakhazam.com/spell.htm...54&source=Live and figure out what it handles. You might find you can nail it all in a line or two.