IO-319 Simplify; no need for boolean var.

sebbASF · sebbASF · commit 0458e66da140 · 2012-04-16T16:36:22.000Z
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/io/trunk@1326678 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/main/java/org/apache/commons/io/FileUtils.java b/src/main/java/org/apache/commons/io/FileUtils.java
@@ -2376,14 +2376,11 @@ public static long sizeOfDirectory(File directory) {
             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);
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -2376,14 +2376,11 @@ public static long sizeOfDirectory(File directory) {`
`2376`	`2376`	`return 0L;`
`2377`	`2377`	`}`
`2378`	`2378`	`for (final File file : files) {`
`2379`		`- boolean isSymLink;`
`2380`	`2379`	`try {`
`2381`		`- isSymLink = isSymlink(file);`
	`2380`	`+ if (!isSymlink(file)) {`
	`2381`	`+ size += sizeOf(file);`
	`2382`	`+ }`
`2382`	`2383`	`} catch (IOException ioe) {`
`2383`		`- isSymLink = true;`
`2384`		`- }`
`2385`		`- if (!isSymLink) {`
`2386`		`- size += sizeOf(file);`
`2387`	`2384`	`}`
`2388`	`2385`	`}`
`2389`	`2386`