Hello!

On Fri, Jan 31, 2014 at 12:52:05PM -0800, Piotr Sikora wrote:

> Hey,
> I agree with Maxim, there is no reason to limit the use of
> ngx_socket_errno only to the cross-platform code.

Note that I still agree with Ruslan's comments to your patch, use 
of ngx_socket_errno to check errors from ioctl() / fcntl() in 
unix-specific code looks unneeded.  And in case of eventfd - it 
looks certainly wrong, as eventfd isn't a socket.

I would rather drop most of the src/os/unix changes, probably with 
the exception of ngx_tcp_nopush() checks.

-- 
Maxim Dounin
http://nginx.org/

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel

Reply via email to