MCTowns

MCTowns

52.4k Downloads

Rewrite remote protocol to not suck

jmhertlein opened this issue ยท 3 comments

commented

Use packet objects and use one connection per person, none of this rapid connecting-disconnecting tomfoolery.

commented

This is a pretty boring thing to have to write, since it gives no real functionality gains over the old way, and it's immensely boring to re-code all the stuff that was boring to code the first time.

So, instead of putting off coding because I don't want to work on this, I'm putting this onto the back burner.

Removed from 2.3 milestone.

commented

So, what you are saying is that its boring? ;)

commented

Yeah. The remote admin client is an interesting feature, but people aren't exactly beating down my door for it, it doesn't offer any fundamentally new way to manage towns, and it does feel like it violates "Keep It Simple, Stupid!" for this project. The commit informs me I deleted 3000 lines of code when I removed all the code from the dev branch, which is probably about 40% of all the code in the project. That's a lot to maintain and debug (and it STILL WASN'T DONE ;_;).

So, while I learned huge amounts during all the time I worked on it, and I think it's a cool utility, it's not worth so much effort for so little gain right now. If anything, at this point I'd rather revamp the in-game methods for viewing towns and such rather than try putting it into a GUI. I think that code could do with a revisit anyway, it hasn't hardly changed since the early days of the project.