[PATCH] Upstream: remove unused struct field

Maxim Dounin mdounin at mdounin.ru
Thu Jan 26 13:51:25 UTC 2017


Hello!

On Tue, Jan 24, 2017 at 05:56:31PM -0800, Thibault Charbonnier wrote:

> Hello!
> 
> Quite minor, but as far as I can tell, this field is not being used 
> anywhere.
> 
> # HG changeset patch
> # User Thibault Charbonnier <thibaultcha at fastmail.com>
> # Date 1485309159 28800
> #      Tue Jan 24 17:52:39 2017 -0800
> # Branch remove-unused-upstream-struct-field
> # Node ID bb5d6b4762998d1ca8f9ad487fa0e9bf6e0094e9
> # Parent  d84f48e571e449ee6c072a8d52cdea8e06b88ef7
> Upstream: remove unused struct field
> 
> diff -r d84f48e571e4 -r bb5d6b476299 src/http/ngx_http_upstream.h
> --- a/src/http/ngx_http_upstream.h    Tue Jan 24 17:02:19 2017 +0300
> +++ b/src/http/ngx_http_upstream.h    Tue Jan 24 17:52:39 2017 -0800
> @@ -151,7 +151,6 @@
>       ngx_msec_t                       connect_timeout;
>       ngx_msec_t                       send_timeout;
>       ngx_msec_t                       read_timeout;
> -    ngx_msec_t                       timeout;
>       ngx_msec_t                       next_upstream_timeout;
> 
>       size_t                           send_lowat;

Thanks, committed (with slightly modified committ log).
And also removed a couple of additional unused fields.

-- 
Maxim Dounin
http://nginx.org/


More information about the nginx-devel mailing list