Reviewed-by: Liming Gao <[email protected]>

> -----Original Message-----
> From: edk2-devel [mailto:[email protected]] On Behalf Of
> Yonghong Zhu
> Sent: Monday, February 29, 2016 11:24 AM
> To: [email protected]
> Subject: [edk2] [Patch] BaseTools: fix LzmaCompress VS2013 make failure
> 
> when make BaseTools by VS2013, LzmaEnc.c report warning C4127:
> conditional expression is constant, so this patch fix this issue.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Yonghong Zhu <[email protected]>
> ---
>  BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c
> b/BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c
> index c6717ee..297db9d 100644
> --- a/BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c
> +++ b/BaseTools/Source/C/LzmaCompress/Sdk/C/LzmaEnc.c
> @@ -1364,12 +1364,13 @@ static UInt32 GetOptimum(CLzmaEnc *p, UInt32
> position, UInt32 *backRes)
>        while (--lenTest >= 2);
>        lenTest = lenTestTemp;
> 
>        if (repIndex == 0)
>          startLen = lenTest + 1;
> -
> +      #ifndef _MSC_VER
>        if (1 /* _maxMode */)
> +      #endif
>          {
>            UInt32 lenTest2 = lenTest + 1;
>            UInt32 limit = lenTest2 + p->numFastBytes;
>            UInt32 nextRepMatchPrice;
>            if (limit > numAvailFull)
> --
> 2.6.1.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> [email protected]
> https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to