Mercurial > hg > nginx

changeset 7136:d409ab1e8e4d stable-1.12

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Added missing "fall through" comments (ticket #1259). Found by gcc7 (-Wimplicit-fallthrough).
author Maxim Dounin <mdounin@mdounin.ru>
date Thu, 27 Apr 2017 16:57:18 +0300
parents 5e1112ac82d5
children 0e05b35beebf
files src/core/ngx_murmurhash.c src/http/ngx_http_parse.c src/os/unix/ngx_process.c
diffstat 3 files changed, 6 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/core/ngx_murmurhash.c	Fri Oct 13 20:23:57 2017 +0300
+++ b/src/core/ngx_murmurhash.c	Thu Apr 27 16:57:18 2017 +0300
@@ -35,8 +35,10 @@
     switch (len) {
     case 3:
         h ^= data[2] << 16;
+        /* fall through */
     case 2:
         h ^= data[1] << 8;
+        /* fall through */
     case 1:
         h ^= data[0];
         h *= 0x5bd1e995;
--- a/src/http/ngx_http_parse.c	Fri Oct 13 20:23:57 2017 +0300
+++ b/src/http/ngx_http_parse.c	Thu Apr 27 16:57:18 2017 +0300
@@ -1390,6 +1390,7 @@
                 goto done;
             case '+':
                 r->plus_in_uri = 1;
+                /* fall through */
             default:
                 state = sw_usual;
                 *u++ = ch;
@@ -1431,6 +1432,7 @@
                 goto done;
             case '+':
                 r->plus_in_uri = 1;
+                /* fall through */
             default:
                 state = sw_usual;
                 *u++ = ch;
@@ -1478,6 +1480,7 @@
                 goto done;
             case '+':
                 r->plus_in_uri = 1;
+                /* fall through */
             default:
                 state = sw_usual;
                 *u++ = ch;
--- a/src/os/unix/ngx_process.c	Fri Oct 13 20:23:57 2017 +0300
+++ b/src/os/unix/ngx_process.c	Thu Apr 27 16:57:18 2017 +0300
@@ -405,6 +405,7 @@
                 break;
             }
             ngx_debug_quit = 1;
+            /* fall through */
         case ngx_signal_value(NGX_SHUTDOWN_SIGNAL):
             ngx_quit = 1;
             action = ", shutting down";