cpp/openttd-patchpack/source Changeset - r27398:53658248e31b

Changeset - r27398:53658248e31b

Parent rev.

Child rev.

[Not reviewed]

master

0 15 0

Rubidium - 16 months ago 2023-05-19 21:22:30
rubidium@openttd.org

Codechange: use fmt::format_to to format the help message

15 files changed with 81 insertions and 100 deletions:

src/ai/ai.hpp

src/ai/ai_core.cpp

src/base_media_base.h

src/base_media_func.h

src/blitter/factory.hpp

src/console_cmds.cpp

src/debug.cpp

src/debug.h

src/driver.cpp

src/driver.h

src/game/game.hpp

src/game/game_core.cpp

src/openttd.cpp

src/script/script_scanner.cpp

src/script/script_scanner.hpp

0 comments (0 inline, 0 general)

src/ai/ai.hpp

➞

Show inline comments

@@ @@ -112,9 +112,9 @@ public: @@
 	static void Save(CompanyID company);
 	/** Wrapper function for AIScanner::GetAIConsoleList */
-	static std::string GetConsoleList(bool newest_only = false);
+	static void GetConsoleList(std::back_insert_iterator<std::string> &output_iterator, bool newest_only);
 	/** Wrapper function for AIScanner::GetAIConsoleLibraryList */
-	static std::string GetConsoleLibraryList();
+	static void GetConsoleLibraryList(std::back_insert_iterator<std::string> &output_iterator);
 	/** Wrapper function for AIScanner::GetAIInfoList */
 	static const ScriptInfoList *GetInfoList();
 	/** Wrapper function for AIScanner::GetUniqueAIInfoList */

src/ai/ai_core.cpp

➞

Show inline comments

@@ @@ -289,14 +289,14 @@ @@
+	}
+}
-/* static */ std::string AI::GetConsoleList(bool newest_only)
+/* static */ void AI::GetConsoleList(std::back_insert_iterator<std::string> &output_iterator, bool newest_only)
+{
-	return AI::scanner_info->GetConsoleList(newest_only);
+	AI::scanner_info->GetConsoleList(output_iterator, newest_only);
+}
-/* static */ std::string AI::GetConsoleLibraryList()
+/* static */ void AI::GetConsoleLibraryList(std::back_insert_iterator<std::string> &output_iterator)
+{
-	 return AI::scanner_library->GetConsoleList(true);
+	 AI::scanner_library->GetConsoleList(output_iterator, true);
+}
 /* static */ const ScriptInfoList *AI::GetInfoList()

src/base_media_base.h

➞

Show inline comments

@@ @@ -192,7 +192,7 @@ public: @@
 	static Tbase_set *GetAvailableSets();
 	static bool SetSet(const std::string &name);
-	static char *GetSetsList(char *p, const char *last);
+	static void GetSetsList(std::back_insert_iterator<std::string> &output_iterator);
 	static int GetNumSets();
 	static int GetIndexOfUsedSet();
 	static const Tbase_set *GetSet(int index);

src/base_media_func.h

➞

Show inline comments

@@ @@ -248,31 +248,27 @@ template <class Tbase_set> @@
 /**
  * Returns a list with the sets.
  * @param p    where to print to
  * @param last the last character to print to
  * @return the last printed character
  * @param output_iterator The iterator to write the string to.
  */
 template <class Tbase_set>
-/* static */ char *BaseMedia<Tbase_set>::GetSetsList(char *p, const char *last)
+/* static */ void BaseMedia<Tbase_set>::GetSetsList(std::back_insert_iterator<std::string> &output_iterator)
+{
-	p += seprintf(p, last, "List of " SET_TYPE " sets:\n");
+	fmt::format_to(output_iterator, "List of " SET_TYPE " sets:\n");
 	for (const Tbase_set *s = BaseMedia<Tbase_set>::available_sets; s != nullptr; s = s->next) {
-		p += seprintf(p, last, "%18s: %s", s->name.c_str(), s->GetDescription({}));
+		fmt::format_to(output_iterator, "{:>18}: {}", s->name, s->GetDescription({}));
 		int invalid = s->GetNumInvalid();
 		if (invalid != 0) {
 			int missing = s->GetNumMissing();
 			if (missing == 0) {
-				p += seprintf(p, last, " (%i corrupt file%s)\n", invalid, invalid == 1 ? "" : "s");
+				fmt::format_to(output_iterator, " ({} corrupt file{})\n", invalid, invalid == 1 ? "" : "s");
 			} else {
-				p += seprintf(p, last, " (unusable: %i missing file%s)\n", missing, missing == 1 ? "" : "s");
+				fmt::format_to(output_iterator, " (unusable: {} missing file{})\n", missing, missing == 1 ? "" : "s");
+			}
 		} else {
-			p += seprintf(p, last, "\n");
+			fmt::format_to(output_iterator, "\n");
+		}
+	}
 	p += seprintf(p, last, "\n");
 	return p;
 	fmt::format_to(output_iterator, "\n");
+}
 #include "network/core/tcp_content_type.h"
