You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2010/02/08 23:20:55 UTC
svn commit: r907825 - /tomcat/trunk/java/javax/servlet/jsp/JspFactory.java
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-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org