Hello!

On Tue, Jan 12, 2021 at 12:40:06AM +0000, Chris Newton wrote:

> I just came across a problem with an innocuous looking line of code in a
> new module I have been working on:
>
> if (tmp_str.len == 0)
>
> ngx_str_set(&tmp_str, "/");
>
>
> The modification of tmp_str.data to '/' was always being made, but the
> length wasn't if the test failed. This turns out to be caused by the style
> used in the definition of the ngx_str_set macro.

I always wonder why people intentionally ignore nginx style and
then complain something doesn't work for them. Following nginx
style and always using curly brackets is how it is expected to
work.

--
Maxim Dounin
http://mdounin.ru/
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel