Fixes issue 253.

This needs a review. I couldn't see any need for the second loop which would be infinite for targets with no upper limit.
This commit is contained in:
North 2011-09-12 22:49:49 +03:00
parent 0c7d9b5e61
commit 17ca2de0c9

View file

@ -213,11 +213,6 @@ public abstract class TargetImpl<T extends TargetImpl<T>> implements Target {
}
chosen = targets.size() >= minNumberOfTargets;
}
while (!doneChosing()) {
if (!player.choose(outcome, this, game)) {
break;
}
}
return chosen = true;
}
@ -231,11 +226,6 @@ public abstract class TargetImpl<T extends TargetImpl<T>> implements Target {
}
chosen = targets.size() >= minNumberOfTargets;
}
while (!doneChosing()) {
if (!player.chooseTarget(outcome, this, source, game)) {
break;
}
}
return chosen = true;
}