Follow-up Comment #6, bug #13621 (project freeciv):

Thanks for looking at this patch.

Could A cleanup for the FIXME reminders
be the following:

- a function game_setting(op, bool *bval,
  int *ival, char* sval) which set the
  requested option (in stdinhand.c)

switch (op->stype) {
  case SSET_BOOL:
    ... checks + set value ...
  case SSET_INT:
    ... checks + set value ...
  case SSET_STRING:
    ... checks + set value ...
}

... do special things at the end ...

- call the function in stdinhand.c after the
  command argument is converted to bool/int/string
  and in ruleset.c after the value is read from
  the ruleset file

- handling of aifill and auto_ai_ toggle only if
  needed


    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?13621>

_______________________________________________
  Nachricht geschickt von/durch Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to