cpp/openttd-patchpack/source Changeset - r28481:4df4e2dbb670

Changeset - r28481:4df4e2dbb670

Parent rev.

Child rev.

[Not reviewed]

master

0 4 0

Patric Stout - 3 months ago 2024-01-16 21:10:34
truebrain@openttd.org

Codechange: minor bits and pieces related to fmt::format() (#11806)

- Don't make run-time formatting what can be done compile-time.
- Be explicit about run-time formatting.
- Fix datetime printing.

4 files changed with 8 insertions and 4 deletions:

src/debug.cpp

src/gamelog.cpp

src/network/core/address.cpp

src/newgrf_profiling.cpp

0 comments (0 inline, 0 general)

src/debug.cpp

➞

Show inline comments

@@ @@ -95,7 +95,7 @@ void DumpDebugFacilityNames(std::back_in @@
 		} else {
 			fmt::format_to(output_iterator, ", ");
+		}
 		fmt::format_to(output_iterator, i->name);
+		fmt::format_to(output_iterator, "{}", i->name);
 		written = true;
+	}
 	if (written) {

src/gamelog.cpp

➞

Show inline comments

@@ @@ -242,7 +242,11 @@ void Gamelog::Print(std::function<void(c @@
+{
 	/* A NewGRF got removed from the game, either manually or by it missing when loading the game. */
 	auto gm = grf_names.find(this->grfid);
 	fmt::format_to(output_iterator, action_type == GLAT_LOAD ? "Missing NewGRF: " : "Removed NewGRF: ");
 	if (action_type == GLAT_LOAD) {
 		fmt::format_to(output_iterator, "Missing NewGRF: ");
 	} else {
 		fmt::format_to(output_iterator, "Removed NewGRF: ");
+	}
 	AddGrfInfo(output_iterator, this->grfid, nullptr, gm != grf_names.end() ? gm->second.gc : nullptr);
 	if (gm == grf_names.end()) {
 		fmt::format_to(output_iterator, ". Gamelog inconsistency: GrfID was never added!");

src/network/core/address.cpp

➞

Show inline comments

@@ @@ -93,7 +93,7 @@ static const char *GetAddressFormatStrin @@
  */
 std::string NetworkAddress::GetAddressAsString(bool with_family)
+{
 	return fmt::format(GetAddressFormatString(this->GetAddress()->ss_family, with_family), this->GetHostname(), this->GetPort());
+	return fmt::format(fmt::runtime(GetAddressFormatString(this->GetAddress()->ss_family, with_family)), this->GetHostname(), this->GetPort());
+}
 /**

src/newgrf_profiling.cpp

➞

Show inline comments

@@ @@ -131,7 +131,7 @@ void NewGRFProfiler::Abort() @@
  */
 std::string NewGRFProfiler::GetOutputFilename() const
+{
 	return fmt::format("{}grfprofile-{%Y%m%d-%H%M}-{:08X}.csv", FiosGetScreenshotDir(), fmt::localtime(time(nullptr)), BSWAP32(this->grffile->grfid));
+	return fmt::format("{}grfprofile-{:%Y%m%d-%H%M}-{:08X}.csv", FiosGetScreenshotDir(), fmt::localtime(time(nullptr)), BSWAP32(this->grffile->grfid));
+}
 /* static */ uint32_t NewGRFProfiler::FinishAll()

0 comments (0 inline, 0 general)