[Freeciv-Dev] [bug #20026] Utility function to render possibly-fractional fractional MP as text

[Jacob Nevins]

Follow-up Comment #6, bug #20026 (project freeciv):

> It probably doesn't affect how you implement this version [...]
It's another argument for centralising it in the first place, though.

___

Reply to this item at:

  

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


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

[Freeciv-Dev] [bug #20026] Utility function to render possibly-fractional fractional MP as text

[Jacob Nevins]

Update of bug #20026 (project freeciv):

  Status:  Ready For Test => Fixed  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

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


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

[Freeciv-Dev] [bug #20026] Utility function to render possibly-fractional fractional MP as text

[Marko Lindqvist]

Follow-up Comment #3, bug #20026 (project freeciv):

It probably doesn't affect how you implement this version (cannot make it
better prepared to future) but I have considered making SINGLE_MOVE value like
30, and letting ruleset define size of the single FRACTION (so that classic
rules would have FRACTION = 10 -> 30 / FRACTION = 3 fractions for each full
movement points)

___

Reply to this item at:

  

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


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

[Freeciv-Dev] [bug #20026] Utility function to render possibly-fractional fractional MP as text

[Jacob Nevins]

Update of bug #20026 (project freeciv):

  Status: In Progress => Ready For Test 

___

Follow-up Comment #2:

* New function move_points_text() in movement.h
** Some of the options aren't used in this patch -- they are for bug #19851,
which I'll rebase on this patch.
* This does actually improve display of remaining MP for units which have fuel
and fractional MP (none in default ruleset) -- previously, the "fuel" bit
would be truncated.

(file #16303, file #16304)
___

Additional Item Attachment:

File name: trunk-move_points_text.patch   Size:19 KB
File name: S2_4-move_points_text.patchSize:19 KB


___

Reply to this item at:

  

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


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

[Freeciv-Dev] [bug #20026] Utility function to render possibly-fractional fractional MP as text

[Jacob Nevins]

Update of bug #20026 (project freeciv):

Severity:   2 - Minor => 1 - Wish   
 Summary: Help browser doesn't handle units with fractional MP
=> Utility function to render possibly-fractional fractional MP as text

___

Follow-up Comment #1:

Huh, well, it's not actually possible to put a unit with fractional MP in the
ruleset, so it's not really a bug.

Repurposing this for invention of a general-purpose movement-point rendering
utility, which I need for bug #19851 and will use in the help dialog anyway,
in case the ruleset ever gets the ability to express fractional MP.

___

Reply to this item at:

  

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


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