Re: [RFC 0/5] EmbeddedPkg/PrePiLib: rework FfsProcessSection

["Ard Biesheuvel" <ard.biesheuvel@arm.com>]

On 7/1/20 10:01 PM, Leif Lindholm wrote:
> https://bugzilla.tianocore.org/show_bug.cgi?id=2820 describes a build
> failure caused by misanalysis by the compiler, but the problematic code
> was pretty grotty, so here's an attempt at cleaning it up.
> 
> This set can also be accessed at:
> https://github.com/leiflindholm/edk2/tree/embedded-fwvol-cleanup
> 
> Note: this code is only build tested.
> 
> Leif Lindholm (5):
>    EmbeddedPkg/PrePiLib: style cleanup in FwVol.c
>    EmbeddedPkg/PrePiLib: drop else if after return
>    EmbeddedPkg/PrePiLib: refactor IS_SECTION2() handling
>    EmbeddedPkg/PrePiLib: drop spurious re-init of CompressedData
>    EmbeddedPkg/PrePiLib: break section extraction info into helper
>      function
> 

For the series:

Tested-by: Ard Biesheuvel <ard.biesheuvel@arm.com>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@arm.com>


>   EmbeddedPkg/Library/PrePiLib/FwVol.c | 214 ++++++++++++++++-----------
>   1 file changed, 125 insertions(+), 89 deletions(-)
>