[Freeciv-Dev] Re: Profiling Civserver again

[蔡恆華 <iquin@xxxxxxxxxxxxx>]

Seems like the patch inflates mapgen.c from ~61K to ~98K.
Of course, some of the macro definitions don't belong there.
Sometimes, code simplicity is more important than speed.
Not sure if it is worth going in this extreme for speed.
Mapgen is only called once per game after all.
However, there are redundant checks which should have been removed.
Another thing to consider is memory consumption,
speeding up the computation at the expense of memory can cause additional
memory swap which slows down everything even more.

Regards,
I.Q.