pcntl_wait / pcntl_waitpid: Document the rusage parameter (fixes #861) - #4917
pcntl_wait / pcntl_waitpid: Document the rusage parameter (fixes #861)#4917AllenJB wants to merge 5 commits into
Conversation
TimWolla
left a comment
TimWolla
left a comment
There was a problem hiding this comment.
pcntl_waitid already has the parameter documented. Can you align the explanation: https://www.php.net/manual/en/function.pcntl-waitid.php?
Done |
TimWolla
left a comment
TimWolla
left a comment
There was a problem hiding this comment.
Except for the example this LGTM. Perhaps it makes sense to split the example off in another PR?
| print "PARENT pcntl_waitpid: No exited children.\n"; | ||
| } else { | ||
| print "PARENT pcntl_waitpid: Child PID {$deadPid} exited with status: " | ||
| . pcntl_wexitstatus($status) ."\n"; |
There was a problem hiding this comment.
The documentation for wexitstatus says:
This function is only useful if pcntl_wifexited() returned true.
So, is a wifexited call missing here? Looking more carefully at the example, it uses quite a few functions from the pcntl extension. It might make sense to include it in https://www.php.net/manual/en/pcntl.example.php instead.
| &reftitle.examples; | ||
| <para> | ||
| <example> |
| <varlistentry> | ||
| <term><parameter>resource_usage</parameter></term> | ||
| <listitem> | ||
| <para> | ||
| The <parameter>resource_usage</parameter> parameter is set to an | ||
| associative <type>array</type> containing resource usage statistics | ||
| from the child process. | ||
|
|
||
| This is supported either if the wait6 system call is available | ||
| (e.g. on FreeBSD), or on Linux through the raw waitid system call. For | ||
| information on the contents see <function>getrusage</function>. | ||
| </para> | ||
| </listitem> | ||
| </varlistentry> |
There was a problem hiding this comment.
Considering how often this is repeated, an XInclude or XML entity to have consistent wording would be better
4 participants
No description provided.