Patrick Georgi submitted this change.

View Change

util/cbfstool/lzma: Make clang-11+'s indentation checker happy

Newest clang compilers warn about "misleading indentation", and because
warnings-are-errors in our builds, that breaks the build.

The lzma code base is vendored in, so we might just have to update it,
but that's a bigger effort than just removing a couple of spaces (the
coding style of the file is horrible, but I will only change it as much
as the compilers ask for).

BUG=chromium:1039526

Change-Id: I6b9d7a760380081af996ea5412d7e3e688048bfd
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38637
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Idwer Vollering <vidwer@gmail.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
---
M util/cbfstool/lzma/C/LzmaEnc.c
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/util/cbfstool/lzma/C/LzmaEnc.c b/util/cbfstool/lzma/C/LzmaEnc.c
index e7d14c5..f2a8320 100644
--- a/util/cbfstool/lzma/C/LzmaEnc.c
+++ b/util/cbfstool/lzma/C/LzmaEnc.c
@@ -1246,7 +1246,7 @@
         startLen = lenTest + 1;
 
       /* if (_maxMode) */
-        {
+      {
           uint32_t lenTest2 = lenTest + 1;
           uint32_t limit = lenTest2 + p->numFastuint8_ts;
           uint32_t nextRepMatchPrice;
@@ -1290,7 +1290,7 @@
               }
             }
           }
-        }
+      }
     }
     }
     /* for (uint32_t lenTest = 2; lenTest <= newLen; lenTest++) */

To view, visit change 38637. To unsubscribe, or for help writing mail filters, visit settings.