Author: markt Date: Wed Apr 27 08:36:54 2016 New Revision: 1741162 URL: http://svn.apache.org/viewvc?rev=1741162&view=rev Log: Review of Map / ConcurrentMap / ConcurrentHashMap Declare field using interface rather than implementation. Expected to be safe since previous, more significant refactorings, have not caused issues.
Modified: tomcat/tc8.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOnEntry.java Modified: tomcat/tc8.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOnEntry.java URL: http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOnEntry.java?rev=1741162&r1=1741161&r2=1741162&view=diff ============================================================================== --- tomcat/tc8.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOnEntry.java (original) +++ tomcat/tc8.0.x/trunk/java/org/apache/catalina/authenticator/SingleSignOnEntry.java Wed Apr 27 08:36:54 2016 @@ -23,6 +23,7 @@ import java.io.Serializable; import java.security.Principal; import java.util.Set; import java.util.concurrent.ConcurrentHashMap; +import java.util.concurrent.ConcurrentMap; import javax.servlet.http.HttpServletRequest; @@ -52,7 +53,7 @@ public class SingleSignOnEntry implement // Marked as transient so special handling can be applied to serialization protected transient Principal principal = null; - protected ConcurrentHashMap<SingleSignOnSessionKey,SingleSignOnSessionKey> sessionKeys = + protected ConcurrentMap<SingleSignOnSessionKey,SingleSignOnSessionKey> sessionKeys = new ConcurrentHashMap<>(); protected String username = null; --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org