Skip Menu |

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

Report information
The Basics
Id: 2693
Status: resolved
Priority: 0/
Queue: Mail-IMAPClient

People
Owner: DJKERNEN__NO_SOLICITING__ [...] cpan.org
Requestors: andrewb [...] omnilab.com.au
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Date: Fri, 30 May 2003 11:47:04 +1000
From: Andrew Bramble <andrewb [...] omnilab.com.au>
To: bug-Mail-IMAPClient [...] rt.cpan.org
Subject: One character typo in Mail::IMAPClient::migrate _debug call line 976
Hello, I have stumbled on a typo in Mail::IMAPClient's migrate method. This produces one warning per migrated message due to a plus in place of a dot at line 976. $peer->_debug("Chunk $chunkCount: " + "Wrote $wroteSoFar bytes (out of $chunk)\n"); } which works much nicer as $peer->_debug("Chunk $chunkCount: " . "Wrote $wroteSoFar bytes (out of $chunk)\n"); } Thankyou for Mail::IMAPClient Andrew Bramble omnilabgroup
Date: Fri, 30 May 2003 11:50:53 +1000
From: Andrew Bramble <andrewb [...] omnilab.com.au>
To: bug-Mail-IMAPClient [...] rt.cpan.org
Subject: RE: One character typo in Mail::IMAPClient::migrate _debug call line 976
Of course I had my mind-reading hat on this morning. Mail::IMAPClient version is 2.2.7 Goofed :) AB Show quoted text
> Hello, > > I have stumbled on a typo in Mail::IMAPClient's migrate method. This > produces one warning per migrated message due to a plus in place of a > dot at line 976. > > $peer->_debug("Chunk $chunkCount: " + > "Wrote $wroteSoFar bytes (out of $chunk)\n"); > } > > which works much nicer as > > $peer->_debug("Chunk $chunkCount: " . > "Wrote $wroteSoFar bytes (out of $chunk)\n"); > } > > > Thankyou for Mail::IMAPClient > > Andrew Bramble > omnilabgroup
This was not a typo! The "+" is the correct operator to use for concatenation. If you need proof just check any decent Java book... ;-) But to make you happy I changed it to a "." in 2.2.8. Cheers, Dave K.