[nginx] HTTP/2: fixed build with -Werror=unused-but-set-variable.

Ruslan Ermilov ru at nginx.com
Thu Feb 8 09:12:41 UTC 2018


details:   http://hg.nginx.org/nginx/rev/a49af443656f
branches:  
changeset: 7202:a49af443656f
user:      Ruslan Ermilov <ru at nginx.com>
date:      Thu Feb 08 12:11:30 2018 +0300
description:
HTTP/2: fixed build with -Werror=unused-but-set-variable.

diffstat:

 src/http/v2/ngx_http_v2_filter_module.c |  13 +++----------
 1 files changed, 3 insertions(+), 10 deletions(-)

diffs (33 lines):

diff -r 641306096f5b -r a49af443656f src/http/v2/ngx_http_v2_filter_module.c
--- a/src/http/v2/ngx_http_v2_filter_module.c	Thu Feb 08 09:55:03 2018 +0300
+++ b/src/http/v2/ngx_http_v2_filter_module.c	Thu Feb 08 12:11:30 2018 +0300
@@ -688,18 +688,11 @@ ngx_http_v2_push_resources(ngx_http_requ
     ngx_str_t                   path, authority;
     ngx_uint_t                  i, push;
     ngx_table_elt_t           **h;
-    ngx_connection_t           *fc;
-    ngx_http_v2_stream_t       *stream;
     ngx_http_v2_loc_conf_t     *h2lcf;
-    ngx_http_v2_connection_t   *h2c;
     ngx_http_complex_value_t   *pushes;
 
-    fc = r->connection;
-
-    ngx_log_debug0(NGX_LOG_DEBUG_HTTP, fc->log, 0, "http2 push resources");
-
-    stream = r->stream;
-    h2c = stream->connection;
+    ngx_log_debug0(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
+                   "http2 push resources");
 
     ngx_str_null(&authority);
 
@@ -744,7 +737,7 @@ ngx_http_v2_push_resources(ngx_http_requ
 
     for (i = 0; i < r->headers_out.link.nelts; i++) {
 
-        ngx_log_debug1(NGX_LOG_DEBUG_HTTP, fc->log, 0,
+        ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
                        "http2 parse link: \"%V\"", &h[i]->value);
 
         start = h[i]->value.data;


More information about the nginx-devel mailing list