You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by se...@apache.org on 2013/08/27 20:57:21 UTC

svn commit: r1517922 - /commons/proper/jci/trunk/fam/src/main/java/org/apache/commons/jci/monitor/FilesystemAlterationMonitor.java

Author: sebb
Date: Tue Aug 27 18:57:21 2013
New Revision: 1517922

URL: http://svn.apache.org/r1517922
Log:
JCI-74 - FilesystemAlterationMonitor.delay is not safely published

Modified:
    commons/proper/jci/trunk/fam/src/main/java/org/apache/commons/jci/monitor/FilesystemAlterationMonitor.java

Modified: commons/proper/jci/trunk/fam/src/main/java/org/apache/commons/jci/monitor/FilesystemAlterationMonitor.java
URL: http://svn.apache.org/viewvc/commons/proper/jci/trunk/fam/src/main/java/org/apache/commons/jci/monitor/FilesystemAlterationMonitor.java?rev=1517922&r1=1517921&r2=1517922&view=diff
==============================================================================
--- commons/proper/jci/trunk/fam/src/main/java/org/apache/commons/jci/monitor/FilesystemAlterationMonitor.java (original)
+++ commons/proper/jci/trunk/fam/src/main/java/org/apache/commons/jci/monitor/FilesystemAlterationMonitor.java Tue Aug 27 18:57:21 2013
@@ -39,7 +39,7 @@ public final class FilesystemAlterationM
     private Map<File, FilesystemAlterationObserver> observers = Collections.unmodifiableMap(new HashMap<File, FilesystemAlterationObserver>());
     
     /** delay between calls to {@link FilesystemAlterationObserver#checkAndNotify()}, default 3000 ms */
-    private long delay = 3000;
+    private volatile long delay = 3000; // volatile because shared with daemon thread
     private Thread thread = null;
 
     private volatile boolean running = true;