Update of patch #1710 (project freeciv):

                  Status:          Ready For Test => In Progress            

    _______________________________________________________

Follow-up Comment #1:

the veteran level is used for two different things:

military units: battle experience
(./server/unittolls.c:maybe_make_veteran())

settlers/workers: skill level
(./server/unittolls.c:maybe_settler_become_veteran())

shouldn't this be two separate values as in 'veteran:military' (battle) and
'veteran:civil' (road, irrigation, ...)?

A settler can survive an attack and be a veteran:military and, after lots of
work, be also a veteran:civil. To keep only one value the flag 'NonMil' could
be used to differentiate between both possibilities. This way
./server/citytools.c:do_make_unit_veteran() would only apply to military
units (!NonMil).

Thoughs?

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?1710>

_______________________________________________
  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