]> git.ipfire.org Git - thirdparty/ulogd2.git/commitdiff
jhash: add "fall through" comments to switch cases
authorJeremy Sowden <jeremy@azazel.net>
Tue, 30 Nov 2021 10:55:29 +0000 (10:55 +0000)
committerPablo Neira Ayuso <pablo@netfilter.org>
Tue, 30 Nov 2021 19:55:17 +0000 (20:55 +0100)
gcc warns about undocumented fall-throughs in switches.  In this case,
the fall-throughs are intended, so add commnts to indicate this to the
compiler.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
include/ulogd/jhash.h

index 38b87801a795a763d4c64963cc9605c113bd6c04..e5ca287e7e5899536425740448ad960ab9b377df 100644 (file)
@@ -66,18 +66,18 @@ static inline u32 jhash(const void *key, u32 length, u32 initval)
 
        c += length;
        switch (len) {
-       case 11: c += ((u32)k[10]<<24);
-       case 10: c += ((u32)k[9]<<16);
-       case 9 : c += ((u32)k[8]<<8);
-       case 8 : b += ((u32)k[7]<<24);
-       case 7 : b += ((u32)k[6]<<16);
-       case 6 : b += ((u32)k[5]<<8);
-       case 5 : b += k[4];
-       case 4 : a += ((u32)k[3]<<24);
-       case 3 : a += ((u32)k[2]<<16);
-       case 2 : a += ((u32)k[1]<<8);
-       case 1 : a += k[0];
-       };
+       case 11: c += ((u32)k[10]<<24); // fall through
+       case 10: c += ((u32)k[9]<<16);  // fall through
+       case 9 : c += ((u32)k[8]<<8);   // fall through
+       case 8 : b += ((u32)k[7]<<24);  // fall through
+       case 7 : b += ((u32)k[6]<<16);  // fall through
+       case 6 : b += ((u32)k[5]<<8);   // fall through
+       case 5 : b += k[4];             // fall through
+       case 4 : a += ((u32)k[3]<<24);  // fall through
+       case 3 : a += ((u32)k[2]<<16);  // fall through
+       case 2 : a += ((u32)k[1]<<8);   // fall through
+       case 1 : a += k[0];             // fall through
+       }
 
        __jhash_mix(a,b,c);