[PATCH] Headers filter: style

Maxim Dounin mdounin at mdounin.ru
Thu Jun 23 19:09:39 UTC 2016


Hello!

On Thu, Jun 23, 2016 at 09:53:37AM -0700, Piotr Sikora wrote:

> Hey Maxim,
> 
> > I would rather not.  The "hv" is used to refer to a particular
> > ngx_http_header_val_t, while "h" here is an array of elements.
> 
> But "h" is used throughout the codebase for ngx_table_elt_t, which
> makes this particular case confusing.
> 
> Also, "h" is used for ngx_table_elt_t regardless of whether it points
> to a particular item or an array, so your argument isn't consistent
> with the current style.

The "h" is used for variables of various types, including 
ngx_http_file_cache_header_t, ngx_str_t, ngx_keyval_t, 
ngx_table_elt_t, ngx_http_fastcgi_header_t, ngx_uint_t, uin32_t, 
and int.  It's just a convenient short name for a variable.

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



More information about the nginx-devel mailing list