changeset 4546:4988fa232629

For the sake of case/switch code readability, 'fall through' comments added.
author Maxim Konovalov <maxim@nginx.com>
date Mon, 19 Mar 2012 14:57:29 +0000
parents ba39af7274ed
children 2577c27cbffd
files src/http/ngx_http_parse.c src/http/ngx_http_upstream.c
diffstat 2 files changed, 5 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/http/ngx_http_parse.c
+++ b/src/http/ngx_http_parse.c
@@ -1153,6 +1153,7 @@ ngx_http_parse_complex_uri(ngx_http_requ
                 break;
             case '+':
                 r->plus_in_uri = 1;
+                /* fall through */
             default:
                 *u++ = ch;
                 break;
--- a/src/http/ngx_http_upstream.c
+++ b/src/http/ngx_http_upstream.c
@@ -3312,6 +3312,8 @@ ngx_http_upstream_process_accel_expires(
         switch (n) {
         case 0:
             u->cacheable = 0;
+            /* fall through */
+
         case NGX_ERROR:
             return NGX_OK;
 
@@ -4460,6 +4462,8 @@ ngx_http_upstream_bind_set_slot(ngx_conf
     case NGX_DECLINED:
         ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
                            "invalid address \"%V\"", &value[1]);
+        /* fall through */
+
     default:
         return NGX_CONF_ERROR;
     }