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