diff --git a/src/base_media_func.h b/src/base_media_func.h --- a/src/base_media_func.h +++ b/src/base_media_func.h @@ -22,8 +22,8 @@ #define fetch_metadata(name) \ item = metadata->GetItem(name, false); \ if (item == nullptr || !item->value.has_value() || item->value->empty()) { \ - DEBUG(grf, 0, "Base " SET_TYPE "set detail loading: %s field missing.", name); \ - DEBUG(grf, 0, " Is %s readable for the user running OpenTTD?", full_filename); \ + Debug(grf, 0, "Base " SET_TYPE "set detail loading: {} field missing.", name); \ + Debug(grf, 0, " Is {} readable for the user running OpenTTD?", full_filename); \ return false; \ } @@ -74,7 +74,7 @@ bool BaseSetGetItem(BaseSet::file_names[i], false); if (item == nullptr || (!item->value.has_value() && !allow_empty_filename)) { - DEBUG(grf, 0, "No " SET_TYPE " file for: %s (in %s)", BaseSet::file_names[i], full_filename); + Debug(grf, 0, "No " SET_TYPE " file for: {} (in {})", BaseSet::file_names[i], full_filename); return false; } @@ -92,7 +92,7 @@ bool BaseSetGetItem(filename, false); if (item == nullptr || !item->value.has_value()) { - DEBUG(grf, 0, "No MD5 checksum specified for: %s (in %s)", filename, full_filename); + Debug(grf, 0, "No MD5 checksum specified for: {} (in {})", filename, full_filename); return false; } const char *c = item->value->c_str(); @@ -105,7 +105,7 @@ bool BaseSetGetItem(filename, false); if (item == nullptr) item = origin->GetItem("default", false); if (item == nullptr || !item->value.has_value()) { - DEBUG(grf, 1, "No origin warning message specified for: %s", filename); + Debug(grf, 1, "No origin warning message specified for: {}", filename); file->missing_warning = stredup(""); } else { file->missing_warning = stredup(item->value->c_str()); @@ -136,12 +136,12 @@ bool BaseSetfound_files++; break; case MD5File::CR_NO_FILE: - DEBUG(grf, 1, "The file %s specified in %s is missing", filename, full_filename); + Debug(grf, 1, "The file {} specified in {} is missing", filename, full_filename); break; } } @@ -153,7 +153,7 @@ template bool BaseMedia::AddFile(const std::string &filename, size_t basepath_length, const std::string &tar_filename) { bool ret = false; - DEBUG(grf, 1, "Checking %s for base " SET_TYPE " set", filename.c_str()); + Debug(grf, 1, "Checking {} for base " SET_TYPE " set", filename); Tbase_set *set = new Tbase_set(); IniFile *ini = new IniFile(); @@ -179,7 +179,7 @@ bool BaseMedia::AddFile(const /* The more complete set takes precedence over the version number. */ if ((duplicate->valid_files == set->valid_files && duplicate->version >= set->version) || duplicate->valid_files > set->valid_files) { - DEBUG(grf, 1, "Not adding %s (%i) as base " SET_TYPE " set (duplicate, %s)", set->name.c_str(), set->version, + Debug(grf, 1, "Not adding {} ({}) as base " SET_TYPE " set (duplicate, {})", set->name, set->version, duplicate->valid_files > set->valid_files ? "less valid files" : "lower version"); set->next = BaseMedia::duplicate_sets; BaseMedia::duplicate_sets = set; @@ -195,7 +195,7 @@ bool BaseMedia::AddFile(const * version number until a new game is started which isn't a big problem */ if (BaseMedia::used_set == duplicate) BaseMedia::used_set = set; - DEBUG(grf, 1, "Removing %s (%i) as base " SET_TYPE " set (duplicate, %s)", duplicate->name.c_str(), duplicate->version, + Debug(grf, 1, "Removing {} ({}) as base " SET_TYPE " set (duplicate, {})", duplicate->name, duplicate->version, duplicate->valid_files < set->valid_files ? "less valid files" : "lower version"); duplicate->next = BaseMedia::duplicate_sets; BaseMedia::duplicate_sets = duplicate; @@ -209,7 +209,7 @@ bool BaseMedia::AddFile(const ret = true; } if (ret) { - DEBUG(grf, 1, "Adding %s (%i) as base " SET_TYPE " set", set->name.c_str(), set->version); + Debug(grf, 1, "Adding {} ({}) as base " SET_TYPE " set", set->name, set->version); } } else { delete set;