Small cleanup to default casualty logic. (#12696)

* Small cleanup to default casualty logic.

No logic changes since `numSelectedCasualties ` was equivalent to `defaultCasualtySelection.size()` already.

* .

game-app/game-core/src/main/java/games/strategy/triplea/delegate/battle/casualty/CasualtySelector.java

@@ -263,30 +263,28 @@ private static Tuple<CasualtyList, List<Unit>> getDefaultCasualties(
     final List<Unit> sorted =
         getCasualtyOrderOfLoss(targetsToPickFrom, player, combatValue, battlesite, costs, data);
     // Remove two hit bb's selecting them first for default casualties
-    int numSelectedCasualties = 0;
     if (allowMultipleHitsPerUnit) {
       for (final Unit unit : sorted) {
         // Stop if we have already selected as many hits as there are targets
+        final int numSelectedCasualties = defaultCasualtySelection.size();
         if (numSelectedCasualties >= hits) {
           return Tuple.of(defaultCasualtySelection, sorted);
         }
         final UnitAttachment ua = unit.getUnitAttachment();
         final int extraHitPoints =
             Math.min((hits - numSelectedCasualties), (ua.getHitPoints() - (1 + unit.getHits())));
         for (int i = 0; i < extraHitPoints; i++) {
-          numSelectedCasualties++;
           defaultCasualtySelection.addToDamaged(unit);
         }
       }
     }
     // Select units
     for (final Unit unit : sorted) {
       // Stop if we have already selected as many hits as there are targets
-      if (numSelectedCasualties >= hits) {
+      if (defaultCasualtySelection.size() >= hits) {
         return Tuple.of(defaultCasualtySelection, sorted);
       }
       defaultCasualtySelection.addToKilled(unit);
-      numSelectedCasualties++;
     }
     return Tuple.of(defaultCasualtySelection, sorted);
   }