[Freeciv-Dev] Re: (PR#11349) 'u' on a transport should unload all units inside

["Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=11349 >

Benoit Hudson wrote:
> <URL: http://rt.freeciv.org/Ticket/Display.html?id=11349 >
> 
> On Sun, Dec 05, 2004 at 09:52:02PM -0800, Jason Short wrote:
> 
>>Well, they don't have to be.  With the current code there's no way
>>they'd both be active at the same time.  It's separate because of
>>historical reasons (just the way it was made) and because recursive
>>transportation hasn't been entirely ruled out in the future.
> 
> 
> I wonder: how hard would it be to make 'u' first try to unload, and if
> it can't, then try to unload all?  Shift-u could remain unload all. 
> That would deal with the recursive transport problem too.

Just change the request_unit_unload() call in menu.c to:

   if (transporter_capacity(punit) > 0) {
     request_unit_unload_all(punit);
   } else {
     request_unit_unload(punit);
   }

-jason