@@ @@ -378,7 +374,7 @@ template <class Tbase_set> @@
 	template bool repl_type::AddFile(const std::string &filename, size_t pathlength, const std::string &tar_filename); \
 	template bool repl_type::HasSet(const struct ContentInfo *ci, bool md5sum); \
 	template bool repl_type::SetSet(const std::string &name); \
-	template char *repl_type::GetSetsList(char *p, const char *last); \
+	template void repl_type::GetSetsList(std::back_insert_iterator<std::string> &output_iterator); \
 	template int repl_type::GetNumSets(); \
 	template int repl_type::GetIndexOfUsedSet(); \
 	template const set_type *repl_type::GetSet(int index); \

src/blitter/factory.hpp

➞

Show inline comments

@@ @@ -146,16 +146,14 @@ public: @@
 	 * @param last The last element of the buffer.
 	 * @return p The location till where we filled the buffer.
 	 */
-	static char *GetBlittersInfo(char *p, const char *last)
+	static void GetBlittersInfo(std::back_insert_iterator<std::string> &output_iterator)
+	{
-		p += seprintf(p, last, "List of blitters:\n");
+		fmt::format_to(output_iterator, "List of blitters:\n");
 		for (auto &it : GetBlitters()) {
 			BlitterFactory *b = it.second;
-			p += seprintf(p, last, "%18s: %s\n", b->name.c_str(), b->GetDescription().c_str());
+			fmt::format_to(output_iterator, "{:>18}: {}\n", b->name, b->GetDescription());
+		}
 		p += seprintf(p, last, "\n");
 		return p;
 		fmt::format_to(output_iterator, "\n");
+	}
 	/**

src/console_cmds.cpp

➞

Show inline comments

@@ @@ -1180,6 +1180,17 @@ static void PrintLineByLine(const std::s @@
+	}
+}
 template <typename F, typename ... Args>
 bool PrintList(F list_function, Args... args)
+{
 	std::string output_str;
 	auto inserter = std::back_inserter(output_str);
 	list_function(inserter, args...);
 	PrintLineByLine(output_str);
 	return true;
+}
 DEF_CONSOLE_CMD(ConListAILibs)
+{
 	if (argc == 0) {
@@ @@ -1187,10 +1198,7 @@ DEF_CONSOLE_CMD(ConListAILibs) @@
 		return true;
+	}
 	const std::string output_str = AI::GetConsoleLibraryList();
 	PrintLineByLine(output_str);
 	return true;
 	return PrintList(AI::GetConsoleLibraryList);
+}
 DEF_CONSOLE_CMD(ConListAI)
@@ @@ -1200,10 +1208,7 @@ DEF_CONSOLE_CMD(ConListAI) @@
 		return true;
+	}
 	const std::string output_str = AI::GetConsoleList();
 	PrintLineByLine(output_str);
 	return true;
 	return PrintList(AI::GetConsoleList, false);
+}
 DEF_CONSOLE_CMD(ConListGameLibs)
@@ @@ -1213,10 +1218,7 @@ DEF_CONSOLE_CMD(ConListGameLibs) @@
 		return true;
+	}
 	const std::string output_str = Game::GetConsoleLibraryList();
 	PrintLineByLine(output_str);
 	return true;
 	return PrintList(Game::GetConsoleLibraryList);
+}
 DEF_CONSOLE_CMD(ConListGame)
@@ @@ -1226,10 +1228,7 @@ DEF_CONSOLE_CMD(ConListGame) @@
 		return true;
+	}
 	const std::string output_str = Game::GetConsoleList();
 	PrintLineByLine(output_str);
 	return true;
 	return PrintList(Game::GetConsoleList, false);
+}
 DEF_CONSOLE_CMD(ConStartAI)

src/debug.cpp

➞

Show inline comments

@@ @@ -85,27 +85,23 @@ struct DebugLevel { @@
 /**
  * Dump the available debug facility names in the help text.
  * @param buf Start address for storing the output.
  * @param last Last valid address for storing the output.
  * @return Next free position in the output.
  * @param output_iterator The iterator to write the string to.
  */
