akr	2011-07-19 16:27:26 +0900 (Tue, 19 Jul 2011)

  New Revision: 32581

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=32581

  Log:
    * io.c (UPDATE_MAXFD): removed.

  Modified files:
    trunk/ChangeLog
    trunk/io.c

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 32580)
+++ ChangeLog	(revision 32581)
@@ -1,3 +1,7 @@
+Tue Jul 19 16:25:15 2011  Tanaka Akira  <akr@f...>
+
+	* io.c (UPDATE_MAXFD): removed.
+
 Tue Jul 19 16:07:45 2011  Tanaka Akira  <akr@f...>
 
 	* io.c (rb_update_max_fd): new function.
Index: io.c
===================================================================
--- io.c	(revision 32580)
+++ io.c	(revision 32581)
@@ -151,14 +151,10 @@
 };
 
 static int max_file_descriptor = NOFILE;
-#define UPDATE_MAXFD(fd) \
-    do { \
-        if (max_file_descriptor < (fd)) max_file_descriptor = (fd); \
-    } while (0)
 void
 rb_update_max_fd(int fd)
 {
-    UPDATE_MAXFD(fd);
+    if (max_file_descriptor < (fd)) max_file_descriptor = (fd);
 }
 
 #define argf_of(obj) (*(struct argf *)DATA_PTR(obj))
@@ -531,7 +527,7 @@
 	    rb_sys_fail(0);
 	}
     }
-    UPDATE_MAXFD(fd);
+    rb_update_max_fd(fd);
     return fd;
 }
 
@@ -4622,7 +4618,7 @@
 	    rb_sys_fail(RSTRING_PTR(fname));
 	}
     }
-    UPDATE_MAXFD(fd);
+    rb_update_max_fd(fd);
     return fd;
 }
 
@@ -4915,8 +4911,8 @@
         }
     }
     if (ret == 0) {
-        UPDATE_MAXFD(pipes[0]);
-        UPDATE_MAXFD(pipes[1]);
+        rb_update_max_fd(pipes[0]);
+        rb_update_max_fd(pipes[1]);
     }
     return ret;
 }
@@ -6540,7 +6536,7 @@
 #else
     if (fstat(fd, &st) == -1) rb_sys_fail(0);
 #endif
-    UPDATE_MAXFD(fd);
+    rb_update_max_fd(fd);
 #if defined(HAVE_FCNTL) && defined(F_GETFL)
     ofmode = rb_io_oflags_fmode(oflags);
     if (NIL_P(vmode)) {
@@ -7695,7 +7691,7 @@
     retval = (int)rb_thread_io_blocking_region(nogvl_io_cntl, &arg, fd);
 #if defined(F_DUPFD)
     if (!io_p && retval != -1 && cmd == F_DUPFD) {
-	UPDATE_MAXFD(retval);
+	rb_update_max_fd(retval);
     }
 #endif
 

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/