Core/Logging: Improved generated code for log statements (length of text is computed at compile time)

3 files changed, 9 insertions, 9 deletions

diff --git a/src/server/game/Maps/GridMap.cpp b/src/server/game/Maps/GridMap.cpp
index 0d5f2587675..d61594736c3 100644
--- a/src/server/game/Maps/GridMap.cpp
+++ b/src/server/game/Maps/GridMap.cpp
@@ -108,8 +108,8 @@ GridMap::LoadResult GridMap::loadData(char const* filename)
         return LoadResult::Ok;
     }
 
-    TC_LOG_ERROR("maps", "Map file '{}' is from an incompatible map version (%.*s v{}), %.*s v{} is expected. Please pull your source, recompile tools and recreate maps using the updated mapextractor, then replace your old map files with new files. If you still have problems search on forum for error TCE00018.",
-        filename, 4, header.mapMagic.data(), header.versionMagic, 4, MapMagic.data(), MapVersionMagic);
+    TC_LOG_ERROR("maps", "Map file '{}' is from an incompatible map version ({} v{}), {} v{} is expected. Please pull your source, recompile tools and recreate maps using the updated mapextractor, then replace your old map files with new files. If you still have problems search on forum for error TCE00018.",
+        filename, std::string_view(header.mapMagic.data(), 4), header.versionMagic, std::string_view(MapMagic.data(), 4), MapVersionMagic);
     fclose(in);
     return LoadResult::InvalidFile;
 }
diff --git a/src/server/game/Maps/TerrainMgr.cpp b/src/server/game/Maps/TerrainMgr.cpp
index 189108a74d1..b2faa91a944 100644
--- a/src/server/game/Maps/TerrainMgr.cpp
+++ b/src/server/game/Maps/TerrainMgr.cpp
@@ -96,8 +96,8 @@ bool TerrainInfo::ExistMap(uint32 mapid, int32 gx, int32 gy, bool log /*= true*/
             if (header.mapMagic != MapMagic || header.versionMagic != MapVersionMagic)
             {
                 if (log)
-                    TC_LOG_ERROR("maps", "Map file '{}' is from an incompatible map version (%.*s v{}), %.*s v{} is expected. Please pull your source, recompile tools and recreate maps using the updated mapextractor, then replace your old map files with new files. If you still have problems search on forum for error TCE00018.",
-                        fileName, 4, header.mapMagic.data(), header.versionMagic, 4, MapMagic.data(), MapVersionMagic);
+                    TC_LOG_ERROR("maps", "Map file '{}' is from an incompatible map version ({} v{}), {} v{} is expected. Please pull your source, recompile tools and recreate maps using the updated mapextractor, then replace your old map files with new files. If you still have problems search on forum for error TCE00018.",
+                        fileName, std::string_view(header.mapMagic.data(), 4), header.versionMagic, std::string_view(MapMagic.data(), 4), MapVersionMagic);
             }
             else
                 ret = true;
diff --git a/src/server/scripts/World/chat_log.cpp b/src/server/scripts/World/chat_log.cpp
index cec8cf6191f..74178d7333f 100644
--- a/src/server/scripts/World/chat_log.cpp
+++ b/src/server/scripts/World/chat_log.cpp
@@ -22,11 +22,11 @@
 #include "Log.h"
 #include "Player.h"
 
-#define TC_LOG_CHAT(TYPE, ...)                           \
-    if (lang != LANG_ADDON && lang != LANG_ADDON_LOGGED) \
-        TC_LOG_DEBUG("chat.log." TYPE, __VA_ARGS__);     \
-    else                                                 \
-        TC_LOG_DEBUG("chat.log.addon." TYPE, __VA_ARGS__);
+#define TC_LOG_CHAT(TYPE, MESSAGE, ...)                           \
+    if (lang != LANG_ADDON && lang != LANG_ADDON_LOGGED)          \
+        TC_LOG_DEBUG("chat.log." TYPE, MESSAGE, __VA_ARGS__);     \
+    else                                                          \
+        TC_LOG_DEBUG("chat.log.addon." TYPE, MESSAGE, __VA_ARGS__);
 
 class ChatLogScript : public PlayerScript
 {