Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add rusage support to pcntl_wait() and pcntl_waitpid() #952

Closed
wants to merge 3 commits into from

Conversation

4 participants
@tony2001
Copy link
Contributor

commented Dec 11, 2014

patch by Anton Stepanenko

}
else {

if (z_rusage) {

This comment has been minimized.

Copy link
@jpauli

jpauli Dec 19, 2014

Contributor

I'd have merged this "if" cond with the one above

}
}

if (z_rusage) {

This comment has been minimized.

Copy link
@jpauli

jpauli Dec 19, 2014

Contributor

I'd have merged this "if" cond with the one above

--FILE--
<?php
$pid = pcntl_fork();
if ($pid == 1) {

This comment has been minimized.

Copy link
@YuriyNasretdinov

YuriyNasretdinov Jan 22, 2015

There should be "-1", probably?

--FILE--
<?php
$pid = pcntl_fork();
if ($pid == 1) {

This comment has been minimized.

Copy link
@YuriyNasretdinov

YuriyNasretdinov Jan 22, 2015

Probably you meant "-1" here?

tony2001 added some commits Dec 11, 2014

@tony2001 tony2001 force-pushed the tony2001:pcntl_rusage branch from 58b3a67 to dde8319 Jan 22, 2015

@jpauli

This comment has been minimized.

Copy link
Contributor

commented May 13, 2015

Merged

@jpauli jpauli closed this May 13, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.