Slay the Spire

Slay the Spire

The Death Knight
arek75 17 Mar, 2021 @ 12:54pm
Crash at the start of act 2.
Did a few runs, then got a persistent crash at the start of Act 2, 2 thieves fight, crashes at the start of the fight with the following error:

java.util.ConcurrentModificationException: null at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:901) ~[?:1.8.0_144] at java.util.ArrayList$Itr.next(ArrayList.java:851) ~[?:1.8.0_144] at basemod.BaseMod.publishStartBattle(BaseMod.java:2535) ~[?:?] at basemod.patches.com.megacrit.cardcrawl.rooms.AbstractRoom.StartBattleHook.Insert(StartBattleHook.java:20) ~[BaseMod.jar:?] at com.megacrit.cardcrawl.rooms.AbstractRoom.update(AbstractRoom.java:295) ~[?:?] at com.megacrit.cardcrawl.dungeons.AbstractDungeon.update(AbstractDungeon.java:2532) ~[?:?] at com.megacrit.cardcrawl.core.CardCrawlGame.update(CardCrawlGame.java:871) ~[?:?] at com.megacrit.cardcrawl.core.CardCrawlGame.render(CardCrawlGame.java:419) [?:?] at com.badlogic.gdx.backends.lwjgl.LwjglApplication.mainLoop(LwjglApplication.java:225) [?:?] at com.badlogic.gdx.backends.lwjgl.LwjglApplication$1.run(LwjglApplication.java:126) [desktop-1.0.jar:?] Controllers: removed manager for application, 0 managers active 19:32:31.525 INFO core.CardCrawlGame> PAUSE()

I had just picked up Pandora's Box, which made me think about something. Does Broken Runeblade try to enumerate the strikes in your deck at the start of combat and assume some minimum number of them? If so, please stop assuming there are basic strikes available to enumerate (or a minimum number of strike cards, even 1), since lots of people remove strikes from their deck to make room for "better" abilities. I specifically tested for this (albeit minimally) in game and this doesn't seem to be the case. Still, the above error looks like an index out of range type error, so making sure you're not making assumptions with array lengths is still a Good Thing(tm)

--Arek75
Last edited by arek75; 18 Mar, 2021 @ 2:30am