Re: [Patch] BaseTools LzmaCompress: Fix GCC warning misleading-indentation

[Ard Biesheuvel <ard.biesheuvel@linaro.org>]

On 9 October 2018 at 09:06, Liming Gao <liming.gao@intel.com> wrote:
> GCC 6 or above reports the warning misleading-indentation.
> This patch fixes it.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Liming Gao <liming.gao@intel.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> ---
>  BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c b/BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c
> index bebe664..e281716 100644
> --- a/BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c
> +++ b/BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c
> @@ -1464,7 +1464,9 @@ static unsigned GetOptimum(CLzmaEnc *p, UInt32 position)
>          unsigned limit = p->numFastBytes + 1;
>          if (limit > numAvailFull)
>            limit = numAvailFull;
> -        for (len = 3; len < limit && data[len] == data2[len]; len++);
> +        for (len = 3; len < limit && data[len] == data2[len]; len++)
> +        {
> +        }
>
>          {
>            unsigned state2 = kLiteralNextStates[state];

Thanks Liming.

Don't we need this change in all copies of the LZMA SDK?