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 2021/09/16 02:31:27 UTC

[commons-io] branch master updated: Simplify.

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-io.git


The following commit(s) were added to refs/heads/master by this push:
     new e424603  Simplify.
e424603 is described below

commit e424603cca03723933a68a4946f32e637ec2ff13
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Wed Sep 15 22:31:25 2021 -0400

    Simplify.
    
    Remove TODO inline comments I previously wrote.
---
 src/main/java/org/apache/commons/io/FileUtils.java | 17 +----------------
 1 file changed, 1 insertion(+), 16 deletions(-)

diff --git a/src/main/java/org/apache/commons/io/FileUtils.java b/src/main/java/org/apache/commons/io/FileUtils.java
index 4f62150..a504761 100644
--- a/src/main/java/org/apache/commons/io/FileUtils.java
+++ b/src/main/java/org/apache/commons/io/FileUtils.java
@@ -1427,11 +1427,7 @@ public class FileUtils {
      */
     public static void forceMkdirParent(final File file) throws IOException {
         Objects.requireNonNull(file, "file");
-        final File parent = getParentFile(file);
-        if (parent == null) {
-            return;
-        }
-        forceMkdir(parent);
+        forceMkdir(getParentFile(file));
     }
 
     /**
@@ -1660,7 +1656,6 @@ public class FileUtils {
         try {
             return PathUtils.isNewer(file.toPath(), chronoZonedDateTime);
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -1694,7 +1689,6 @@ public class FileUtils {
         try {
             return PathUtils.isNewer(file.toPath(), reference.toPath());
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -1730,7 +1724,6 @@ public class FileUtils {
         try {
             return PathUtils.isNewer(file.toPath(), instant);
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -1749,7 +1742,6 @@ public class FileUtils {
         try {
             return PathUtils.isNewer(file.toPath(), timeMillis);
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -1887,7 +1879,6 @@ public class FileUtils {
         try {
             return PathUtils.isOlder(file.toPath(), reference.toPath());
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -1922,7 +1913,6 @@ public class FileUtils {
         try {
             return PathUtils.isOlder(file.toPath(), instant);
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -1941,7 +1931,6 @@ public class FileUtils {
         try {
             return PathUtils.isOlder(file.toPath(), timeMillis);
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -2967,7 +2956,6 @@ public class FileUtils {
         try {
             return PathUtils.sizeOf(file.toPath());
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -2996,7 +2984,6 @@ public class FileUtils {
         try {
             return PathUtils.sizeOfAsBigInteger(file.toPath());
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -3020,7 +3007,6 @@ public class FileUtils {
         try {
             return PathUtils.sizeOfDirectory(directory.toPath());
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }
     }
@@ -3039,7 +3025,6 @@ public class FileUtils {
         try {
             return PathUtils.sizeOfDirectoryAsBigInteger(directory.toPath());
         } catch (final IOException e) {
-            // TODO Update method signature
             throw new UncheckedIOException(e);
         }