Re: [PATCH v2] BaseTools/VolInfo: Show encapsulation sections

["Bob Feng" <bob.c.feng@intel.com>]

Reviewed-by: Bob Feng <bob.c.feng@intel.com>

-----Original Message-----
From: Konstantin Aladyshev <aladyshev22@gmail.com> 
Sent: Tuesday, July 19, 2022 8:29 PM
To: devel@edk2.groups.io
Cc: Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com>; Konstantin Aladyshev <aladyshev22@gmail.com>
Subject: [PATCH v2] BaseTools/VolInfo: Show encapsulation sections

Currently there is no labels for start and end of the encapsulation sections. Therefore it is not possible to see where the encapsulation section ends and another section starts.
Add labels for start and end of encapsulation sections to fix the issue.

Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
---
 BaseTools/Source/C/VolInfo/VolInfo.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/BaseTools/Source/C/VolInfo/VolInfo.c b/BaseTools/Source/C/VolInfo/VolInfo.c
index c1f81f2dcb..ce1775f7fd 100644
--- a/BaseTools/Source/C/VolInfo/VolInfo.c
+++ b/BaseTools/Source/C/VolInfo/VolInfo.c
@@ -1903,7 +1903,9 @@ Returns:
         return EFI_SECTION_ERROR;       } +      printf ("/------------ Encapsulation section start -----------------\\\n");       Status = ParseSection (UncompressedBuffer, UncompressedLength);+      printf ("\\------------ Encapsulation section end -------------------/\n");        if (CompressionType == EFI_STANDARD_COMPRESSION) {         //@@ -2022,6 +2024,7 @@ Returns:
           return EFI_SECTION_ERROR;         } +        printf ("/------------ Encapsulation section start -----------------\\\n");         Status = ParseSection (                   ToolOutputBuffer,                   ToolOutputLength@@ -2030,6 +2033,7 @@ Returns:
           Error (NULL, 0, 0003, "parse of decoded GUIDED section failed", NULL);           return EFI_SECTION_ERROR;         }+        printf ("\\------------ Encapsulation section end -------------------/\n");        //       // Check for CRC32 sections which we can handle internally if needed.@@ -2042,6 +2046,7 @@ Returns:
         //         // CRC32 guided section         //+        printf ("/------------ Encapsulation section start -----------------\\\n");         Status = ParseSection (                   SectionBuffer + DataOffset,                   BufferLength - DataOffset@@ -2050,6 +2055,7 @@ Returns:
           Error (NULL, 0, 0003, "parse of CRC32 GUIDED section failed", NULL);           return EFI_SECTION_ERROR;         }+        printf ("\\------------ Encapsulation section end -------------------/\n");       } else {         //         // We don't know how to parse it now.-- 
2.25.1