[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-16 Thread Jacob Nevins
Update of bug #22189 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-15 Thread pepeto
Follow-up Comment #3, bug #22189 (project freeciv): Do we really need to sort the full list of candidates. If I understand correctly, we only need to know the best candidate. Also, I suggest to test if the unit is better candidate than our previous one in the iteration.

[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-15 Thread Jacob Nevins
Update of bug #22189 (project freeciv): Status: Ready For Test = In Progress ___ Follow-up Comment #4: Do we really need to sort the full list of candidates. [...] I suggest to test if the unit is

[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-15 Thread Jacob Nevins
Update of bug #22189 (project freeciv): Status: In Progress = Ready For Test ___ Additional Item Attachment: File name: trunk-S2_5-prefer-unnested-transports-bis.patch Size:2 KB

[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-15 Thread pepeto
Follow-up Comment #5, bug #22189 (project freeciv): (Note also that depth could be calculated using unit_transport_depth().) ___ Reply to this item at: http://gna.org/bugs/?22189 ___

[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-15 Thread Jacob Nevins
Follow-up Comment #6, bug #22189 (project freeciv): Note also that depth could be calculated using unit_transport_depth(). ...and clearly should, in case we ever decide to cache it. Third attempt attached. Sorry for being asleep. (file #21024)

[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-15 Thread pepeto
Follow-up Comment #7, bug #22189 (project freeciv): Sorry for being asleep. Do not worry, you don't! ___ Reply to this item at: http://gna.org/bugs/?22189 ___ Message posté via/par Gna!

[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-14 Thread Jacob Nevins
URL: http://gna.org/bugs/?22189 Summary: Prefer to load units onto less nested transporters Project: Freeciv Submitted by: jtn Submitted on: Sat 14 Jun 2014 17:18:04 BST Category: None Severity: 3 - Normal

[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-14 Thread Marko Lindqvist
Follow-up Comment #1, bug #22189 (project freeciv): I propose that transporter_for_unit() ranks candidate transports by nesting depth, and a transport with the least nesting depth is chosen. That's the most logical solution. In theory (I don't propose really doing it) we could even in the

[Freeciv-Dev] [bug #22189] Prefer to load units onto less nested transporters

2014-06-14 Thread Jacob Nevins
Update of bug #22189 (project freeciv): Status:None = Ready For Test Assigned to:None = jtn ___ Follow-up Comment #2: Since I couldn't