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/