Follow-up Comment #6, patch #4077 (project freeciv):

Read the pathc, not tested yet. Looks promising, though I found some

- As you touch the code blocks in clients by moving them, you could and should
fix the style:



if (xxx) {

- There was "true" and "false" used instead of "TRUE" and "FALSE" at least in
one place
- In actions.c add comment about the directory include files reside
- Comment typofix "pleace" -> "please"
- "disbled" -> "disabled"
- Capitalize also second word in gen_actions specenum names "Poison City",
"Sabotage Unit"
- Unless you have good reason not to, move either action_init() or
action_free() call so that various frees are in reverse order to inits
(lifetime of one object type is contained within lifetime of another object
- action_init() and action_free() should be plural instead: actions_init() &


Reply to this item at:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to