Follow-up Comment #9, bug #13810 (project freeciv):

> _(pset_short_help(i))

Why not put the _() in setting_short_help() and write
in the comment header that it returns the translated
version? This goes for all such dupicated expressions.

Why do you use the setting number as first argument
to almost every setting function? The only function
that takes a setting id should be setting_by_number().

> for (i = 0; i < SETTINGS_NUM; i++) {

Use an iteration macro, either as the for loop or
using the generic iterator interface in utility/
iterator.h. Iterate on 'pset' which is a 'struct
setting *'.


----------------------------------------------------------------
なんだかむかつく。

    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


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

Reply via email to