Mercurial > hg > openttd
changeset 13030:12fa346326b0 draft
(svn r17528) -Codechange: use QSortT instead of qsort for sorting FiosItems
author | rubidium <rubidium@openttd.org> |
---|---|
date | Sun, 13 Sep 2009 17:39:33 +0000 |
parents | 0175b6426e2c |
children | 2ac3b388fb4c |
files | src/fios.cpp src/fios.h src/misc_gui.cpp |
diffstat | 3 files changed, 10 insertions(+), 12 deletions(-) [+] |
line wrap: on
line diff
--- a/src/fios.cpp +++ b/src/fios.cpp @@ -13,6 +13,7 @@ #include "stdafx.h" #include "openttd.h" +#include "core/sort_func.hpp" #include "fios.h" #include "fileio_func.h" #include "tar_type.h" @@ -43,15 +44,13 @@ extern void GetOldSaveGameName(const char *file, char *title, const char *last); /** - * Compare two FiosItem's. Used with qsort when sorting the file list. - * @param a A pointer to the first FiosItem to compare. - * @param b A pointer to the second FiosItem to compare. + * Compare two FiosItem's. Used with sort when sorting the file list. + * @param da A pointer to the first FiosItem to compare. + * @param db A pointer to the second FiosItem to compare. * @return -1, 0 or 1, depending on how the two items should be sorted. */ -int CDECL compare_FiosItems(const void *a, const void *b) +int CDECL CompareFiosItems(const FiosItem *da, const FiosItem *db) { - const FiosItem *da = (const FiosItem *)a; - const FiosItem *db = (const FiosItem *)b; int r = 0; if ((_savegame_sort_order & SORT_BY_NAME) == 0 && da->mtime != db->mtime) { @@ -309,7 +308,7 @@ { byte order = _savegame_sort_order; _savegame_sort_order = SORT_BY_NAME | SORT_ASCENDING; - qsort(_fios_items.Begin(), _fios_items.Length(), sizeof(FiosItem), compare_FiosItems); + QSortT(_fios_items.Begin(), _fios_items.Length(), CompareFiosItems); _savegame_sort_order = order; } @@ -324,7 +323,7 @@ scanner.Scan(NULL, subdir, true, true); } - qsort(_fios_items.Get(sort_start), _fios_items.Length() - sort_start, sizeof(FiosItem), compare_FiosItems); + QSortT(_fios_items.Get(sort_start), _fios_items.Length() - sort_start, CompareFiosItems); /* Show drives */ if (mode != SLD_NEW_GAME) FiosGetDrives();
--- a/src/fios.h +++ b/src/fios.h @@ -115,6 +115,6 @@ /* Determines type of savegame (or tells it is not a savegame) */ FiosType FiosGetSavegameListCallback(SaveLoadDialogMode mode, const char *file, const char *ext, char *title, const char *last); -int CDECL compare_FiosItems(const void *a, const void *b); +int CDECL CompareFiosItems(const FiosItem *a, const FiosItem *b); #endif /* FIOS_H */
--- a/src/misc_gui.cpp +++ b/src/misc_gui.cpp @@ -40,6 +40,7 @@ #include "newgrf_cargo.h" #include "tilehighlight_func.h" #include "querystring_gui.h" +#include "core/sort_func.hpp" #include "table/strings.h" @@ -1746,9 +1747,7 @@ } uint s_amount = _fios_items.Length() - sort_start - sort_end; - if (s_amount > 0) { - qsort(_fios_items.Get(sort_start), s_amount, sizeof(FiosItem), compare_FiosItems); - } + QSortT(_fios_items.Get(sort_start), s_amount, CompareFiosItems); } extern void StartupEngines();