You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@archiva.apache.org by jd...@apache.org on 2008/12/17 01:10:13 UTC

svn commit: r727229 - /archiva/trunk/archiva-modules/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/RepositoryContentFactory.java

Author: jdumay
Date: Tue Dec 16 16:10:13 2008
New Revision: 727229

URL: http://svn.apache.org/viewvc?rev=727229&view=rev
Log:
map variables should be final if they are to be used with the synchronized keyword

Modified:
    archiva/trunk/archiva-modules/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/RepositoryContentFactory.java

Modified: archiva/trunk/archiva-modules/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/RepositoryContentFactory.java
URL: http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/RepositoryContentFactory.java?rev=727229&r1=727228&r2=727229&view=diff
==============================================================================
--- archiva/trunk/archiva-modules/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/RepositoryContentFactory.java (original)
+++ archiva/trunk/archiva-modules/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/RepositoryContentFactory.java Tue Dec 16 16:10:13 2008
@@ -53,12 +53,18 @@
      */
     private ArchivaConfiguration archivaConfiguration;
 
-    private Map<String, ManagedRepositoryContent> managedContentMap;
+    private final Map<String, ManagedRepositoryContent> managedContentMap;
 
-    private Map<String, RemoteRepositoryContent> remoteContentMap;
+    private final Map<String, RemoteRepositoryContent> remoteContentMap;
 
     private PlexusContainer container;
 
+    public RepositoryContentFactory()
+    {
+        managedContentMap = new HashMap<String, ManagedRepositoryContent>();
+        remoteContentMap = new HashMap<String, RemoteRepositoryContent>();
+    }
+
     /**
      * Get the ManagedRepositoryContent object for the repository Id specified.
      * 
@@ -154,9 +160,6 @@
     public void initialize()
         throws InitializationException
     {
-        managedContentMap = new HashMap<String, ManagedRepositoryContent>();
-        remoteContentMap = new HashMap<String, RemoteRepositoryContent>();
-
         archivaConfiguration.addChangeListener( this );
     }