Unit.getGoodsCapacity() is hardwired to throw an exception, and this
is happening in BR#3571591.  The code in DefaultTransferHandler that
calls getGoodsCapacity is a minor convenience and could be dropped,
but I am surprised by the need for the exception.  Is this just
something we forgot to fix when Units became GoodsLocations, or am I
missing something important?

    public int getGoodsCapacity() {
        throw new RuntimeException("Do not call this method, unless we 
implement spoilage.");
    }

Cheers,
Mike Pope

Attachment: signature.asc
Description: This is a digitally signed message part.

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Freecol-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/freecol-developers

Reply via email to