Skip Menu |

This queue is for tickets about the Mail-Box CPAN distribution.

Report information
The Basics
Id: 6330
Status: resolved
Priority: 0/
Queue: Mail-Box
People
Owner: Nobody in particular
Requestors: ville.skytta [...] iki.fi
Cc:
AdminCc:
Bug Information
Severity: Unimportant
Broken in: 2.054
Fixed in: (no value)

History Show all quoted text
  Mon May 17 13:20:13 2004 scop [...] cpan.org - Ticket created
Subject: Trivial: avoid test warning
2.055 (not in "broken in" list in rt.cpan.org) emits a test warning with perl 5.8.3: 51folder/21tiehash....ok 98/101Use of uninitialized value in hash element at 51folder/21tiehash.t line 56, <GEN0> line 3400. The attached patch avoids it.
--- tests/51folder/21tiehash.t.orig 2004-05-17 20:17:33.000000000 +0300 +++ tests/51folder/21tiehash.t 2004-05-17 20:18:43.000000000 +0300 @@ -55,3 +55,3 @@ # Double messages will not be added. -$folder{ (undef) } = $folder{$msgid}->clone; +$folder{undef} = $folder{$msgid}->clone; cmp_ok(keys %folder , "==", 44);
  Mon Aug 02 10:06:56 2004 MARKOV [...] cpan.org - Status changed from 'new' to 'resolved'
  Mon Aug 02 10:06:56 2004 MARKOV [...] cpan.org - Correspondence added
There is a huge difference between $f{ (undef) } = 1; $f{undef} = 1; because the latter is equivalent to $f{'undef'} = 1; I am glad with the regression tests of MailBox: it shows another bug of Perl 5.8.3.... Apparently, the warning is produced before the tie check has been done, which is a mistake. 5.8.3 also has unicode bugs. Haven't had any complaints about 5.8.4 so far...