[Freeciv-Dev] [patch #4931] pf_fuel_map_iterate(): adjustements

2014-08-03 Thread pepeto
Follow-up Comment #10, patch #4931 (project freeciv): I think the only change that is notable for the user is that units with more than 1 turn of fuel really can wait at all refuel points. Sometimes, the calculation was not done due to a problem of synchronization between the normal and the

[Freeciv-Dev] [patch #4931] pf_fuel_map_iterate(): adjustements

2014-08-02 Thread Jacob Nevins
Follow-up Comment #9, patch #4931 (project freeciv): Can you summarise the changes players might see as a result of this patch (if any), for release notes? Does it fix any cases where planes could run out of fuel, or anything like that?

[Freeciv-Dev] [patch #4931] pf_fuel_map_iterate(): adjustements

2014-07-21 Thread pepeto
Update of patch #4931 (project freeciv): Status: Ready For Test = Done Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #4931] pf_fuel_map_iterate(): adjustements

2014-07-19 Thread pepeto
Update of patch #4931 (project freeciv): Status: Ready For Test = In Progress Assigned to:None = pepeto ___ Follow-up Comment #4: I think a version

[Freeciv-Dev] [patch #4931] pf_fuel_map_iterate(): adjustements

2014-07-19 Thread pepeto
Update of patch #4931 (project freeciv): Status: In Progress = Ready For Test Planned Release: 2.6.0 = 2.4.3, 2.5.0, 2.6.0 ___ Follow-up Comment #5: Attached patch for

[Freeciv-Dev] [patch #4931] pf_fuel_map_iterate(): adjustements

2014-07-12 Thread pepeto
Follow-up Comment #2, patch #4931 (project freeciv): Patch #4926, comment 1: I said since bug #20361, there were a SINGLE_MOVE cost hack except for pf_fuel_map. Actually, this was not right. This hack was present there too and I forgot to remove it in patch #4889. This part my be moved to patch

[Freeciv-Dev] [patch #4931] pf_fuel_map_iterate(): adjustements

2014-07-12 Thread Emmet Hikory
Follow-up Comment #3, patch #4931 (project freeciv): Aha, so patch #4926 doesn't actually finish cleaning up the SINGLE_MOVE hack, and so seeing a very similar section being removed in this patch caused me to raise a false positive. My apologies for the noise (I tend to test patches in

[Freeciv-Dev] [patch #4931] pf_fuel_map_iterate(): adjustements

2014-07-11 Thread pepeto
Additional Item Attachment, patch #4931 (project freeciv): File name: pf_fuel_map_adjust.patch Size:13 KB ___ Reply to this item at: http://gna.org/patch/?4931 ___ Message posté

[Freeciv-Dev] [patch #4931] pf_fuel_map_iterate(): adjustements

2014-07-11 Thread Emmet Hikory
Follow-up Comment #1, patch #4931 (project freeciv): The removal of the obsolete SINGLE_MOVE cost hack will conflict with the application of patch #4926. Which is the right solution here? ___ Reply to this item at: