From c9522fb740200ccef6230cec452c48efb31e5394 Mon Sep 17 00:00:00 2001 From: Mattes D Date: Thu, 11 May 2023 22:05:17 +0200 Subject: Removed all Printf-family functions from StringUtils. Replaced them with fmt::format calls, including changes to the format strings. Also changed the format strings to use FMT_STRING, so that the format is checked compile-time against the arguments. Also fixed code-style violations already present in the code. --- src/Entities/Player.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/Entities/Player.cpp') diff --git a/src/Entities/Player.cpp b/src/Entities/Player.cpp index 9f4fcb971..ba1059b7c 100644 --- a/src/Entities/Player.cpp +++ b/src/Entities/Player.cpp @@ -1682,7 +1682,7 @@ AString cPlayer::GetSuffix(void) const -AString cPlayer::GetPlayerListName(void) const +AString cPlayer::GetPlayerListName() const { const AString & Color = GetColor(); @@ -1692,7 +1692,7 @@ AString cPlayer::GetPlayerListName(void) const } else if ((GetName().length() <= 14) && !Color.empty()) { - return Printf("%s%s", Color.c_str(), GetName().c_str()); + return fmt::format(FMT_STRING("{}{}"), Color, GetName()); } else { @@ -3092,8 +3092,8 @@ void cPlayer::OnRemoveFromWorld(cWorld & a_World) if (!cRoot::Get()->GetPluginManager()->CallHookPlayerDestroyed(*this)) { - cRoot::Get()->BroadcastChatLeave(Printf("%s has left the game", GetName().c_str())); - LOGINFO("Player %s has left the game", GetName().c_str()); + cRoot::Get()->BroadcastChatLeave(fmt::format(FMT_STRING("{} has left the game"), GetName())); + LOGINFO("Player %s has left the game", GetName()); } // Remove ourself from everyone's lists: -- cgit v1.2.3