Unnecessary instantiation of Thread objects
BlayTheNinth opened this issue ยท 2 comments
All of the packet handlers are instantiating new Thread
s, when all that is really needed here are Runnable
s.
See PacketConfigurationUpdate.java#L36 for example.
They are then just casted up to Runnable
s further down the line, so them being Thread
s in the first place is a waste in terms of memory allocation and performance (see Thread.java for the overhead that's happening for every packet at the moment).