Follow-up Comment #4, patch #1589 (project freeciv):
I read especially common/city.[ch]:
* I don't understand the _placeholder for searching_ comments. Some values
are initialized somewhere else (e.g. pcity->worklist), some others don't
* _pcity->illness_trade = 0;_ could be moved into the
* Why do the memset() calls use _something * sizeof(*array)_ instead of
sizeof(array) which seems really safer?
> reduce size of the city structure to its minimal size.
* It is not really the case, but I am not sure it would be very useful for
Freeciv. The comment
/* Nothing behind the union please! */
may be dropped, because it doesn't make sense here.
Reply to this item at:
Message posté via/par Gna!
Freeciv-dev mailing list