commits@javamail.java.net

[mercurial:122] Synchronize the Calendar object instead of the method.

From: <shannon_at_kenai.com>
Date: Sat, 28 Feb 2009 01:44:44 +0000 (GMT)

Repository: mercurial
Revision: 122
Author: Bill Shannon <bill.shannon_at_sun.com>
Date: 2009-02-12 00:12:47 UTC
Link:
http://kenai.com/projects/javamail/sources/mercurial/revision/122

Log Message:
-----------
Synchronize the Calendar object instead of the method.

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

Diffs:
-----
diff -r 655d28bc5acf -r 58bf72eaa418
mail/src/main/java/com/sun/mail/imap/protocol/SearchSequence.java
--- a/mail/src/main/java/com/sun/mail/imap/protocol/SearchSequence.java
Wed Feb 11 11:18:33 2009 -0800
+++ b/mail/src/main/java/com/sun/mail/imap/protocol/SearchSequence.java
Wed Feb 11 16:12:47 2009 -0800
@@ -358,16 +358,19 @@
     // A GregorianCalendar object in the current timezone
     private static Calendar cal = new GregorianCalendar();
 
- /*
- * Synchronized to coordinate access to shared Calendar object.
- */
     private static synchronized String toIMAPDate(Date date) {
        StringBuffer s = new StringBuffer();
- cal.setTime(date);
 
- s.append(cal.get(Calendar.DATE)).append("-");
- s.append(monthTable[cal.get(Calendar.MONTH)]).append('-');
- s.append(cal.get(Calendar.YEAR));
+ /*
+ * Synchronized to coordinate access to shared Calendar object.
+ */
+ synchronized (cal) {
+ cal.setTime(date);
+
+ s.append(cal.get(Calendar.DATE)).append("-");
+ s.append(monthTable[cal.get(Calendar.MONTH)]).append('-');
+ s.append(cal.get(Calendar.YEAR));
+ }
 
        return s.toString();
     }