[Congdar]

It seems CalculateHeadingToTarget() returns a sint8
Everywhere it is used as a float though since heading is a float.
Also, in FaceTarget() heading is getting updated regardless of it needing to be updated or not because it is getting set directly since it is not a private or protected variable so here's an additional change for FaceTarget()

void Mob::FaceTarget(Mob* MobToFace) {
	if(!MobToFace) {
		if(!target) {
			return;
		}
		else {
			MobToFace = target;
		}
	}

	float oldheading = heading;
	float newheading = CalculateHeadingToTarget(MobToFace->GetX(), MobToFace->GetY());
	if(oldheading != newheading) {
		SetHeading(newheading);
		SendPosUpdate();
	}
}

Would changing the return value of CalculateHeadingToTarget() be as simple as this:
waypoints.cpp

sint8 Mob::CalculateHeadingToTarget(float in_x, float in_y) {

to:

float Mob::CalculateHeadingToTarget(float in_x, float in_y) {

and

	return (sint8) (256*(360-angle)/360.0f);

to:

	return (256*(360-angle)/360.0f);