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-vfs.git


The following commit(s) were added to refs/heads/master by this push:
     new f6893171 Fix Checkstyle issues
f6893171 is described below

commit f689317156633667f6962ecdbf5053a1cc7d3602
Author: Gary Gregory <[email protected]>
AuthorDate: Wed Oct 11 10:43:52 2023 -0400

    Fix Checkstyle issues
    
    Make some private classes final
---
 .../java/org/apache/commons/vfs2/impl/PrivilegedFileReplicator.java   | 4 ++--
 .../java/org/apache/commons/vfs2/provider/sftp/SftpClientFactory.java | 2 +-
 commons-vfs2/src/main/java/org/apache/commons/vfs2/tasks/VfsTask.java | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git 
a/commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/PrivilegedFileReplicator.java
 
b/commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/PrivilegedFileReplicator.java
index fcfe5e07..aac6e655 100644
--- 
a/commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/PrivilegedFileReplicator.java
+++ 
b/commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/PrivilegedFileReplicator.java
@@ -38,7 +38,7 @@ public class PrivilegedFileReplicator implements 
FileReplicator, VfsComponent {
     /**
      * An action that closes the wrapped replicator.
      */
-    private class CloseAction implements PrivilegedAction<Object> {
+    private final class CloseAction implements PrivilegedAction<Object> {
         /**
          * Performs the action.
          */
@@ -51,7 +51,7 @@ public class PrivilegedFileReplicator implements 
FileReplicator, VfsComponent {
     /**
      * An action that initializes the wrapped replicator.
      */
-    private class InitAction implements PrivilegedExceptionAction<Object> {
+    private final class InitAction implements 
PrivilegedExceptionAction<Object> {
         /**
          * Performs the action.
          */
diff --git 
a/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/sftp/SftpClientFactory.java
 
b/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/sftp/SftpClientFactory.java
index 52d8d3cf..8ffa8f91 100644
--- 
a/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/sftp/SftpClientFactory.java
+++ 
b/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/sftp/SftpClientFactory.java
@@ -51,7 +51,7 @@ public final class SftpClientFactory {
     private static final String KEY_STRICT_HOST_KEY_CHECKING = 
"StrictHostKeyChecking";
 
     /** Interface JSchLogger with JCL. */
-    private static class JSchLogger implements Logger {
+    private static final class JSchLogger implements Logger {
         @Override
         public boolean isEnabled(final int level) {
             switch (level) {
diff --git 
a/commons-vfs2/src/main/java/org/apache/commons/vfs2/tasks/VfsTask.java 
b/commons-vfs2/src/main/java/org/apache/commons/vfs2/tasks/VfsTask.java
index 77aa9780..10f1fb77 100644
--- a/commons-vfs2/src/main/java/org/apache/commons/vfs2/tasks/VfsTask.java
+++ b/commons-vfs2/src/main/java/org/apache/commons/vfs2/tasks/VfsTask.java
@@ -34,7 +34,7 @@ public class VfsTask extends Task {
     /**
      * A commons-logging wrapper for Ant logging.
      */
-    private class AntLogger implements Log {
+    private final class AntLogger implements Log {
         @Override
         public void debug(final Object o) {
             log(String.valueOf(o), Project.MSG_DEBUG);
@@ -127,7 +127,7 @@ public class VfsTask extends Task {
     /**
      * Closes the VFS manager when the project finishes.
      */
-    private class CloseListener implements SubBuildListener {
+    private final class CloseListener implements SubBuildListener {
         @Override
         public void buildFinished(final BuildEvent event) {
             closeManager();

Reply via email to