commits@javamail.java.net

[mercurial:87] Fix bug that caused CAPABILITY command to be issued unnecessarily.

From: <shannon_at_kenai.com>
Date: Tue, 11 Nov 2008 01:05:08 +0000 (GMT)

Repository: mercurial
Revision: 87
Author: Bill Shannon <bill.shannon_at_sun.com>
Date: 2008-11-05 19:16:47 UTC

Log Message:
-----------
Fix bug that caused CAPABILITY command to be issued unnecessarily.
A field was being initialized via a call into the IMAPProtocol class
from the superclass constructor via an overridden method, and then
overwritten by the field initialization in the IMAPProtocol class
itself. Argh!

Modified Paths:
--------------
    mail/src/main/java/com/sun/mail/imap/protocol/IMAPProtocol.java

Diffs:
-----
diff -r e6cc06a28737 -r dc8ba0762a6d
mail/src/main/java/com/sun/mail/imap/protocol/IMAPProtocol.java
--- a/mail/src/main/java/com/sun/mail/imap/protocol/IMAPProtocol.java
Wed Nov 05 11:12:52 2008 -0800
+++ b/mail/src/main/java/com/sun/mail/imap/protocol/IMAPProtocol.java
Wed Nov 05 11:16:47 2008 -0800
@@ -73,8 +73,12 @@
     // WARNING: authenticated may be set to true in superclass
     // constructor, don't initialize it here.
 
- private Map capabilities = null;
- private List authmechs = null;
+ private Map capabilities;
+ // WARNING: capabilities may be initialized as a result of
superclass
+ // constructor, don't initialize it here.
+ private List authmechs;
+ // WARNING: authmechs may be initialized as a result of superclass
+ // constructor, don't initialize it here.
     private String[] searchCharsets; // array of search charsets
 
     private String name;