Bug #76953 for Config-General: ForceArray causes undefined value warnings with empty config keys

Wed May 02 21:44:52 2012 stephen.thirlwall [...] strategicdata.com.au - Ticket created

Subject:

ForceArray causes undefined value warnings with empty config keys

This is actually a duplicate of #65860, but I thought a more descriptive subject line might be useful. A simple repro case is this single line config. novalue With -ForceArray => 1, this gives the warning: Use of uninitialized value $value in pattern match (m//) at Config/General.pm line 872

Wed May 02 21:49:05 2012 stephen.thirlwall [...] strategicdata.com.au - Correspondence added

Here's a failing test case - the 2nd one throws the warning. diff --git a/t/run.t b/t/run.t index 0217d81..3a9f759 100644 --- a/t/run.t +++ b/t/run.t @@ -727,6 +727,9 @@ my $cfg53 = new Config::General(-AllowSingleQuoteInterpolation => 1, -String => my %hash53 = $cfg53->getall(); is($hash53{have}, "'1'", "check -AllowSingleQuoteInterpolation"); +# check undefined values don't throw warnings with -ForceArray rt#76953 +Config::General->new( -String => "novalue" )->getall(); +Config::General->new( -String => "novalue", -ForceArray => 1 )->getall(); # Make sure no warnings were seen during the test. ok( !@WARNINGS_FOUND, "No unexpected warnings seen" );

Subject:

run.t.patch

diff --git a/t/run.t b/t/run.t index 0217d81..3a9f759 100644 --- a/t/run.t +++ b/t/run.t @@ -727,6 +727,9 @@ my $cfg53 = new Config::General(-AllowSingleQuoteInterpolation => 1, -String => my %hash53 = $cfg53->getall(); is($hash53{have}, "'1'", "check -AllowSingleQuoteInterpolation"); +# check undefined values don't throw warnings with -ForceArray rt#76953 +Config::General->new( -String => "novalue" )->getall(); +Config::General->new( -String => "novalue", -ForceArray => 1 )->getall(); # Make sure no warnings were seen during the test. ok( !@WARNINGS_FOUND, "No unexpected warnings seen" );

Wed May 02 21:49:07 2012 stephen.thirlwall [...] strategicdata.com.au - Status changed from 'new' to 'open'

Wed May 02 21:50:14 2012 TJC [...] cpan.org - Cc TJC added

Wed Jul 18 08:59:06 2012 tlinden [...] cpan.org - Correspondence added

fixed in 2.51.

Wed Jul 18 08:59:07 2012 tlinden [...] cpan.org - Status changed from 'open' to 'resolved'