svn commit: r1741197 - /tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java

[ma...@apache.org]

Author: markt
Date: Wed Apr 27 10:07:25 2016
New Revision: 1741197

URL: http://svn.apache.org/viewvc?rev=1741197&view=rev
Log:
Reduce visibility.
Use final.

Modified:
    tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java

Modified: tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java?rev=1741197&r1=1741196&r2=1741197&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java (original)
+++ tomcat/trunk/java/org/apache/catalina/mapper/Mapper.java Wed Apr 27 10:07:25 2016
@@ -50,7 +50,7 @@ public final class Mapper {
 
     private static final Log log = LogFactory.getLog(Mapper.class);
 
-    static final StringManager sm = StringManager.getManager(Mapper.class);
+    private static final StringManager sm = StringManager.getManager(Mapper.class);
 
     // ----------------------------------------------------- Instance Variables
 
@@ -58,20 +58,21 @@ public final class Mapper {
     /**
      * Array containing the virtual hosts definitions.
      */
+    // Package private to facilitate testing
     volatile MappedHost[] hosts = new MappedHost[0];
 
 
     /**
      * Default host name.
      */
-    String defaultHostName = null;
+    private String defaultHostName = null;
 
 
     /**
      * Mapping from Context object to Context version to support
      * RequestDispatcher mappings.
      */
-    Map<Context, ContextVersion> contextObjectToContextVersionMap =
+    private final Map<Context, ContextVersion> contextObjectToContextVersionMap =
             new ConcurrentHashMap<>();
 
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org