Follow-up Comment #5, patch #1500 (project freeciv): > Are you sure about SSET_META? I would have saide SSET_FLEXIBLE.
I'm not sure about this setting and did keep it from the original patch. As SSET_META is used for similar settings (timeout, ...) this seems to be a good choice. A definition of the purpose of these settings is missing. We have (from setting_is_changeable()): SSET_MAP_SIZE: (only 'size' and 'topology') SSET_MAP_GEN: (generator related settings) Only change map options if we don't yet have a map SSET_MAP_ADD: (special map settings; scenarios?) SSET_PLAYERS: ('minplayer', 'maxplayer', 'aifill') SSET_GAME_INIT: (at the start of the game) SSET_RULES: (fixed game settings)) Only change start params and most rules if we don't yet have a map, or if we do have a map but its a scenario one (ie, the game has never actually been started). SSET_RULES_FLEXIBLE: (flexible game settings) SSET_META: (network settings; not related to the game) These can always be changed. >From the list above I see that there is a good argument for SSET_META - its game related. I'm OK with both choices _______________________________________________________ Reply to this item at: <http://gna.org/patch/?1500> _______________________________________________ Nachricht geschickt von/durch Gna! http://gna.org/ _______________________________________________ Freeciv-dev mailing list Freecivfirstname.lastname@example.org https://mail.gna.org/listinfo/freeciv-dev