[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