-char *DumpDebugFacilityNames(char *buf, char *last)
+void DumpDebugFacilityNames(std::back_insert_iterator<std::string> &output_iterator)
+{
-	size_t length = 0;
+	bool written = false;
 	for (const DebugLevel *i = debug_level; i != endof(debug_level); ++i) {
 		if (length == 0) {
 			buf = strecpy(buf, "List of debug facility names:\n", last);
 		if (!written) {
 			fmt::format_to(output_iterator, "List of debug facility names:\n");
 		} else {
 			buf = strecpy(buf, ", ", last);
 			length += 2;
 			fmt::format_to(output_iterator, ", ");
+		}
 		buf = strecpy(buf, i->name, last);
 		length += strlen(i->name);
 		fmt::format_to(output_iterator, i->name);
 		written = true;
+	}
 	if (length > 0) {
 		buf = strecpy(buf, "\n\n", last);
 	if (written) {
 		fmt::format_to(output_iterator, "\n\n");
+	}
 	return buf;
+}
 /**

src/debug.h

➞

Show inline comments

@@ @@ -56,7 +56,7 @@ extern int _debug_console_level; @@
 extern int _debug_random_level;
 #endif
-char *DumpDebugFacilityNames(char *buf, char *last);
+void DumpDebugFacilityNames(std::back_insert_iterator<std::string> &output_iterator);
 void SetDebugString(const char *s, void (*error_func)(const std::string &));
 const char *GetDebugString();

src/driver.cpp

➞

Show inline comments

@@ @@ -179,28 +179,24 @@ bool DriverFactoryBase::SelectDriverImpl @@
 /**
  * Build a human readable list of available drivers, grouped by type.
  * @param p The buffer to write to.
  * @param last The last element in the buffer.
  * @return The end of the written buffer.
  * @param output_iterator The iterator to write the string to.
  */
-char *DriverFactoryBase::GetDriversInfo(char *p, const char *last)
+void DriverFactoryBase::GetDriversInfo(std::back_insert_iterator<std::string> &output_iterator)
+{
 	for (Driver::Type type = Driver::DT_BEGIN; type != Driver::DT_END; type++) {
-		p += seprintf(p, last, "List of %s drivers:\n", GetDriverTypeName(type));
+		fmt::format_to(output_iterator, "List of {} drivers:\n", GetDriverTypeName(type));
 		for (int priority = 10; priority >= 0; priority--) {
 			for (auto &it : GetDrivers()) {
 				DriverFactoryBase *d = it.second;
 				if (d->type != type) continue;
 				if (d->priority != priority) continue;
-				p += seprintf(p, last, "%18s: %s\n", d->name, d->GetDescription());
+				fmt::format_to(output_iterator, "{:>18}: {}\n", d->name, d->GetDescription());
+			}
+		}
-		p += seprintf(p, last, "\n");
+		fmt::format_to(output_iterator, "\n");
+	}
 	return p;
+}
 /**

src/driver.h

➞

Show inline comments

@@ @@ -127,7 +127,7 @@ public: @@
+	}
 	static void SelectDriver(const std::string &name, Driver::Type type);
-	static char *GetDriversInfo(char *p, const char *last);
+	static void GetDriversInfo(std::back_insert_iterator<std::string> &output_iterator);
 	/**
 	 * Get a nice description of the driver-class.

src/game/game.hpp

➞

Show inline comments

@@ @@ -83,9 +83,9 @@ public: @@
 	static void Save();
 	/** Wrapper function for GameScanner::GetConsoleList */
-	static std::string GetConsoleList(bool newest_only = false);
+	static void GetConsoleList(std::back_insert_iterator<std::string> &output_iterator, bool newest_only);
 	/** Wrapper function for GameScanner::GetConsoleLibraryList */
-	static std::string GetConsoleLibraryList();
+	static void GetConsoleLibraryList(std::back_insert_iterator<std::string> &output_iterator);
 	/** Wrapper function for GameScanner::GetInfoList */
 	static const ScriptInfoList *GetInfoList();
 	/** Wrapper function for GameScanner::GetUniqueInfoList */

src/game/game_core.cpp

➞

Show inline comments

@@ @@ -219,14 +219,14 @@ @@
+	}
+}
-/* static */ std::string Game::GetConsoleList(bool newest_only)
+/* static */ void Game::GetConsoleList(std::back_insert_iterator<std::string> &output_iterator, bool newest_only)
+{
-	return Game::scanner_info->GetConsoleList(newest_only);
+	Game::scanner_info->GetConsoleList(output_iterator, newest_only);
+}
-/* static */ std::string Game::GetConsoleLibraryList()
+/* static */ void Game::GetConsoleLibraryList(std::back_insert_iterator<std::string> &output_iterator)
+{
-	 return Game::scanner_library->GetConsoleList(true);
+	Game::scanner_library->GetConsoleList(output_iterator, true);
+}
 /* static */ const ScriptInfoList *Game::GetInfoList()

src/openttd.cpp

➞

Show inline comments

