Make specific action-related methods final.

src/org/apollo/game/action/Action.java

@@ -41,7 +41,7 @@ public abstract class Action<T extends Mob> extends ScheduledTask {
 	 * 
 	 * @return The mob.
 	 */
-	public T getMob() {
+	public final T getMob() {
 		return mob;
 	}
 

src/org/apollo/game/action/DistancedAction.java

@@ -54,12 +54,11 @@ public abstract class DistancedAction<T extends Mob> extends Action<T> {
 	}
 
 	@Override
-	public void execute() {
+	public final void execute() {
-		if (reached) {
+		if (reached) { // Don't check again in case the player has moved away since it was reached
-			// some actions (e.g. agility) will cause the player to move away again
+			executeAction();
-			// so we don't check once the player got close enough once
+			// TODO checking the walking queue size is a really cheap fix, and relies on the client not
-			executeAction(); // TODO checking the walking queue size is a really cheap fix, and relies on the client not
+			// being edited... this class needs to be completely re-written.
-								// being edited... this class needs to be completely re-written.
 		} else if (mob.getPosition().getDistance(position) <= distance && mob.getWalkingQueue().size() == 0) {
 			reached = true;
 			setDelay(delay);

src/org/apollo/game/scheduling/ScheduledTask.java

@@ -65,7 +65,7 @@ public abstract class ScheduledTask {
 	 * @param delay The delay.
 	 * @throws IllegalArgumentException If the delay is less than or equal to zero.
 	 */
-	public void setDelay(int delay) {
+	public final void setDelay(int delay) {
 		if (delay < 0) {
 			throw new IllegalArgumentException("Delay cannot be less than 0.");
 		}