[Clirr-devel] CVS: clirr/core/src/java/net/sf/clirr/core MessageManager.java, 1.3, 1.4

[Lars Küh. <lk...@us...>]

Update of /cvsroot/clirr/clirr/core/src/java/net/sf/clirr/core
In directory ddv4jf1.ch3.sourceforge.com:/tmp/cvs-serv20844/core/src/java/net/sf/clirr/core

Modified Files:
	MessageManager.java 
Log Message:
code cleanup and fixes for findbug problems

Index: MessageManager.java
===================================================================
RCS file: /cvsroot/clirr/clirr/core/src/java/net/sf/clirr/core/MessageManager.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- MessageManager.java	3 Aug 2005 17:16:13 -0000	1.3
+++ MessageManager.java	24 Mar 2009 21:29:07 -0000	1.4
@@ -19,6 +19,8 @@
 
 package net.sf.clirr.core;
 
+import java.io.Serializable;
+import java.util.Collections;
 import java.util.Iterator;
 import java.util.ArrayList;
 import java.util.Comparator;
@@ -31,13 +33,15 @@
 public final class MessageManager
 {
     private static MessageManager instance;
-    private ArrayList messages = new ArrayList();
+    private final ArrayList messages = new ArrayList();
 
     /**
      * Utility class to sort messages by their numeric ids.
      */
-    private static class MessageComparator implements Comparator
+    private static class MessageComparator implements Comparator, Serializable
     {
+        private static final long serialVersionUID = -7967384385861738856L;
+
         public int compare(Object o1, Object o2)
         {
             Message m1 = (Message) o1;
@@ -85,7 +89,7 @@
      */
     public void checkUnique()
     {
-        java.util.Collections.sort(messages, new MessageComparator());
+        Collections.sort(messages, new MessageComparator());
         int lastId = -1;
         for (Iterator i = messages.iterator(); i.hasNext();)
         {