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

> static inline bool setting_boolval(conct struct setting_s *op)

That should be "const" of course.

I'm also not sure about the naming scheme, since
with the full "setting_" prefix this makes it a
little awkward to use in expressions.

Perhaps just SBOOLVAL or some similar short
macro-like name (beware conflicts though, both
literal and conceptual :)).


----------------------------------------------------------------
カンック?

    _______________________________________________________

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