[前][次][番号順一覧][スレッド一覧]

ruby-changes:23953

From: akr <ko1@a...>
Date: Sun, 10 Jun 2012 10:46:55 +0900 (JST)
Subject: [ruby-changes:23953] akr:r36004 (trunk): update async-signal-safe comments.

akr	2012-06-10 10:46:45 +0900 (Sun, 10 Jun 2012)

  New Revision: 36004

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

  Log:
    update async-signal-safe comments.

  Modified files:
    trunk/process.c

Index: process.c
===================================================================
--- process.c	(revision 36003)
+++ process.c	(revision 36004)
@@ -2543,7 +2543,7 @@
 
     obj = rb_ary_entry(options, EXEC_OPTION_DUP2);
     if (!NIL_P(obj)) {
-        if (run_exec_dup2(obj, e->dup2_tmpbuf, soptions, errmsg, errmsg_buflen) == -1) /* async-signal-safe */
+        if (run_exec_dup2(obj, e->dup2_tmpbuf, soptions, errmsg, errmsg_buflen) == -1) /* hopefully async-signal-safe */
             return -1;
     }
 
@@ -2650,8 +2650,8 @@
 {
     int ret;
     before_exec_non_async_signal_safe(); /* async-signal-safe if forked_child is true */
-    ret = rb_exec_async_signal_safe(e, errmsg, errmsg_buflen);
-    preserving_errno(after_exec_non_async_signal_safe()); /* not async-signal-safe because after_exec calls rb_thread_start_timer_thread.  */
+    ret = rb_exec_async_signal_safe(e, errmsg, errmsg_buflen); /* hopefully async-signal-safe */
+    preserving_errno(after_exec_non_async_signal_safe()); /* not async-signal-safe because it calls rb_thread_start_timer_thread.  */
     return ret;
 }
 
@@ -2678,7 +2678,7 @@
 }
 
 #ifdef HAVE_FORK
-/* This function should be async-signal-safe.  Actually it isn't. */
+/* This function should be async-signal-safe.  Hopefully it is. */
 static int
 rb_exec_atfork(void* arg, char *errmsg, size_t errmsg_buflen)
 {

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

[前][次][番号順一覧][スレッド一覧]