On 30/06/2009, Matthias Pfafferodt <matthias.pfaffer...@mapfa.de> wrote: > > I would rename settings as options, ...
In the client the name "option" is already used for settings that are client-only (client/options.[ch]), so I don't think renaming to this in the server would be very helpful or useful. Is there some particular reason why you want to rename "setting" as "option" in the server? > ... move the all the struct as well as all the inline functions into > settings.c and define the missing functions. This way the struct > is only defined within settings.c (like the struct command in > command.c). Good. > TODO: > > 4. possibility of callback functions for options if changed > - aifill > - aitoggle (I suppose you meant 'autotoggle' instead of 'aitoggle'.) Yes, this is probably a good idea. The other possibility is to change these "side-effect settings" to commands. > 5. add functions to check / set options > - option_check(const struct settings_s *op, union setting_value bisval) > - option_set(struct settings_s *op, union setting_value bisval) It's not really consistent to prefix the functions with "option_" and then operate on "settings_s" pointers. I would suggest to just use "setting_" as the prefix and use "setting" as the struct name, in keeping with general object oriented conventions. You can also call the union something less verbose, perhaps 'union setval' (since it's not really part of the 'setting' "class", but just a little helper data structure). > 7. new option > - skill <string:easy|normal|hard> skill level of new AI players Fine, but in its own ticket please (each new feature should have its own patch). > 8. new command > - default (set default values) A command to do this has already existed a long time in warclient; it is called /reset and I think this is a slightly better name than /default (for example in the future it could take parameters so you could "/reset settings", "/reset map", "/reset teams" etc.). > 9. special fields within the ruleset > - game.default = [0/1] if all options should be set to the default values > - game.showchanged = [0/1] if changed values should be listed Is there a reason why you would want to disallow users from using the '/show changed' command? ---------------------------------------------------------------- 計画には反対しません。 _______________________________________________ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
