Get rid of agentdata system all together

etfdevs / ETF

ETF mod

GNU General Public License v3.0

14 stars 3 forks source link

Get rid of agentdata system all together #34

Open ensiform opened 2 years ago

ensiform commented 2 years ago

This extra entity system is very wasteful and we can actually (re)use a lot of fields within the entitystate and playerstate if needed, as ET has a ton of extra free unused fields on player types and in general in ETF they are not used at all.

[ ] Restructure disguise/invis to use new fields
[ ] Remove agentdata code all together
[ ] See #33
[ ] Make gas hallucinations use a different local cache since agentdata won't be necessary anymore

ghost commented 2 years ago

Hello, please excuse me if I'm totally wrong, but wouldn't be UEIDs perfect for fixing this? Tremulous uses UEIDs. I assume it would be hard to completely rewrite ETF's entity system to match the Tremulous style of how to deal with entities. Anyways, would it be worth at all? I have to admit I don't have the skills, to make that happen. It just crossed my mind and I followed Tremulous for a while and saw that they really spent a lot of time to improve the entity system and memory managemant.

ensiform commented 2 years ago

I do not see the relevance to this "problem". There is no need to have extra entities at all.

ghost commented 2 years ago

Ah, ok, it seems I didn't even understand the problem. Sorry for that.

ensiform commented 2 years ago

I have not currently pushed the changes yet, but I have already fixed the corpse task and the gas hallucination task. Haven't pushed because not yet confident I have done it right without more clients available to testing.