Re: [patch 6/8] MdeModulePkg/TerminalDxe: Initialize variable after declaration

["Tian, Feng" <feng.tian@intel.com>]

Reviewed-by: Feng Tian <feng.tian@Intel.com>

Thanks
Feng

-----Original Message-----
From: Bi, Dandan 
Sent: Monday, December 19, 2016 3:20 PM
To: edk2-devel@lists.01.org
Cc: Tian, Feng <feng.tian@intel.com>
Subject: [patch 6/8] MdeModulePkg/TerminalDxe: Initialize variable after declaration

Cc: Feng Tian <feng.tian@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
---
 MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c b/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c
index 9625f4d..c9b4ffc 100644
--- a/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c
+++ b/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c
@@ -186,10 +186,11 @@ TerminalConOutOutputString (
   UTF8_CHAR                   Utf8Char;
   CHAR8                       GraphicChar;
   CHAR8                       AsciiChar;
   EFI_STATUS                  Status;
   UINT8                       ValidBytes;
+  CHAR8                       CrLfStr[2];
   //
   //  flag used to indicate whether condition happens which will cause
   //  return EFI_WARN_UNKNOWN_GLYPH
   //
   BOOLEAN                     Warning;
@@ -324,11 +325,12 @@ TerminalConOutOutputString (
           // the next character, but the driver thinks it has wrapped
           // already.  Print CR LF to synchronize the terminal with
           // the driver, but only if we're not in the middle of
           // printing an escape sequence.
           //
-          CHAR8 CrLfStr[] = {'\r', '\n'};
+          CrLfStr[0] = '\r';
+          CrLfStr[1] = '\n';
 
           Length = sizeof(CrLfStr);
 
           Status = TerminalDevice->SerialIo->Write (
                                                 TerminalDevice->SerialIo,
-- 
1.9.5.msysgit.1