https://git.reactos.org/?p=reactos.git;a=commitdiff;h=1ac9e484657f12b548f8424b938ad6a5275ffc29

commit 1ac9e484657f12b548f8424b938ad6a5275ffc29
Author:     Timo Kreuzer <[email protected]>
AuthorDate: Sun Apr 28 18:46:40 2019 +0200
Commit:     HervĂ© Poussineau <[email protected]>
CommitDate: Mon May 27 13:22:40 2019 +0200

    [MINIZIP] Add curly braces to if-statement and fix indentation
    
    Fixes GCC 8 warning:
    sdk/lib/3rdparty/zlib/contrib/minizip/zip.c:521:5: error: this 'for' clause 
does not guard... [-Werror=misleading-indentation]
         for (i=(int)uReadSize-3; (i--)>0;)
         ^~~
    sdk/lib/3rdparty/zlib/contrib/minizip/zip.c:529:7: note: ...this statement, 
but the latter is misleadingly indented as if it were guarded by the 'for'
           if (uPosFound!=0)
           ^~
---
 sdk/lib/3rdparty/zlib/contrib/minizip/zip.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/sdk/lib/3rdparty/zlib/contrib/minizip/zip.c 
b/sdk/lib/3rdparty/zlib/contrib/minizip/zip.c
index d5ab730933..198c1a8940 100644
--- a/sdk/lib/3rdparty/zlib/contrib/minizip/zip.c
+++ b/sdk/lib/3rdparty/zlib/contrib/minizip/zip.c
@@ -519,15 +519,17 @@ local ZPOS64_T zip64local_SearchCentralDir(const 
zlib_filefunc64_32_def* pzlib_f
       break;
 
     for (i=(int)uReadSize-3; (i--)>0;)
+    {
       if (((*(buf+i))==0x50) && ((*(buf+i+1))==0x4b) &&
         ((*(buf+i+2))==0x05) && ((*(buf+i+3))==0x06))
       {
         uPosFound = uReadPos+i;
         break;
       }
+    }
 
-      if (uPosFound!=0)
-        break;
+    if (uPosFound!=0)
+      break;
   }
   TRYFREE(buf);
   return uPosFound;

Reply via email to