[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 #7, bug #22189 (project freeciv): > Sorry for being asleep. Do not worry, you don't! ___ Reply to this item at: ___ Message posté via/par Gna

[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 #5, bug #22189 (project freeciv): (Note also that depth could be calculated using unit_transport_depth().) ___ Reply to this item at: ___ Mess

[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 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

[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-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 th

[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 l

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

2014-06-14 Thread Jacob Nevins
URL: 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 - Norma