Fixing #128. Had to remove Attack#equals and #hashCode, which might e…

…nd up affecting other features that depend on them.

javelin/model/unit/Monster.java

@@ -381,9 +381,8 @@ public int getbaseattackbonus() {
 				classesbab += classdata.advancebab(i);
 			}
 		}
-		return classesbab
-				+ new Long(Math.round(originalhd * WeaponFocus.BAB.get(type)))
-						.intValue();
+		double bab = originalhd * WeaponFocus.BAB.get(type);
+		return classesbab + new Long(Math.round(bab)).intValue();
 	}
 
 	/**

javelin/model/unit/attack/Attack.java

@@ -128,16 +128,6 @@ public Attack clone() {
 		}
 	}
 
-	@Override
-	public boolean equals(Object o) {
-		return name.equals(((Attack) o).name);
-	}
-
-	@Override
-	public int hashCode() {
-		return name.hashCode();
-	}
-
 	public Spell geteffect() {
 		if (temporaryeffect != null) {
 			return temporaryeffect;

javelin/model/unit/feat/attack/PowerAttack.java

@@ -39,14 +39,18 @@ public void add(Combatant c) {
 		update(c.source);
 	}
 
-	private AttackSequence createattack(AttackSequence attack, int target) {
-		attack = attack.clone();
-		for (Attack a : attack) {
+	private AttackSequence createattack(AttackSequence sequence, int target) {
+		if (sequence.powerful) {
+			throw new RuntimeException(
+					"Cannot derivate from a Power Attack sequence!");
+		}
+		sequence = sequence.clone();
+		for (Attack a : sequence) {
 			a.bonus -= target;
 			a.damage[2] += target;
 		}
-		attack.powerful = true;
-		return attack;
+		sequence.powerful = true;
+		return sequence;
 	}
 
 	@Override