Move printing double newline character ('\n\n') from the beginning of ACPI table checksum validation message to the end of the raw binary data dump.
This way acpiview table dump looks similar regardless of whether Table Checksum is validated or not. Signed-off-by: Krzysztof Koch <krzysztof.k...@arm.com> --- Changes can be seen at: https://github.com/KrzysztofKoch1/edk2/commit/db22c06817971220875e98ec891ed73359f823de Notes: v1: - Unify acpiview output with/without checksum reporting [Krzysztof] ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c b/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c index b5965507b498b9fa9bc4d3124b2285f509004c4f..f9dbbd35449e74fdf81f09e627920b10583a9816 100644 --- a/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c +++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c @@ -1,7 +1,7 @@ /** @file ACPI parser - Copyright (c) 2016 - 2018, ARM Limited. All rights reserved. + Copyright (c) 2016 - 2019, ARM Limited. All rights reserved. SPDX-License-Identifier: BSD-2-Clause-Patent **/ @@ -139,7 +139,7 @@ VerifyChecksum ( ((OriginalAttribute&(BIT4|BIT5|BIT6))>>4)) ); } - Print (L"\n\nTable Checksum : OK\n\n"); + Print (L"Table Checksum : OK\n\n"); } else { IncrementErrorCount (); if (GetColourHighlighting ()) { @@ -149,7 +149,7 @@ VerifyChecksum ( ((OriginalAttribute&(BIT4|BIT5|BIT6))>>4)) ); } - Print (L"\n\nTable Checksum : FAILED (0x%X)\n\n", Checksum); + Print (L"Table Checksum : FAILED (0x%X)\n\n", Checksum); } if (GetColourHighlighting ()) { gST->ConOut->SetAttribute (gST->ConOut, OriginalAttribute); @@ -219,7 +219,7 @@ DumpRaw ( // Print ASCII data for the final line. AsciiBuffer[AsciiBufferIndex] = '\0'; - Print (L" %a", AsciiBuffer); + Print (L" %a\n\n", AsciiBuffer); } /** -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#42998): https://edk2.groups.io/g/devel/message/42998 Mute This Topic: https://groups.io/mt/32240030/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-