You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by rg...@apache.org on 2009/11/06 20:56:41 UTC
svn commit: r833539 - in /commons/proper/vfs/branches/VFS281:
core/src/main/java/org/apache/commons/vfs/provider/AbstractFileSystem.java
xdocs/changes.xml
Author: rgoers
Date: Fri Nov 6 19:56:40 2009
New Revision: 833539
URL: http://svn.apache.org/viewvc?rev=833539&view=rev
Log:
Apply patch for VFS-287
Modified:
commons/proper/vfs/branches/VFS281/core/src/main/java/org/apache/commons/vfs/provider/AbstractFileSystem.java
commons/proper/vfs/branches/VFS281/xdocs/changes.xml
Modified: commons/proper/vfs/branches/VFS281/core/src/main/java/org/apache/commons/vfs/provider/AbstractFileSystem.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/branches/VFS281/core/src/main/java/org/apache/commons/vfs/provider/AbstractFileSystem.java?rev=833539&r1=833538&r2=833539&view=diff
==============================================================================
--- commons/proper/vfs/branches/VFS281/core/src/main/java/org/apache/commons/vfs/provider/AbstractFileSystem.java (original)
+++ commons/proper/vfs/branches/VFS281/core/src/main/java/org/apache/commons/vfs/provider/AbstractFileSystem.java Fri Nov 6 19:56:40 2009
@@ -544,6 +544,10 @@
if (listeners != null)
{
listeners.remove(listener);
+ if (listeners.isEmpty())
+ {
+ listenerMap.remove(file.getName());
+ }
}
}
}
Modified: commons/proper/vfs/branches/VFS281/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/vfs/branches/VFS281/xdocs/changes.xml?rev=833539&r1=833538&r2=833539&view=diff
==============================================================================
--- commons/proper/vfs/branches/VFS281/xdocs/changes.xml (original)
+++ commons/proper/vfs/branches/VFS281/xdocs/changes.xml Fri Nov 6 19:56:40 2009
@@ -23,6 +23,9 @@
<body>
<release version="2.0" date="in SVN" description="">
+ <action dev="rgoers" type="fix" issue="VFS-287" due-to="Mircea-Eugen Ionica">
+ LocalFileName objects are not released from AbstractFileSystem.listenerMap when all listeners are removed.
+ </action>
<action dev="rgoers" type="fix" issue="VFS-216" due-to="Reetu Mutti">
The FTP Configuration includes an option to set a timeout for the data connection, but not for the socket
timeout. This is a problem, as idle sockets can cause your download to hang forever and never timeout.