[PATCH] Use ngx_socket_errno where appropriate.
Maxim Dounin
mdounin at mdounin.ru
Mon Feb 3 13:04:44 UTC 2014
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/
More information about the nginx-devel
mailing list