Closed GoogleCodeExporter closed 8 years ago
Oh forgot to add this is on revision 101
Original comment by azzur...@gmail.com
on 8 Nov 2010 at 11:13
oh ok it has to be this change (the addition of Thread.remove() in revision 99)
https://code.google.com/p/evolutionchamber/source/diff?spec=svn99&r=99&format=si
de&path=/trunk/EvolutionChamber/src/java/com/fray/evo/EvolutionChamber.java
Dunno what this does so somebody else please take a look at it ^^
Original comment by azzur...@gmail.com
on 8 Nov 2010 at 11:17
I don't understand the point of this code either. It does not appear to break
anything when removed.
Original comment by AudioL...@gmail.com
on 9 Nov 2010 at 1:13
Neither do I. The exception is thrown because it's trying to modify the
threads list as it's iterating over it. The threads list is cleared once all
the threads have stopped, so I don't think the threads.remove() code is
necessary.
Original comment by mike.angstadt
on 10 Nov 2010 at 5:31
It looks like Lomilar fixed this in in r117. Closing.
Original comment by mike.angstadt
on 24 Nov 2010 at 2:35
Original issue reported on code.google.com by
azzur...@gmail.com
on 8 Nov 2010 at 11:11