From 532391db12e1c803657738d69a6f312c2b3cb506 Mon Sep 17 00:00:00 2001 From: bjarni Date: Sat, 2 Jun 2007 15:47:38 +0000 Subject: (svn r10025) -Fix: [build windows] sorting planes for capacity didn't check mail capacity. Now mail capacity is used if passenger capacity is the same The sort will still resort to EngineID if both mail and passenger capacities are the same --- src/build_vehicle_gui.cpp | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/src/build_vehicle_gui.cpp b/src/build_vehicle_gui.cpp index bfc896810..517ad7b52 100644 --- a/src/build_vehicle_gui.cpp +++ b/src/build_vehicle_gui.cpp @@ -332,13 +332,20 @@ static int CDECL AircraftEngineRunningCostSorter(const void *a, const void *b) static int CDECL AircraftEngineCargoSorter(const void *a, const void *b) { - const int va = AircraftVehInfo(*(const EngineID*)a)->passenger_capacity; - const int vb = AircraftVehInfo(*(const EngineID*)b)->passenger_capacity; - const int r = va - vb; + int va = AircraftVehInfo(*(const EngineID*)a)->passenger_capacity; + int vb = AircraftVehInfo(*(const EngineID*)b)->passenger_capacity; + int r = va - vb; if (r == 0) { - /* Use EngineID to sort instead since we want consistent sorting */ - return EngineNumberSorter(a, b); + /* The planes has the same passenger capacity. Check mail capacity instead */ + va = AircraftVehInfo(*(const EngineID*)a)->mail_capacity; + vb = AircraftVehInfo(*(const EngineID*)b)->mail_capacity; + r = va - vb; + + if (r == 0) { + /* Use EngineID to sort instead since we want consistent sorting */ + return EngineNumberSorter(a, b); + } } return _internal_sort_order ? -r : r; } -- cgit v1.2.3-54-g00ecf