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

ruby-changes:23930

From: akr <ko1@a...>
Date: Sat, 9 Jun 2012 10:40:31 +0900 (JST)
Subject: [ruby-changes:23930] akr:r35981 (trunk): update async-signal-safe comments.

akr	2012-06-09 10:40:22 +0900 (Sat, 09 Jun 2012)

  New Revision: 35981

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

  Log:
    update async-signal-safe comments.

  Modified files:
    trunk/process.c

Index: process.c
===================================================================
--- process.c	(revision 35980)
+++ process.c	(revision 35981)
@@ -1062,7 +1062,7 @@
 #define try_with_sh(prog, argv, envp) (void)0
 #endif
 
-/* This function should be async-signal-safe.  Actually it isn't because after_exec(). */
+/* This function should be async-signal-safe.  Actually it is. */
 static int
 proc_exec_cmd(const char *prog, VALUE argv_str, VALUE envp_str)
 {
@@ -1127,7 +1127,7 @@
 #endif
 }
 
-/* This function should be async-signal-safe.  Actually it isn't because after_exec(). */
+/* This function should be async-signal-safe.  Actually it is. */
 static int
 proc_exec_sh(const char *str, VALUE envp_str)
 {
@@ -2600,13 +2600,13 @@
     }
 
     if (e->use_shell) {
-	proc_exec_sh(RSTRING_PTR(e->invoke.sh.shell_script), e->envp_str); /* not async-signal-safe because after_exec. */
+	proc_exec_sh(RSTRING_PTR(e->invoke.sh.shell_script), e->envp_str); /* async-signal-safe */
     }
     else {
 	char *abspath = NULL;
 	if (!NIL_P(e->invoke.cmd.command_abspath))
 	    abspath = RSTRING_PTR(e->invoke.cmd.command_abspath);
-	proc_exec_cmd(abspath, e->invoke.cmd.argv_str, e->envp_str); /* not async-signal-safe because after_exec. */
+	proc_exec_cmd(abspath, e->invoke.cmd.argv_str, e->envp_str); /* async-signal-safe */
     }
 #if !defined(HAVE_FORK)
     preserving_errno(rb_run_exec_options_err(sargp, NULL, errmsg, errmsg_buflen));

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

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