[Freeciv-Dev] Re: Unit move rate cleanup (PR#1343)

["Per I. Mathisen" <Per.Inge.Mathisen@xxxxxxxxxxx>]

On Sun, 24 Mar 2002, Raahul Kumar wrote:
> This cleans up unit move rate by getting rid of struct player *pplayer and
> replacing val with move_rate. I've also made it return a static int, and
> introduced an assert and -1 return value for errors.

 /***************************************************************
-...
+Unit move rate calculates the move rate of the unit taking into
+account the penalty for reduced hitpoints(affects sea and land units

Nitpick:                                  ^--space

+  default:
+    assert(0);
+    exit(1);
+    move_rate = -1;

No, never exit(1) when the game can be continued in a playable state!

And if you must exit, first call freelog(LOG_FATAL, ...) for a proper
error message, and then do exit(EXIT_FAILURE).

Other than that, I think the patch is very good and should go in.

Yours,
Per

"Capitalism is the extraordinary belief that the nastiest of men,
for the nastiest of reasons, will somehow work for the benefit of
us all." -- John Maynard Keynes