scarf005 / Marisa

A continuation mod for Slay The Spire, which adds Marisa from touhou project as a playable character.
Other
15 stars 7 forks source link

exception when playing unstable bomb #202

Closed allaryin closed 1 year ago

allaryin commented 1 year ago

How to reproduce

  1. play unstable bomb
  2. crash some times

I believe this might only be happening when there is only one enemy in the fight? I just played through act 1, and almost (but not quite) every time I played Unstable Bomb, there was a crash with the same exception.

Don't feel like reading Kotlin right now, but I'm guessing this is just a bad bounds check (num enemies - 1) sort of deal.

Crash log

Version Info:

Mods in stacktrace:

scarf005 commented 1 year ago

i wasn't able to reproduce it, could you provide exact console command instructions to reproduce the issue reliably? in the video i fight two enemies and a single enemy, yet it does not crashes.

https://user-images.githubusercontent.com/54838975/233778034-d21c2fd8-d822-4316-91c3-49038ef88641.mp4

HeJuiczehn commented 1 year ago

just encountered the same bug with same crash log. when use "upgarde!" 2 times, and then "unstable bomb" will cause random damage (5 to 4) 4 times, and the game definitely crash if use this card. maybe random number generator requires the first number not bigger than the latter one or something.

scarf005 commented 1 year ago

thank you. i was able to reproduce the issue, and it was indeed a random range issue. i'm currently working on a bugfix for it.