Fix: Violation of strict weak ordering in engine value/running cost sorter

This commit is contained in:
Jonathan G Rennison
2023-04-23 11:59:58 +01:00
parent 7a836ee50f
commit dde4761e9d

View File

@@ -418,7 +418,7 @@ static bool GenericEngineValueVsRunningCostSorter(const GUIEngineListItem &a, co
* since we want consistent sorting. * since we want consistent sorting.
* Also if both have no power then sort with reverse of running cost to simulate * Also if both have no power then sort with reverse of running cost to simulate
* previous sorting behaviour for wagons. */ * previous sorting behaviour for wagons. */
if (v_a == 0 && v_b == 0) return !EngineRunningCostSorter(a, b); if (v_a == 0 && v_b == 0) return EngineRunningCostSorter(b, a);
if (v_a == v_b) return EngineNumberSorter(a, b); if (v_a == v_b) return EngineNumberSorter(a, b);
return _engine_sort_direction != (v_a < v_b); return _engine_sort_direction != (v_a < v_b);
} }