cpp/openttd-patchpack/source Changeset - r23629:cfb4492bdba9

Changeset - r23629:cfb4492bdba9

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

glx - 5 years ago 2019-04-12 15:55:35
glx@openttd.org

Codechange: use std::vector for _sorted_railtypes

2 files changed with 8 insertions and 10 deletions:

src/rail.h

src/rail_cmd.cpp

0 comments (0 inline, 0 general)

src/rail.h

➞

Show inline comments

@@ @@ -464,14 +464,13 @@ void ResetRailTypes(); @@
 void InitRailTypes();
 RailType AllocateRailType(RailTypeLabel label);
 extern RailType _sorted_railtypes[RAILTYPE_END];
 extern uint8 _sorted_railtypes_size;
 extern std::vector<RailType> _sorted_railtypes;
 extern RailTypes _railtypes_hidden_mask;
 /**
  * Loop header for iterating over railtypes, sorted by sortorder.
  * @param var Railtype.
  */
-#define FOR_ALL_SORTED_RAILTYPES(var) for (uint8 index = 0; index < _sorted_railtypes_size && (var = _sorted_railtypes[index], true) ; index++)
+#define FOR_ALL_SORTED_RAILTYPES(var) for (uint8 index = 0; index < _sorted_railtypes.size() && (var = _sorted_railtypes[index], true) ; index++)
 #endif /* RAIL_H */

src/rail_cmd.cpp

➞

Show inline comments

@@ @@ -44,8 +44,7 @@ @@
 typedef std::vector<Train *> TrainList;
 RailtypeInfo _railtypes[RAILTYPE_END];
 RailType _sorted_railtypes[RAILTYPE_END];
 uint8 _sorted_railtypes_size;
 std::vector<RailType> _sorted_railtypes;
 RailTypes _railtypes_hidden_mask;
 /** Enum holding the signal offset in the sprite sheet according to the side it is representing. */
@@ @@ -130,9 +129,9 @@ void ResolveRailTypeGUISprites(RailtypeI @@
  * @param second The railtype to compare.
  * @return True iff the first should be sorted before the second.
  */
-static int CDECL CompareRailTypes(const RailType *first, const RailType *second)
+static bool CompareRailTypes(const RailType &first, const RailType &second)
+{
-	return GetRailTypeInfo(*first)->sorting_order - GetRailTypeInfo(*second)->sorting_order;
+	return GetRailTypeInfo(first)->sorting_order < GetRailTypeInfo(second)->sorting_order;
+}
 /**
@@ @@ -146,13 +145,13 @@ void InitRailTypes() @@
 		if (HasBit(rti->flags, RTF_HIDDEN)) SetBit(_railtypes_hidden_mask, rt);
+	}
-	_sorted_railtypes_size = 0;
+	_sorted_railtypes.clear();
 	for (RailType rt = RAILTYPE_BEGIN; rt != RAILTYPE_END; rt++) {
 		if (_railtypes[rt].label != 0 && !HasBit(_railtypes_hidden_mask, rt)) {
-			_sorted_railtypes[_sorted_railtypes_size++] = rt;
+			_sorted_railtypes.push_back(rt);
+		}
+	}
-	QSortT(_sorted_railtypes, _sorted_railtypes_size, CompareRailTypes);
+	std::sort(_sorted_railtypes.begin(), _sorted_railtypes.end(), CompareRailTypes);
+}
 /**

0 comments (0 inline, 0 general)