You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2004/03/04 22:23:18 UTC

cvs commit: jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/core StandardWrapperValve.java

remm        2004/03/04 13:23:18

  Modified:    catalina/src/share/org/apache/catalina/core
                        StandardWrapperValve.java
  Log:
  - Looking at a diff by Bill in the JBoss CVS, it seems safer to make
    this volatile. I'm not an expert on these issues, though, so let me know if
    this is wrong.
  
  Revision  Changes    Path
  1.26      +6 -6      jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/core/StandardWrapperValve.java
  
  Index: StandardWrapperValve.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/core/StandardWrapperValve.java,v
  retrieving revision 1.25
  retrieving revision 1.26
  diff -u -r1.25 -r1.26
  --- StandardWrapperValve.java	27 Feb 2004 14:58:42 -0000	1.25
  +++ StandardWrapperValve.java	4 Mar 2004 21:23:18 -0000	1.26
  @@ -64,11 +64,11 @@
       // Some JMX statistics. This vavle is associated with a StandardWrapper.
       // We exponse the StandardWrapper as JMX ( j2eeType=Servlet ). The fields
       // are here for performance.
  -    private long processingTime;
  -    private long maxTime;
  -    private long minTime = Long.MAX_VALUE;
  -    private int requestCount;
  -    private int errorCount;
  +    private volatile long processingTime;
  +    private volatile long maxTime;
  +    private volatile long minTime = Long.MAX_VALUE;
  +    private volatile int requestCount;
  +    private volatile int errorCount;
   
   
       /**
  
  
  

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