@@ @@ -155,11 +155,12 @@ void FatalErrorI(const std::string &str) @@
  */
 static void ShowHelp()
+{
 	char buf[8192];
 	char *p = buf;
 	std::string str;
 	str.reserve(8192);
 	p += seprintf(p, lastof(buf), "OpenTTD %s\n", _openttd_revision);
 	p = strecpy(p,
 	std::back_insert_iterator<std::string> output_iterator = std::back_inserter(str);
 	fmt::format_to(output_iterator, "OpenTTD {}\n", _openttd_revision);
 	str +=
 		"\n"
 		"\n"
 		"Command line options:\n"
@@ @@ -191,46 +192,42 @@ static void ShowHelp() @@
 		"  -q savegame         = Write some information about the savegame and exit\n"
 		"  -Q                  = Don't scan for/load NewGRF files on startup\n"
 		"  -QQ                 = Disable NewGRF scanning/loading entirely\n"
 		"\n",
 		lastof(buf)
 	);
 		"\n";
 	/* List the graphics packs */
-	p = BaseGraphics::GetSetsList(p, lastof(buf));
+	BaseGraphics::GetSetsList(output_iterator);
 	/* List the sounds packs */
-	p = BaseSounds::GetSetsList(p, lastof(buf));
+	BaseSounds::GetSetsList(output_iterator);
 	/* List the music packs */
-	p = BaseMusic::GetSetsList(p, lastof(buf));
+	BaseMusic::GetSetsList(output_iterator);
 	/* List the drivers */
-	p = DriverFactoryBase::GetDriversInfo(p, lastof(buf));
+	DriverFactoryBase::GetDriversInfo(output_iterator);
 	/* List the blitters */
-	p = BlitterFactory::GetBlittersInfo(p, lastof(buf));
+	BlitterFactory::GetBlittersInfo(output_iterator);
 	/* List the debug facilities. */
-	p = DumpDebugFacilityNames(p, lastof(buf));
+	DumpDebugFacilityNames(output_iterator);
 	/* We need to initialize the AI, so it finds the AIs */
 	AI::Initialize();
 	const std::string ai_list = AI::GetConsoleList(true);
 	p = strecpy(p, ai_list.c_str(), lastof(buf));
 	AI::GetConsoleList(output_iterator, true);
 	AI::Uninitialize(true);
 	/* We need to initialize the GameScript, so it finds the GSs */
 	Game::Initialize();
 	const std::string game_list = Game::GetConsoleList(true);
 	p = strecpy(p, game_list.c_str(), lastof(buf));
 	Game::GetConsoleList(output_iterator, true);
 	Game::Uninitialize(true);
 	/* ShowInfo put output to stderr, but version information should go
 	 * to stdout; this is the only exception */
 #if !defined(_WIN32)
-	printf("%s\n", buf);
+	printf("%s\n", str.c_str());
 #else
-	ShowInfoI(buf);
+	ShowInfoI(str);
 #endif
+}

src/script/script_scanner.cpp

➞

Show inline comments

@@ @@ -138,18 +138,15 @@ void ScriptScanner::RegisterScript(Scrip @@
+	}
+}
-std::string ScriptScanner::GetConsoleList(bool newest_only) const
+void ScriptScanner::GetConsoleList(std::back_insert_iterator<std::string> &output_iterator, bool newest_only) const
+{
 	std::string p;
 	p += fmt::format("List of {}:\n", this->GetScannerName());
 	fmt::format_to(output_iterator, "List of {}:\n", this->GetScannerName());
 	const ScriptInfoList &list = newest_only ? this->info_single_list : this->info_list;
 	for (const auto &item : list) {
 		ScriptInfo *i = item.second;
-		p += fmt::format("{:>10} (v{:d}): {}\n", i->GetName(), i->GetVersion(), i->GetDescription());
+		fmt::format_to(output_iterator, "{:>10} (v{:d}): {}\n", i->GetName(), i->GetVersion(), i->GetDescription());
+	}
 	p += "\n";
 	return p;
 	fmt::format_to(output_iterator, "\n");
+}
 /** Helper for creating a MD5sum of all files within of a script. */

src/script/script_scanner.hpp

➞

Show inline comments

@@ @@ -55,8 +55,10 @@ public: @@
 	/**
 	 * Get the list of registered scripts to print on the console.
 	 * @param output_iterator The iterator to write the output to.
 	 * @param newest_only Whether to only show the newest scripts.
 	 */
-	std::string GetConsoleList(bool newest_only) const;
+	void GetConsoleList(std::back_insert_iterator<std::string> &output_iterator, bool newest_only) const;
 	/**
 	 * Check whether we have a script with the exact characteristics as ci.

0 comments (0 inline, 0 general)