external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-18 11:48:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3020 from kevlahnota/newmaster2

Browse Source 
prevent OutOfMemoryError on chooseNumber
This commit is contained in:
Anthony Calosa
2023-04-27 18:54:21 +08:00
committed by GitHub
parent 83d1578b73 d9c59b80d1
commit 1efb90c593
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
forge-gui/src/main/java/forge/player/PlayerControllerHuman.java
View File

@@ -682,7 +682,11 @@ public class PlayerControllerHuman extends PlayerController implements IGameCont
return min; return min;
} }
final ImmutableList.Builder<Integer> choices = ImmutableList.builder(); final ImmutableList.Builder<Integer> choices = ImmutableList.builder();
for (int i = 0; i <= max - min; i++) { // todo check for X cost or any max value for optional costs like multikicker, etc to determine the correct max value,
// fixes crash for word of command OutOfMemoryError when selecting a card with announce X or Multikicker since
// it will build from 0 to Integer.MAX_VALUE...
int size = max == Integer.MAX_VALUE ? 9 : max - min;
for (int i = 0; i <= size; i++) {
choices.add(Integer.valueOf(i + min)); choices.add(Integer.valueOf(i + min));
} }
return getGui().one(title, choices.build()).intValue(); return getGui().one(title, choices.build()).intValue();