search

BZFlag-Dev / bzflag

3D multi-player tank battle game
https://www.bzflag.org
Other
287 stars 80 forks source link

Complete bz_eGetWorldEvent #5

Open allejo opened 9 years ago

allejo commented 9 years ago

Currently, the bz_eGetWorldEvent allows you change the map the bzfs instance without killing the actual process but the event does not parse the options block to update the respective settings. In other words, if you start with a map that's FFA and try to use this event to switch to a CTF map, it'll remain FFA. This applies to everything in the options block, so BZDB variables, map settings, etc.

This would most likely require a refactor of how the options blocked is parsed since there's no function or class to parse a world file.

blast007 commented 9 years ago

This change is probably a bit too big and risky for a point release and should be a goal for 2.6.

allejo commented 9 years ago

What are some possible risks involved? Not that I'm against the idea of moving this to 2.6, I'm just curious.

blast007 commented 9 years ago

It would need to touch a lot of code and require a significant amount of testing to ensure we're not getting into some wacky state that's some crazy hybrid of, for instance, CTF and FFA. I'd rather such a feature be done during a longer development/testing phase like a new major release.

allejo commented 9 years ago

That seems fair. 2.6 it is then. We can make use of this event to add an official mapchange plug-in as well without the weird bash script setup