Author: maxim
Date: 2012-03-19 14:57:29 +0000 (Mon, 19 Mar 2012)
New Revision: 4547

Log:
For the sake of case/switch code readability, 'fall through'
comments added.


Modified:
trunk/src/http/ngx_http_parse.c
trunk/src/http/ngx_http_upstream.c

Modified: trunk/src/http/ngx_http_parse.c
===================================================================
--- trunk/src/http/ngx_http_parse.c 2012-03-16 19:15:33 UTC (rev 4546)
+++ trunk/src/http/ngx_http_parse.c 2012-03-19 14:57:29 UTC (rev 4547)
@@ -1153,6 +1153,7 @@
break;
case '+':
r->plus_in_uri = 1;
+ /* fall through */
default:
*u++ = ch;
break;

Modified: trunk/src/http/ngx_http_upstream.c
===================================================================
--- trunk/src/http/ngx_http_upstream.c 2012-03-16 19:15:33 UTC (rev 4546)
+++ trunk/src/http/ngx_http_upstream.c 2012-03-19 14:57:29 UTC (rev 4547)
@@ -3312,6 +3312,8 @@
switch (n) {
case 0:
u->cacheable = 0;
+ /* fall through */
+
case NGX_ERROR:
return NGX_OK;

@@ -4460,6 +4462,8 @@
case NGX_DECLINED:
ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
"invalid address \"%V\"", &value[1]);
+ /* fall through */
+
default:
return NGX_CONF_ERROR;
}

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel