Remove MovementCache - #147
Conversation
|
I feel like the movement cache is still a useful optimization. Did you notice anything particular that broke because of it? |
These patches I'm removing are doing too much for too little/nothing, imo. So why clutter the MC server code more with classes like these? |
|
I agree in that it probably doesn't have as much impact as say, the multi-threaded entity tracker. But I feel that it's best to keep these "micro" optimizations. (I believe this patch was more geared towards memory allocations rather than TPS) If it broke something or ended up making performance worse/increasing memory allocation, it would make sense to revert it. But otherwise, I don't see any reason to |
No description provided.