Maxim Dounin
March 27, 2013 05:08PM
Hello!

On Wed, Mar 27, 2013 at 01:54:57PM -0700, Piotr Sikora wrote:

> Hey,
> I've noticed that Maxim's recent commit (r5142) was missing a check
> for AF_UNIX existence (NGX_HAVE_UNIX_DOMAIN) and it seems
> that the check was missing in a few other places as well.
>
> Attached patch adds those, even though apparently they aren't really needed.

They aren't needed, and I omitted it intentionally. AF_UNIX is
required to be defined by POSIX, and it is defined even on win32.
I rather think about removing the #if's in places where they are
used only to protect AF_UNIX constant.

--
Maxim Dounin
http://nginx.org/en/donation.html

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

[PATCH] Add missing NGX_HAVE_UNIX_DOMAIN checks

Piotr Sikora 782 March 27, 2013 04:56PM

Re: [PATCH] Add missing NGX_HAVE_UNIX_DOMAIN checks

Maxim Dounin 402 March 27, 2013 05:08PM



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

Online Users

Guests: 206
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 500 on July 15, 2024
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready