Le Sunday 03 May 2009 22:58:26 Jeff King, vous avez écrit :
> On Wed, Apr 29, 2009 at 09:12:23AM -0400, Bill Pemberton wrote:
> > The script was using bareword filehandles. This is considered a bad
> > practice so they have been changed to indirect filehandles.
>
> I think this is a real improvement; using indirect filehandles mean they
> get scoped properly, which can avoid errors (especially forgetting to
> close() them, which happens automagically when they go out of scope).
> Assuming, of course, that the scoping added by your change is correct,
> and doesn't close a handle during a loop that we may have wanted to keep
> open (I didn't check carefully).
>
> But in the patch itself:
> > - open(C,">",$compose_filename)
> > + open my $C,'>',$compose_filename
>
> There are actually two things happening here:
>
> 1. s/C/my $C/, which I think is good
>
> 2. losing the parentheses around open(). This is a style issue, but I
> think we usually prefer the parenthesized form of most perl
> builtins (and certainly in the absence of other information, it
> should be left as-is).
>
And why not go the full way and using IO::File?
my $fh = new IO::File;
$fh->open("/the/file", O_RDONLY|...)
--
Francis Galiegue
fge@xxxxxxxxxxxx
Ingénieur système
Mob : +33 (0) 683 877 875
Tel : +33 (0) 178 945 552
One2team
40 avenue Raymond Poincaré
75116 Paris
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html