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/