On Thu, Jan 04, 2007 at 10:37:51PM +0000, Jonathan Worthington wrote:
> Lee Duhem (via RT) wrote:
> >This patch suppress some uninitialized value warning in 
> >config/inter/yacc.pm when use --maintainer option at configure.
> >  
> I'm kinda concerned about what will happen if $3 was 0 (and was meant to 
> be). Perhaps it should be:
> 
> $prog_patch = defined $3 ? $3 : "";
> 

$3 come from line 88 of config/inter/yacc.pm
    if ( $stdout =~ /Bison .*? (\d+) \. (\d+) (\w)?/x ) {

and I'm try this under Perl 5.8.8, and C<use warnings; use strict;>,
not get any warning about uninitialized value or similar.

But anyway, your advice is better than my patch. Please apply this
and I'll close [perl #41163]. 

Thanks.

Lee

Reply via email to