From c3aa3f140cd8625a81f34849cc7b963a779f596e Mon Sep 17 00:00:00 2001 From: frosch Date: Sat, 3 May 2025 18:40:37 +0200 Subject: [PATCH] Codechange: Use more std::string_view. (#14197) --- src/saveload/saveload.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/saveload/saveload.cpp b/src/saveload/saveload.cpp index 0fc8d60b27..2bf80a0a02 100644 --- a/src/saveload/saveload.cpp +++ b/src/saveload/saveload.cpp @@ -2846,7 +2846,7 @@ static const SaveLoadFormat _saveload_formats[] = { * @param full_name Name of the savegame format. If empty it picks the first available one * @return Pair containing reference to SaveLoadFormat struct giving all characteristics of this type of savegame, and a compression level to use. */ -static std::pair GetSavegameFormat(const std::string &full_name) +static std::pair GetSavegameFormat(std::string_view full_name) { /* Find default savegame format, the highest one with which files can be written. */ auto it = std::find_if(std::rbegin(_saveload_formats), std::rend(_saveload_formats), [](const auto &slf) { return slf.init_write != nullptr; }); @@ -2858,12 +2858,12 @@ static std::pair GetSavegameFormat(const std::s /* Get the ":..." of the compression level out of the way */ size_t separator = full_name.find(':'); bool has_comp_level = separator != std::string::npos; - const std::string name(full_name, 0, has_comp_level ? separator : full_name.size()); + std::string_view name = has_comp_level ? full_name.substr(0, separator) : full_name; for (const auto &slf : _saveload_formats) { if (slf.init_write != nullptr && name.compare(slf.name) == 0) { if (has_comp_level) { - auto complevel = std::string_view(full_name).substr(separator + 1); + auto complevel = full_name.substr(separator + 1); /* Get the level and determine whether all went fine. */ auto level = ParseInteger(complevel);