Alex Colomar
January 24, 2023 06:24PM
Hi Maxim,

On 1/23/23 03:59, Maxim Dounin wrote:
> # HG changeset patch
> # User Maxim Dounin <mdounin@mdounin.ru>
> # Date 1674441584 -10800
> # Mon Jan 23 05:39:44 2023 +0300
> # Branch quic
> # Node ID 849f3b4043ee07a65bbfc4ad136e4246002ec00c
> # Parent 6bb884dc72916dc675df65d02abee0c9cfabc916
> QUIC: improved SO_COOKIE configure test.
>
> In nginx source code the inttypes.h include, if available, is used to define
> standard integer types. Changed the SO_COOKIE configure test to follow this.

May I ask for curiosity why do you prefer <inttypes.h> for the standard
integer types?

Normally, the preferred header file is <stdint.h>, since it's smaller;
unless you use the macros or functions from <inttypes.h>, which are
things like PRId32, or imaxabs(3).

Is there any portability issue with <stdint.h>?

Thanks,

Alex

>
> diff --git a/auto/os/linux b/auto/os/linux
> --- a/auto/os/linux
> +++ b/auto/os/linux
> @@ -263,7 +263,7 @@ ngx_feature="SO_COOKIE"
> ngx_feature_name="NGX_HAVE_SO_COOKIE"
> ngx_feature_run=no
> ngx_feature_incs="#include <sys/socket.h>
> - #include <stdint.h>"
> + $NGX_INCLUDE_INTTYPES_H"
> ngx_feature_path=
> ngx_feature_libs=
> ngx_feature_test="socklen_t optlen = sizeof(uint64_t);
> _______________________________________________
> nginx-devel mailing list
> nginx-devel@nginx.org
> https://mailman.nginx.org/mailman/listinfo/nginx-devel

--
http://www.alejandro-colomar.es/

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] QUIC: improved SO_COOKIE configure test

Maxim Dounin 402 January 22, 2023 10:00PM

Re: [PATCH] QUIC: improved SO_COOKIE configure test

Roman Arutyunyan 114 January 23, 2023 07:30AM

Re: [PATCH] QUIC: improved SO_COOKIE configure test

Maxim Dounin 102 January 23, 2023 07:00PM

Re: [PATCH] QUIC: improved SO_COOKIE configure test

Alex Colomar 117 January 24, 2023 06:24PM

Re: [PATCH] QUIC: improved SO_COOKIE configure test

Maxim Dounin 106 January 24, 2023 10:20PM

Re: [PATCH] QUIC: improved SO_COOKIE configure test

Alex Colomar 154 January 25, 2023 04:20AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 271
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready