svn commit: r907825 - /tomcat/trunk/java/javax/servlet/jsp/JspFactory.java

[markt]

Author: markt
Date: Mon Feb  8 22:20:55 2010
New Revision: 907825

URL: http://svn.apache.org/viewvc?rev=907825&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48170
Remove unnecessary syncs that cause issues under heavy load

Modified:
    tomcat/trunk/java/javax/servlet/jsp/JspFactory.java

Modified: tomcat/trunk/java/javax/servlet/jsp/JspFactory.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/javax/servlet/jsp/JspFactory.java?rev=907825&r1=907824&r2=907825&view=diff
==============================================================================
--- tomcat/trunk/java/javax/servlet/jsp/JspFactory.java (original)
+++ tomcat/trunk/java/javax/servlet/jsp/JspFactory.java Mon Feb  8 22:20:55 2010
@@ -43,7 +43,7 @@
 
 public abstract class JspFactory {
 
-    private static JspFactory deflt = null;
+    private static volatile JspFactory deflt = null;
     
     /**
      * Sole constructor. (For invocation by subclass constructors, 
@@ -62,7 +62,7 @@
      * @param deflt        The default factory implementation
      */
 
-    public static synchronized void setDefaultFactory(JspFactory deflt) {
+    public static void setDefaultFactory(JspFactory deflt) {
         JspFactory.deflt = deflt;
     }
 
@@ -72,7 +72,7 @@
      * @return the default factory for this implementation
      */
 
-    public static synchronized JspFactory getDefaultFactory() {
+    public static JspFactory getDefaultFactory() {
         return deflt;
     }
 



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