Ben Tilly wrote:
> On 8/22/05, Rafael Garcia-Suarez <[EMAIL PROTECTED]> wrote:
> > Ben Tilly wrote:
> > > On 8/22/05, Rafael Garcia-Suarez <[EMAIL PROTECTED]> wrote:
> [...]
> > > Huh, try again.
> > 
> > Confusing.
> > 
> > From Open2.diff :
> > 
> > -    $pid = open2(\*RDRFH, \*WTRFH, 'some cmd and args');
> > +    $pid = open2(\*CHLD_IN, \*CHLD_OUT, 'some cmd and args');
> > 
> > I read CHLD_IN as "child's input", so surely that should be the other way
> > around ?
> > 
> > Moreover, Open3.diff has :
> > 
> > -    $pid = open3(\*WTRFH, \*RDRFH, \*ERRFH,
> > +    $pid = open3(\*CHLD_IN, \*CHLD_OUT, \*CHLD_ERR,
> >                     'some cmd and args', 'optarg', ...);
> > 
> > I hereby suppose that Open2.diff is backwards.
> > 
> Gah.  I didn't mean to demonstrate how the variable names in the API
> cause me confusion, but I guess I just did. :-(
> 
> Yes, Open2.diff is backwards.

OK :) thus, applied backwards as change #25320 :)

Reply via email to