From 49d11362051bcbd3ffe2165ca98a21eeab3672bc Mon Sep 17 00:00:00 2001
From: Nick Mathewson <nickm@torproject.org>
Date: Tue, 5 Jul 2011 00:11:59 -0400
Subject: [PATCH] Fix select.c compilation on systems with no NFDBITS

---
 select.c | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/select.c b/select.c
index 892ddfaf..332434d3 100644
--- a/select.c
+++ b/select.c
@@ -50,17 +50,22 @@
 #include "log-internal.h"
 #include "evmap-internal.h"
 
-#ifndef howmany
-#define howmany(x, y)   (((x)+((y)-1))/(y))
-#endif
-
 #ifndef _EVENT_HAVE_FD_MASK
 /* This type is mandatory, but Android doesn't define it. */
-#undef NFDBITS
-#define NFDBITS (sizeof(long)*8)
 typedef unsigned long fd_mask;
 #endif
 
+#ifndef NFDBITS
+#define NFDBITS (sizeof(fd_mask)*8)
+#endif
+
+/* Divide positive x by y, rounding up. */
+#define DIV_ROUNDUP(x, y)   (((x)+((y)-1))/(y))
+
+/* How many bytes to allocate for N fds? */
+#define SELECT_ALLOC_SIZE(n) \
+	(DIV_ROUNDUP(n, NFDBITS) * sizeof(fd_mask))
+
 struct selectop {
 	int event_fds;		/* Highest fd in fd set */
 	int event_fdsz;
@@ -100,7 +105,7 @@ select_init(struct event_base *base)
 	if (!(sop = mm_calloc(1, sizeof(struct selectop))))
 		return (NULL);
 
-	if (select_resize(sop, howmany(32 + 1, NFDBITS)*sizeof(fd_mask))) {
+	if (select_resize(sop, SELECT_ALLOC_SIZE(32 + 1))) {
 		select_free_selectop(sop);
 		return (NULL);
 	}
@@ -253,8 +258,7 @@ select_add(struct event_base *base, int fd, short old, short events, void *p)
 		/* In theory we should worry about overflow here.  In
 		 * reality, though, the highest fd on a unixy system will
 		 * not overflow here. XXXX */
-		while (fdsz <
-		    (int) (howmany(fd + 1, NFDBITS) * sizeof(fd_mask)))
+		while (fdsz < (int) SELECT_ALLOC_SIZE(fd + 1))
 			fdsz *= 2;
 
 		if (fdsz != sop->event_fdsz) {