diff --git a/projects/openttd_vs80.vcproj b/projects/openttd_vs80.vcproj index 03062dc074..a57c1f4dd0 100644 --- a/projects/openttd_vs80.vcproj +++ b/projects/openttd_vs80.vcproj @@ -999,6 +999,10 @@ RelativePath=".\..\src\engine_func.h" > + + diff --git a/projects/openttd_vs90.vcproj b/projects/openttd_vs90.vcproj index 89fbb1c9c7..9b2dd1c485 100644 --- a/projects/openttd_vs90.vcproj +++ b/projects/openttd_vs90.vcproj @@ -996,6 +996,10 @@ RelativePath=".\..\src\engine_func.h" > + + diff --git a/source.list b/source.list index 7e80cea7a2..c9efdd2bda 100644 --- a/source.list +++ b/source.list @@ -175,6 +175,7 @@ elrail_func.h core/endian_func.hpp engine_base.h engine_func.h +engine_gui.h engine_type.h core/enum_type.hpp fiber.hpp diff --git a/src/autoreplace_gui.cpp b/src/autoreplace_gui.cpp index 35ca3a9242..88d5a9db82 100644 --- a/src/autoreplace_gui.cpp +++ b/src/autoreplace_gui.cpp @@ -22,6 +22,7 @@ #include "engine_func.h" #include "engine_base.h" #include "window_gui.h" +#include "engine_gui.h" #include "table/sprites.h" #include "table/strings.h" diff --git a/src/build_vehicle_gui.cpp b/src/build_vehicle_gui.cpp index b7c030494b..715938a9d6 100644 --- a/src/build_vehicle_gui.cpp +++ b/src/build_vehicle_gui.cpp @@ -29,6 +29,7 @@ #include "widgets/dropdown_func.h" #include "string_func.h" #include "window_gui.h" +#include "engine_gui.h" #include "table/sprites.h" #include "table/strings.h" diff --git a/src/engine.cpp b/src/engine.cpp index 558fcd8e04..c3d1f51851 100644 --- a/src/engine.cpp +++ b/src/engine.cpp @@ -115,34 +115,6 @@ Engine::~Engine() free(this->name); } -/** Sort all items using qsort() and given 'CompareItems' function - * @param el list to be sorted - * @param compare function for evaluation of the quicksort - */ -void EngList_Sort(EngineList *el, EngList_SortTypeFunction compare) -{ - size_t size = el->size(); - /* out-of-bounds access at the next line for size == 0 (even with operator[] at some systems) - * generally, do not sort if there are less than 2 items */ - if (size < 2) return; - qsort(&((*el)[0]), size, sizeof(EngineID), compare); // MorphOS doesn't know vector::at(int) ... -} - -/** Sort selected range of items (on indices @ ) - * @param el list to be sorted - * @param compare function for evaluation of the quicksort - * @param begin start of sorting - * @param num_items count of items to be sorted - */ -void EngList_SortPartial(EngineList *el, EngList_SortTypeFunction compare, uint begin, uint num_items) -{ - assert(begin <= (uint)el->size()); - assert(begin + num_items <= (uint)el->size()); - if (num_items < 2) return; - qsort(&((*el)[begin]), num_items, sizeof(EngineID), compare); -} - - /** Sets cached values in Player::num_vehicles and Group::num_vehicles */ void SetCachedEngineCounts() diff --git a/src/engine_func.h b/src/engine_func.h index 2e68831aca..2e5049948b 100644 --- a/src/engine_func.h +++ b/src/engine_func.h @@ -29,8 +29,4 @@ bool IsEngineBuildable(EngineID engine, VehicleType type, PlayerID player); CargoID GetEngineCargoType(EngineID engine); void SetCachedEngineCounts(); -typedef int CDECL EngList_SortTypeFunction(const void*, const void*); ///< argument type for EngList_Sort() -void EngList_Sort(EngineList *el, EngList_SortTypeFunction compare); ///< qsort of the engine list -void EngList_SortPartial(EngineList *el, EngList_SortTypeFunction compare, uint begin, uint num_items); ///< qsort of specified portion of the engine list - #endif /* ENGINE_H */ diff --git a/src/engine_gui.cpp b/src/engine_gui.cpp index a67d23fcb4..bd48d778a7 100644 --- a/src/engine_gui.cpp +++ b/src/engine_gui.cpp @@ -15,6 +15,7 @@ #include "variables.h" #include "newgrf_engine.h" #include "strings_func.h" +#include "engine_gui.h" #include "table/strings.h" #include "table/sprites.h" @@ -190,3 +191,32 @@ void DrawNewsNewVehicleAvail(Window *w, const NewsItem *ni) GfxFillRect(25, 56, w->width - 56, 112, PALETTE_TO_STRUCT_GREY | (1 << USE_COLORTABLE)); dei->info_proc(engine, w->width >> 1, 129, w->width - 52); } + + +/** Sort all items using qsort() and given 'CompareItems' function + * @param el list to be sorted + * @param compare function for evaluation of the quicksort + */ +void EngList_Sort(EngineList *el, EngList_SortTypeFunction compare) +{ + size_t size = el->size(); + /* out-of-bounds access at the next line for size == 0 (even with operator[] at some systems) + * generally, do not sort if there are less than 2 items */ + if (size < 2) return; + qsort(&((*el)[0]), size, sizeof(EngineID), compare); // MorphOS doesn't know vector::at(int) ... +} + +/** Sort selected range of items (on indices @ ) + * @param el list to be sorted + * @param compare function for evaluation of the quicksort + * @param begin start of sorting + * @param num_items count of items to be sorted + */ +void EngList_SortPartial(EngineList *el, EngList_SortTypeFunction compare, uint begin, uint num_items) +{ + assert(begin <= (uint)el->size()); + assert(begin + num_items <= (uint)el->size()); + if (num_items < 2) return; + qsort(&((*el)[begin]), num_items, sizeof(EngineID), compare); +} + diff --git a/src/engine_gui.h b/src/engine_gui.h new file mode 100644 index 0000000000..12957da925 --- /dev/null +++ b/src/engine_gui.h @@ -0,0 +1,16 @@ +/* $Id$ */ + +/** @file engine_gui.h Engine GUI functions, used by build_vehicle_gui and autoreplace_gui */ + +#ifndef ENGINE_GUI_H +#define ENGINE_GUI_H + +#include + +typedef std::vector EngineList; + +typedef int CDECL EngList_SortTypeFunction(const void*, const void*); ///< argument type for EngList_Sort() +void EngList_Sort(EngineList *el, EngList_SortTypeFunction compare); ///< qsort of the engine list +void EngList_SortPartial(EngineList *el, EngList_SortTypeFunction compare, uint begin, uint num_items); ///< qsort of specified portion of the engine list + +#endif /* ENGINE_GUI_H */ diff --git a/src/engine_type.h b/src/engine_type.h index affa722ce3..66a4956075 100644 --- a/src/engine_type.h +++ b/src/engine_type.h @@ -14,11 +14,8 @@ #include "player_type.h" #include "strings_type.h" -#include - typedef uint16 EngineID; typedef uint16 EngineRenewID; -typedef std::vector EngineList; ///< engine list type struct Engine;