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
code.


    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?22194>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to