Closed Zoldszemesostoros closed 10 months ago
Hi @Zoldszemesostoros this sounds like a great refactor. I strongly encourage you to submit a PR with the changes you've made.
What kind of features did you remove? Definitely include the fundamental changes you bullet pointed.
I will submit a PR next week.
What kind of features did you remove?
Logging using an database; variable colored food particles; variable sized viruses
A few months ago, I toyed with this project. I have made several server-side changes, including
massFood.push({ /* lots of things */ })
becomesmassFood.push(new MassFood(playerFiring, cellIndex))
spectators
array for them.I have also made a sort-of-working game replay feature. I don't think every change should be upstreamed - I have got rid of some features I don't need, but others might, and I have made some changes to the protocol too - but I believe some of them can be useful. I can give you the source, or I can cherry-pick changes and submit PR-s if you are interested. I won't have too much time in July, so I can do the second option only in August.