[PATCH] Add missing NGX_HAVE_UNIX_DOMAIN checks
Maxim Dounin
mdounin at mdounin.ru
Wed Mar 27 21:07:43 UTC 2013
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
More information about the nginx-devel
mailing list