[PATCH] proposal to remove unused macro ngx_sleep()

crespin nginx-forum at nginx.us
Thu Aug 7 14:05:16 UTC 2014


Hello,

ngx_sleep() macro is unused.

regards,

yves


# HG changeset patch
# User Yves Crespin<yves.crespin at e-quartz.fr>
# Date 1407420247 -7200
# Node ID 8120c2937062998497546a3ecb38b78bb302fe34
# Parent  97fcc94109c0ea7b929f7fa5a8c7ee95ecf7cb5c
remove unused ngx_sleep() macro

In win32 implementation, even for a long sleep (60s), ngx_msleep()
is used instead of ngx_sleep().
Implementation of |sleep()| can use |SIGALRM| and it's used in nginx.

diff -r 97fcc94109c0 -r 8120c2937062 src/os/unix/ngx_time.h
--- a/src/os/unix/ngx_time.h	Thu Aug 07 15:04:43 2014 +0200
+++ b/src/os/unix/ngx_time.h	Thu Aug 07 16:04:07 2014 +0200
@@ -60,7 +60,6 @@
 
 #define ngx_gettimeofday(tp)  (void) gettimeofday(tp, NULL);
 #define ngx_msleep(ms)        (void) usleep(ms * 1000)
-#define ngx_sleep(s)          (void) sleep(s)
 
 
 #endif /* _NGX_TIME_H_INCLUDED_ */

Posted at Nginx Forum: http://forum.nginx.org/read.php?2,252387,252387#msg-252387



More information about the nginx mailing list