Page MenuHomePhabricator

Pass an integer to stream_select() for correctness
ClosedPublic

Authored by epriestley on Mar 19 2014, 12:09 PM.
Tags
None
Referenced Files
F14047720: D8567.diff
Thu, Nov 14, 4:59 AM
F14041748: D8567.diff
Mon, Nov 11, 9:09 PM
F14031780: D8567.id20335.diff
Sat, Nov 9, 11:54 AM
F14028745: D8567.diff
Fri, Nov 8, 3:21 PM
F14011548: D8567.diff
Fri, Nov 1, 3:02 AM
F13983148: D8567.id20333.diff
Sun, Oct 20, 4:26 AM
F13978522: D8567.diff
Fri, Oct 18, 11:37 PM
F13975391: D8567.diff
Fri, Oct 18, 9:55 AM
Subscribers

Details

Summary

Ref T4627. Passing $timeout_sec is slightly more correct here, although there is no change to program behavior.

Test Plan

Used this script to verify that passing 0 and 0.999321 (for example) have the same effect to stream_select():

<?php

$spec = array(
  0 => array('pipe', 'r'),
  1 => array('pipe', 'w'),
  2 => array('pipe', 'w'),
);

$pipes = array();
$proc = proc_open('sleep 30', $spec, $pipes);
list($stdin, $stdout, $stderr) = $pipes;

$read = array($stdout, $stderr);
$write = array();
$except = array();

$select_start = microtime(true);
stream_select($read, $write, $except, 0.999321, 80000);
$select_end = microtime(true);

printf(
  "select() took %d us to return.\n",
  ($select_end - $select_start) * 1000000);

Diff Detail

Repository
rPHU libphutil
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Pass an integer to stream_select() for correctness.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 19 2014, 5:12 PM
epriestley updated this revision to Diff 20335.

Closed by commit rPHUa5fd944af95d (authored by @epriestley).