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 2025/04/05 01:10:32 UTC

(commons-io) branch master updated: Unnecessary @SuppressWarnings("resource")

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 403c9b1f6 Unnecessary @SuppressWarnings("resource")
403c9b1f6 is described below

commit 403c9b1f69b8a9692c687505d3d81863814b47a3
Author: Gary D. Gregory <ga...@gmail.com>
AuthorDate: Fri Apr 4 19:43:35 2025 -0400

    Unnecessary @SuppressWarnings("resource")
---
 src/main/java/org/apache/commons/io/file/PathUtils.java | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/main/java/org/apache/commons/io/file/PathUtils.java b/src/main/java/org/apache/commons/io/file/PathUtils.java
index a9099d42c..0a7e1a5f1 100644
--- a/src/main/java/org/apache/commons/io/file/PathUtils.java
+++ b/src/main/java/org/apache/commons/io/file/PathUtils.java
@@ -358,7 +358,6 @@ public static Path copyFile(final URL sourceFile, final Path targetFile, final C
      * @throws IOException if an I/O error occurs.
      * @see Files#copy(Path, Path, CopyOption...)
      */
-    @SuppressWarnings("resource") // getFileSystem() is a getter
     public static Path copyFileToDirectory(final Path sourceFile, final Path targetDirectory, final CopyOption... copyOptions) throws IOException {
         // Path.resolve() naturally won't work across FileSystem unless we convert to a String
         final Path sourceFileName = Objects.requireNonNull(sourceFile.getFileName(), "source file name");