FileUtils.lastModifiedFileTime(File) calls Objects.requireNonNull() on

the wrong object

Author: garydgregory

src/changes/changes.xml

@@ -49,6 +49,7 @@ The <action> type attribute can be add,update,fix,remove.
     <release version="2.16.2" date="YYYY-MM-DD" description="Java 8 is required.">
       <!-- FIX -->
       <action dev="ggregory" type="fix"                due-to="Dependabot">Add missing unit tests.</action>
+      <action dev="ggregory" type="fix"                due-to="Dependabot">FileUtils.lastModifiedFileTime(File) calls Objects.requireNonNull() on the wrong object.</action>
       <!-- UPDATE -->
       <action dev="ggregory" type="update"             due-to="Dependabot">Bump tests commons.bytebuddy.version from 1.14.13 to 1.14.14 #615.</action>
       <action dev="ggregory" type="update"             due-to="Dependabot">Bump tests commons-codec:commons-codec from 1.16.1 to 1.17.0.</action>

src/main/java/org/apache/commons/io/FileUtils.java

@@ -2138,7 +2138,7 @@ public static FileTime lastModifiedFileTime(final File file) throws IOException
         // https://bugs.openjdk.java.net/browse/JDK-8177809
         // File.lastModified() is losing milliseconds (always ends in 000)
         // This bug is in OpenJDK 8 and 9, and fixed in 10.
-        return Files.getLastModifiedTime(Objects.requireNonNull(file.toPath(), "file"));
+        return Files.getLastModifiedTime(Objects.requireNonNull(file, "file").toPath());
     }
 
     /**