Remove fields/etc that are un-necessary for server-side functionality

cheahjs / TerrariaAPI-Server

Fork is now over at https://github.com/NyxStudios/TerrariaAPI-Server

https://tshock.co

32 stars 24 forks source link

Remove fields/etc that are un-necessary for server-side functionality #20

Closed MarioE closed 11 years ago

MarioE commented 11 years ago

For example, there are 2001 Dust instances and 501 Gore instances, and each instance in their arrays is created. We should remove stuff like this to decrease memory usage.

hakusaro commented 11 years ago

Okay, go ahead.

Olink commented 11 years ago

maybe after we fix the initial bugs;)

MarioE commented 11 years ago

Nevermind, this is rather pointless. I did a quick implementation of this (https://github.com/MarioE/TerrariaAPI-Server/) and it definitely was not much of an optimization (removed Dust, Gore, Star, ItemText, CombatText, Cloud).

Perhaps when an update is unlikely to occur, we could do this. It's not really worth the effort to do it each time the server updates, that's for sure.