You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2013/11/28 04:38:30 UTC
svn commit: r1546266 -
/commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/cache/SoftRefFilesCache.java
Author: ggregory
Date: Thu Nov 28 03:38:30 2013
New Revision: 1546266
URL: http://svn.apache.org/r1546266
Log:
PMD: These nested if statements could be combined.
Modified:
commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/cache/SoftRefFilesCache.java
Modified: commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/cache/SoftRefFilesCache.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/cache/SoftRefFilesCache.java?rev=1546266&r1=1546265&r2=1546266&view=diff
==============================================================================
--- commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/cache/SoftRefFilesCache.java (original)
+++ commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/cache/SoftRefFilesCache.java Thu Nov 28 03:38:30 2013
@@ -96,12 +96,9 @@ public class SoftRefFilesCache extends A
{
final FileSystemAndNameKey key = refReverseMap.get(ref);
- if (key != null)
+ if (key != null && removeFile(key))
{
- if (removeFile(key))
- {
- close(key.getFileSystem());
- }
+ close(key.getFileSystem());
}
}
finally