You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ec...@apache.org on 2014/11/17 23:12:14 UTC

svn commit: r1640242 - in /commons/proper/vfs/trunk: core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java src/changes/changes.xml

Author: ecki
Date: Mon Nov 17 22:12:13 2014
New Revision: 1640242

URL: http://svn.apache.org/r1640242
Log:
[VFS-142] Remove instead of null out FileContentThreadData

Modified:
    commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java
    commons/proper/vfs/trunk/src/changes/changes.xml

Modified: commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java?rev=1640242&r1=1640241&r2=1640242&view=diff
==============================================================================
--- commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java (original)
+++ commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/provider/DefaultFileContent.java Mon Nov 17 22:12:13 2014
@@ -525,7 +525,7 @@ public final class DefaultFileContent im
         }
         finally
         {
-            threadData.set(null);
+            threadData.remove();
         }
     }
 

Modified: commons/proper/vfs/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/src/changes/changes.xml?rev=1640242&r1=1640241&r2=1640242&view=diff
==============================================================================
--- commons/proper/vfs/trunk/src/changes/changes.xml (original)
+++ commons/proper/vfs/trunk/src/changes/changes.xml Mon Nov 17 22:12:13 2014
@@ -26,6 +26,9 @@
 <!--       <action issue="VFS-443" dev="ggregory" type="update" due-to="nickallen"> -->
 <!--        [Local] Need an easy way to convert from a FileObject to a File. -->
 <!--       </action> -->
+      <action issue="VFS-142" dev="ecki" type="fix" due-to="Ryan Boettcher">
+       Use ThreadLocal.remove() to clean out FileContentThreadData objects.
+      </action>
       <action issue="VFS-545" dev="ecki" type="fix">
        Make DefaultFilesCache remove reference to filesystem when it is cleared (closed).
       </action>