develooper Front page | perl.perl5.porters | Postings from August 2005

Re: Documentation change for IPC::Open*.pm

Thread Previous
From:
Rafael Garcia-Suarez
Date:
August 22, 2005 09:07
Subject:
Re: Documentation change for IPC::Open*.pm
Message ID:
20050822180811.610d32fe@grubert.mandrakesoft.com
Ben Tilly wrote:
> On 8/22/05, Rafael Garcia-Suarez <rgarciasuarez@mandriva.com> wrote:
> > Ben Tilly wrote:
> > > On 8/22/05, Rafael Garcia-Suarez <rgarciasuarez@mandriva.com> 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 :)

Thread Previous


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About