[coreboot-gerrit] New patch to review for coreboot: 6741d38 util/cbfstool/lzma/C/LzmaEnc.c: Remove extraneous parentheses

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Thu Nov 13 07:11:28 CET 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7452

-gerrit

commit 6741d38172c1e5466d1a54c38f9400565570608e
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Thu Nov 13 17:09:36 2014 +1100

    util/cbfstool/lzma/C/LzmaEnc.c: Remove extraneous parentheses
    
    Change-Id: I1fc3ad2542a80c74ad21585a189b6ba9c0e1e9aa
    Found-by: Clang
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 util/cbfstool/lzma/C/LzmaEnc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/cbfstool/lzma/C/LzmaEnc.c b/util/cbfstool/lzma/C/LzmaEnc.c
index e7d14c5..0a07a9c 100644
--- a/util/cbfstool/lzma/C/LzmaEnc.c
+++ b/util/cbfstool/lzma/C/LzmaEnc.c
@@ -772,7 +772,7 @@ static uint32_t ReadMatchDistances(struct CLzmaEnc *p, uint32_t *numDistancePair
 
 #define MakeAsChar(p) (p)->backPrev = (uint32_t)(-1); (p)->prev1IsChar = false;
 #define MakeAsShortRep(p) (p)->backPrev = 0; (p)->prev1IsChar = false;
-#define IsShortRep(p) ((p)->backPrev == 0)
+#define IsShortRep(p) (p)->backPrev == 0
 
 static uint32_t GetRepLen1Price(struct CLzmaEnc *p, uint32_t state, uint32_t posState)
 {



More information about the coreboot-gerrit mailing list