From f97707a28a412b4cb24d25c43d0de09ae46248ec Mon Sep 17 00:00:00 2001
From: Gene Walters <genewalt@amazon.com>
Date: Tue, 7 Dec 2021 16:14:08 -0800
Subject: [PATCH] Removing hand-written /n from all AZLOG_ now that we properly
 add \n automatically

Signed-off-by: Gene Walters <genewalt@amazon.com>
---
 Code/Framework/AzCore/AzCore/Console/Console.cpp   | 14 +++++++-------
 .../Code/Source/BarrierInputClient.cpp             |  8 ++++----
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/Code/Framework/AzCore/AzCore/Console/Console.cpp b/Code/Framework/AzCore/AzCore/Console/Console.cpp
index d6781e1370..ff7a08e127 100644
--- a/Code/Framework/AzCore/AzCore/Console/Console.cpp
+++ b/Code/Framework/AzCore/AzCore/Console/Console.cpp
@@ -243,7 +243,7 @@ namespace AZ
 
             if (StringFunc::StartsWith(curr->m_name, command, false))
             {
-                AZLOG_INFO("- %s : %s\n", curr->m_name, curr->m_desc);
+                AZLOG_INFO("- %s : %s", curr->m_name, curr->m_desc);
 
                 if (commandSubset.size() < MaxConsoleCommandPlusArgsLength)
                 {
@@ -433,29 +433,29 @@ namespace AZ
             {
                 if ((curr->GetFlags() & requiredSet) != requiredSet)
                 {
-                    AZLOG_WARN("%s failed required set flag check\n", curr->m_name);
+                    AZLOG_WARN("%s failed required set flag check", curr->m_name);
                     continue;
                 }
 
                 if ((curr->GetFlags() & requiredClear) != ConsoleFunctorFlags::Null)
                 {
-                    AZLOG_WARN("%s failed required clear flag check\n", curr->m_name);
+                    AZLOG_WARN("%s failed required clear flag check", curr->m_name);
                     continue;
                 }
 
                 if ((curr->GetFlags() & ConsoleFunctorFlags::IsCheat) != ConsoleFunctorFlags::Null)
                 {
-                    AZLOG_WARN("%s is marked as a cheat\n", curr->m_name);
+                    AZLOG_WARN("%s is marked as a cheat", curr->m_name);
                 }
 
                 if ((curr->GetFlags() & ConsoleFunctorFlags::IsDeprecated) != ConsoleFunctorFlags::Null)
                 {
-                    AZLOG_WARN("%s is marked as deprecated\n", curr->m_name);
+                    AZLOG_WARN("%s is marked as deprecated", curr->m_name);
                 }
 
                 if ((curr->GetFlags() & ConsoleFunctorFlags::NeedsReload) != ConsoleFunctorFlags::Null)
                 {
-                    AZLOG_WARN("Changes to %s will only take effect after level reload\n", curr->m_name);
+                    AZLOG_WARN("Changes to %s will only take effect after level reload", curr->m_name);
                 }
 
                 // Letting this intentionally fall-through, since in editor we can register common variables multiple times
@@ -468,7 +468,7 @@ namespace AZ
                     {
                         CVarFixedString value;
                         curr->GetValue(value);
-                        AZLOG_INFO("> %s : %s\n", curr->GetName(), value.empty() ? "<empty>" : value.c_str());
+                        AZLOG_INFO("> %s : %s", curr->GetName(), value.empty() ? "<empty>" : value.c_str());
                     }
                     flags = curr->GetFlags();
                 }
diff --git a/Gems/BarrierInput/Code/Source/BarrierInputClient.cpp b/Gems/BarrierInput/Code/Source/BarrierInputClient.cpp
index ec4cbcfbd1..f564a0c1bc 100644
--- a/Gems/BarrierInput/Code/Source/BarrierInputClient.cpp
+++ b/Gems/BarrierInput/Code/Source/BarrierInputClient.cpp
@@ -254,7 +254,7 @@ namespace BarrierInput
 
     static bool barrierBye([[maybe_unused]]BarrierClient* pContext, [[maybe_unused]]int* pArgs, [[maybe_unused]]Stream* pStream, [[maybe_unused]]int streamLeft)
     {
-        AZLOG_INFO("BarrierClient: Server said bye. Disconnecting\n");
+        AZLOG_INFO("BarrierClient: Server said bye. Disconnecting");
         return false;
     }
 
@@ -284,7 +284,7 @@ namespace BarrierInput
             const char* packetStart = stream.GetData();
             if (packetLength > streamLength)
             {
-                AZLOG_INFO("BarrierClient: Packet overruns buffer (Packet Length: %d Buffer Length: %d), probably lots of data on clipboard?\n", packetLength, streamLength);
+                AZLOG_INFO("BarrierClient: Packet overruns buffer (Packet Length: %d Buffer Length: %d), probably lots of data on clipboard?", packetLength, streamLength);
                 return false;
             }
 
@@ -377,7 +377,7 @@ namespace BarrierInput
             const int lengthReceived = AZ::AzSock::Recv(m_socket, stream.GetBuffer(), stream.GetBufferSize(), 0);
             if (lengthReceived <= 0)
             {
-                AZLOG_INFO("BarrierClient: Receive failed, reconnecting.\n");
+                AZLOG_INFO("BarrierClient: Receive failed, reconnecting.");
                 connected = false;
                 continue;
             }
@@ -386,7 +386,7 @@ namespace BarrierInput
             stream.SetLength(lengthReceived);
             if (!ProcessPackets(this, stream))
             {
-                AZLOG_INFO("BarrierClient: Packet processing failed, reconnecting.\n");
+                AZLOG_INFO("BarrierClient: Packet processing failed, reconnecting.");
                 connected = false;
                 continue;
             }