Follow-up Comment #10, bug #22194 (project freeciv):

> Leaving ticket open until this is resolved.

I think this patch had also internal conflict of goals. You calculate denomlen
for the constant denominator, but then you display MP reduced to lowest terms.
So if move_fragments = 10, you get denomlen == strlen("10") == 2, but in case
of 2/10 display just 1/5.

And now that I read the patch, you seem to have forgotten part of CodingStyle
that says that there's empty line between declaration of variables and actual


Reply to this item at:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to