Author: [email protected]
Date: Tue Dec 16 07:48:53 2008
New Revision: 2885

Modified:
   trunk/src/ca/sqlpower/architect/ddl/TypeMap.java

Log:
Fixed a find bugs warning. The variable mainInstance is now final because it is intended to be a singleton instance which can't be modified. Also limited the access of this variable to private instead of protected since there is a getter method for this variable.

Modified: trunk/src/ca/sqlpower/architect/ddl/TypeMap.java
==============================================================================
--- trunk/src/ca/sqlpower/architect/ddl/TypeMap.java    (original)
+++ trunk/src/ca/sqlpower/architect/ddl/TypeMap.java Tue Dec 16 07:48:53 2008
@@ -69,8 +69,8 @@
        ArrayList EMPTY_LIST = new ArrayList();
        Pattern SPACE_STRIPPER = Pattern.compile("[\\s]+");
        
-       // singleton instance
-       protected static TypeMap mainInstance = new TypeMap();
+ // singleton instance. Since this class contains a getter method for this variable, this can be private.
+       private static final TypeMap mainInstance = new TypeMap();

        public static void main(String[] args) {
                TypeMap tm = TypeMap.getInstance();     

Reply via email to