svn commit: r345108 - /jakarta/commons/proper/pool/branches/1_3_RELEASE_BRANCH/src/java/org/apache/commons/pool/impl/GenericObjectPool.java

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r345108 - /jakarta/commons/proper/pool/branches/1_3_RELEASE_BRANCH/src/java/org/apache/commons/pool/impl/GenericObjectPool.java

rdonkin
Author: rdonkin
Date: Wed Nov 16 12:35:11 2005
New Revision: 345108

URL: http://svn.apache.org/viewcvs?rev=345108&view=rev
Log:
Variable may be accessed by multiple threads without synchronization and should therefore be declared volatile. Submitted by Sandy McArthur. Issue #37321.

Modified:
    jakarta/commons/proper/pool/branches/1_3_RELEASE_BRANCH/src/java/org/apache/commons/pool/impl/GenericObjectPool.java

Modified: jakarta/commons/proper/pool/branches/1_3_RELEASE_BRANCH/src/java/org/apache/commons/pool/impl/GenericObjectPool.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/pool/branches/1_3_RELEASE_BRANCH/src/java/org/apache/commons/pool/impl/GenericObjectPool.java?rev=345108&r1=345107&r2=345108&view=diff
==============================================================================
--- jakarta/commons/proper/pool/branches/1_3_RELEASE_BRANCH/src/java/org/apache/commons/pool/impl/GenericObjectPool.java (original)
+++ jakarta/commons/proper/pool/branches/1_3_RELEASE_BRANCH/src/java/org/apache/commons/pool/impl/GenericObjectPool.java Wed Nov 16 12:35:11 2005
@@ -1118,7 +1118,7 @@
      * @see #setTimeBetweenEvictionRunsMillis
      */
     class Evictor implements Runnable {
-        private boolean _cancelled = false;
+        private volatile boolean _cancelled = false;
         private long _delay = 0L;
 
         public Evictor(long delay) {



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]