collinsmith / riiablo

Diablo II remade using Java and LibGDX
http://riiablo.com
Apache License 2.0
872 stars 99 forks source link

Set item counts persist across multiple loads #72

Closed collinsmith closed 4 years ago

collinsmith commented 4 years ago

Leaving and joining a number of games is throwing a ArrayIndexOutOfBoundsException [7]. The error is caused by CharData#equippedSets not being reset between games. I need to investigate when CharData should be reset, and which fields need to be cleared (could be more). The fix could be as simple as clearing equippedSets when the CharData items are loaded.

collinsmith commented 4 years ago

It appears additional fields will also need to be cleared -- this will be resolved in an improved CharData implementation which also fixes many consistency issues with D2S and CharData