cpp/openttd-patchpack/source Changeset - r27202:5a7c474a171e

Changeset - r27202:5a7c474a171e

Parent rev.

Child rev.

[Not reviewed]

master

0 10 0

Tyler Trahan - 19 months ago 2023-04-30 08:23:05
tyler@tylertrahan.com

Codechange: Pass more std::string to StringFilter::AddLine() (#10743)

10 files changed with 28 insertions and 24 deletions:

src/industry.h

src/network/network_content_gui.cpp

src/network/network_gui.cpp

src/object_gui.cpp

src/rail_gui.cpp

src/road_gui.cpp

src/stringfilter.cpp

src/stringfilter_type.h

src/strings.cpp

src/town.h

0 comments (0 inline, 0 general)

src/industry.h

➞

Show inline comments

@@ @@ -184,10 +184,10 @@ struct Industry : IndustryPool::PoolItem @@
 		memset(&counts, 0, sizeof(counts));
+	}
-	inline const char *GetCachedName() const
+	inline const std::string &GetCachedName() const
+	{
 		if (this->cached_name.empty()) this->FillCachedName();
-		return this->cached_name.c_str();
 		return this->cached_name;
+	}
 private:

src/network/network_content_gui.cpp

➞

Show inline comments

@@ @@ -467,9 +467,9 @@ class NetworkContentListWindow : public @@
 	static bool CDECL TagNameFilter(const ContentInfo * const *a, ContentListFilterData &filter)
+	{
 		filter.string_filter.ResetState();
-		for (auto &tag : (*a)->tags) filter.string_filter.AddLine(tag.c_str());
 		for (auto &tag : (*a)->tags) filter.string_filter.AddLine(tag);
-		filter.string_filter.AddLine((*a)->name.c_str());
 		filter.string_filter.AddLine((*a)->name);
 		return filter.string_filter.GetState();
+	}

src/network/network_gui.cpp

➞

Show inline comments

@@ @@ -379,7 +379,7 @@ protected: @@
 		assert((*item) != nullptr);
 		sf.ResetState();
-		sf.AddLine((*item)->info.server_name.c_str());
 		sf.AddLine((*item)->info.server_name);
 		return sf.GetState();
+	}

src/object_gui.cpp

➞

Show inline comments

@@ @@ -142,11 +142,9 @@ public: @@
 	static bool CDECL TagNameFilter(ObjectClassID const *oc, StringFilter &filter)
+	{
 		ObjectClass *objclass = ObjectClass::Get(*oc);
 		char buffer[DRAW_STRING_BUFFER];
 		GetString(buffer, objclass->name, lastof(buffer));
 		filter.ResetState();
-		filter.AddLine(buffer);
+		filter.AddLine(GetString(objclass->name));
 		return filter.GetState();
+	}

src/rail_gui.cpp

➞

Show inline comments

@@ @@ -1046,11 +1046,8 @@ public: @@
 	/** Filter station classes by class name. */
 	static bool CDECL TagNameFilter(StationClassID const * sc, StringFilter &filter)
+	{
 		char buffer[DRAW_STRING_BUFFER];
 		GetString(buffer, StationClass::Get(*sc)->name, lastof(buffer));
 		filter.ResetState();
-		filter.AddLine(buffer);
+		filter.AddLine(GetString(StationClass::Get(*sc)->name));
 		return filter.GetState();
+	}

src/road_gui.cpp

➞

Show inline comments

@@ @@ -1249,11 +1249,8 @@ public: @@
 	/** Filter classes by class name. */
 	static bool CDECL TagNameFilter(RoadStopClassID const *sc, StringFilter &filter)
+	{
 		char buffer[DRAW_STRING_BUFFER];
 		GetString(buffer, RoadStopClass::Get(*sc)->name, lastof(buffer));
 		filter.ResetState();
-		filter.AddLine(buffer);
+		filter.AddLine(GetString(RoadStopClass::Get(*sc)->name));
 		return filter.GetState();
+	}

src/stringfilter.cpp

➞

Show inline comments

@@ @@ -125,9 +125,20 @@ void StringFilter::AddLine(const char *s @@
+ *
  * @param str Another line from the item.
  */
 void StringFilter::AddLine(const std::string &str)
+{
 	AddLine(str.c_str());
+}
 /**
  * Pass another text line from the current item to the filter.
+ *
  * You can call this multiple times for a single item, if the filter shall apply to multiple things.
  * Before processing the next item you have to call ResetState().
+ *
  * @param str Another line from the item.
  */
 void StringFilter::AddLine(StringID str)
+{
 	char buffer[DRAW_STRING_BUFFER];
 	GetString(buffer, str, lastof(buffer));
 	AddLine(buffer);
 	AddLine(GetString(str));
+}

src/stringfilter_type.h

➞

Show inline comments

@@ @@ -60,6 +60,7 @@ public: @@
 	void ResetState();
 	void AddLine(const char *str);
 	void AddLine(const std::string &str);
 	void AddLine(StringID str);
 	/**

src/strings.cpp

➞

Show inline comments

@@ @@ -1458,7 +1458,7 @@ static char *FormatString(char *buff, co @@
 				static bool use_cache = true;
 				if (use_cache) { // Use cached version if first call
 					AutoRestoreBackup cache_backup(use_cache, false);
 					buff = strecpy(buff, i->GetCachedName(), last);
+					buff = strecpy(buff, i->GetCachedName().c_str(), last);
 				} else if (_scan_for_gender_data) {
 					/* Gender is defined by the industry type.
 					 * STR_FORMAT_INDUSTRY_NAME may have the town first, so it would result in the gender of the town name */
@@ @@ -1541,7 +1541,7 @@ static char *FormatString(char *buff, co @@
 				static bool use_cache = true;
 				if (use_cache) { // Use cached version if first call
 					AutoRestoreBackup cache_backup(use_cache, false);
 					buff = strecpy(buff, t->GetCachedName(), last);
+					buff = strecpy(buff, t->GetCachedName().c_str(), last);
 				} else if (!t->name.empty()) {
 					int64 args_array[] = {(int64)(size_t)t->name.c_str()};
 					StringParameters tmp_params(args_array);

src/town.h

➞

Show inline comments

@@ @@ -125,11 +125,11 @@ struct Town : TownPool::PoolItem<&_town_ @@
 	void UpdateVirtCoord();
-	inline const char *GetCachedName() const
+	inline const std::string &GetCachedName() const
+	{
-		if (!this->name.empty()) return this->name.c_str();
 		if (!this->name.empty()) return this->name;
 		if (this->cached_name.empty()) this->FillCachedName();
-		return this->cached_name.c_str();
 		return this->cached_name;
+	}
 	static inline Town *GetByTile(TileIndex tile)

0 comments (0 inline, 0 general)