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 2012/04/16 18:36:22 UTC

svn commit: r1326678 - /commons/proper/io/trunk/src/main/java/org/apache/commons/io/FileUtils.java

Author: sebb
Date: Mon Apr 16 16:36:22 2012
New Revision: 1326678

URL: http://svn.apache.org/viewvc?rev=1326678&view=rev
Log:
IO-319 Simplify; no need for boolean var.

Modified:
    commons/proper/io/trunk/src/main/java/org/apache/commons/io/FileUtils.java

Modified: commons/proper/io/trunk/src/main/java/org/apache/commons/io/FileUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/io/trunk/src/main/java/org/apache/commons/io/FileUtils.java?rev=1326678&r1=1326677&r2=1326678&view=diff
==============================================================================
--- commons/proper/io/trunk/src/main/java/org/apache/commons/io/FileUtils.java (original)
+++ commons/proper/io/trunk/src/main/java/org/apache/commons/io/FileUtils.java Mon Apr 16 16:36:22 2012
@@ -2376,14 +2376,11 @@ public class FileUtils {
             return 0L;
         }
         for (final File file : files) {
-            boolean isSymLink;
             try {
-                isSymLink = isSymlink(file);
+                if (!isSymlink(file)) {
+                    size += sizeOf(file);
+                }
             } catch (IOException ioe) {
-                isSymLink = true;
-            }
-            if (!isSymLink) {
-                size += sizeOf(file);
             }
         }