Author: ggregory
Date: Mon Jul 23 16:36:42 2018
New Revision: 1836505

URL: http://svn.apache.org/viewvc?rev=1836505&view=rev
Log:
Use final.

Modified:
    
commons/proper/vfs/trunk/commons-vfs2-examples/src/main/java/org/apache/commons/vfs2/example/Shell.java
    
commons/proper/vfs/trunk/commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/StandardFileSystemManager.java
    
commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/DefaultFileContentTest.java

Modified: 
commons/proper/vfs/trunk/commons-vfs2-examples/src/main/java/org/apache/commons/vfs2/example/Shell.java
URL: 
http://svn.apache.org/viewvc/commons/proper/vfs/trunk/commons-vfs2-examples/src/main/java/org/apache/commons/vfs2/example/Shell.java?rev=1836505&r1=1836504&r2=1836505&view=diff
==============================================================================
--- 
commons/proper/vfs/trunk/commons-vfs2-examples/src/main/java/org/apache/commons/vfs2/example/Shell.java
 (original)
+++ 
commons/proper/vfs/trunk/commons-vfs2-examples/src/main/java/org/apache/commons/vfs2/example/Shell.java
 Mon Jul 23 16:36:42 2018
@@ -116,17 +116,17 @@ public final class Shell {
         }
     }
 
-    private void info(String[] cmd) throws Exception {
+    private void info(final String[] cmd) throws Exception {
         if (cmd.length > 1) {
             info(cmd[1]);
         } else {
             System.out.println(
                     "Default manager: \"" + mgr.getClass().getName() + "\" " + 
"version " + getVersion(mgr.getClass()));
-            String[] schemes = mgr.getSchemes();
-            List<String> virtual = new ArrayList<>();
-            List<String> physical = new ArrayList<>();
+            final String[] schemes = mgr.getSchemes();
+            final List<String> virtual = new ArrayList<>();
+            final List<String> physical = new ArrayList<>();
             for (int i = 0; i < schemes.length; i++) {
-                Collection<Capability> caps = 
mgr.getProviderCapabilities(schemes[i]);
+                final Collection<Capability> caps = 
mgr.getProviderCapabilities(schemes[i]);
                 if (caps != null) {
                     if (caps.contains(Capability.VIRTUAL) || 
caps.contains(Capability.COMPRESS)
                             || caps.contains(Capability.DISPATCHER)) {
@@ -145,14 +145,14 @@ public final class Shell {
         }
     }
 
-    private void info(String scheme) throws Exception {
+    private void info(final String scheme) throws Exception {
         System.out.println("Provider Info for scheme \"" + scheme + "\":");
         Collection<Capability> caps;
         caps = mgr.getProviderCapabilities(scheme);
         if (caps != null && !caps.isEmpty()) {
             System.out.println("  capabilities: " + caps);
         }
-        FileOperationProvider[] ops = mgr.getOperationProviders(scheme);
+        final FileOperationProvider[] ops = mgr.getOperationProviders(scheme);
         if (ops != null && ops.length > 0) {
             System.out.println("  operations: " + ops);
         }
@@ -334,10 +334,10 @@ public final class Shell {
         return cmd.toArray(new String[cmd.size()]);
     }
 
-    private static String getVersion(Class<?> cls) {
+    private static String getVersion(final Class<?> cls) {
         try {
             return cls.getPackage().getImplementationVersion();
-        } catch (Exception ignored) {
+        } catch (final Exception ignored) {
             return "N/A";
         }
     }

Modified: 
commons/proper/vfs/trunk/commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/StandardFileSystemManager.java
URL: 
http://svn.apache.org/viewvc/commons/proper/vfs/trunk/commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/StandardFileSystemManager.java?rev=1836505&r1=1836504&r2=1836505&view=diff
==============================================================================
--- 
commons/proper/vfs/trunk/commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/StandardFileSystemManager.java
 (original)
+++ 
commons/proper/vfs/trunk/commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/StandardFileSystemManager.java
 Mon Jul 23 16:36:42 2018
@@ -139,7 +139,7 @@ public class StandardFileSystemManager e
         if (classLoader != null) {
             return classLoader;
         }
-        ClassLoader cl = Thread.currentThread().getContextClassLoader();
+        final ClassLoader cl = Thread.currentThread().getContextClassLoader();
         if (cl != null) {
             return cl;
         }

Modified: 
commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/DefaultFileContentTest.java
URL: 
http://svn.apache.org/viewvc/commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/DefaultFileContentTest.java?rev=1836505&r1=1836504&r2=1836505&view=diff
==============================================================================
--- 
commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/DefaultFileContentTest.java
 (original)
+++ 
commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/DefaultFileContentTest.java
 Mon Jul 23 16:36:42 2018
@@ -19,8 +19,8 @@ public class DefaultFileContentTest {
 
     @Test
     public void testMarkingWorks() throws Exception {
-        File temp = File.createTempFile("temp-file-name", ".tmp");
-        FileSystemManager fileSystemManager = VFS.getManager();
+        final File temp = File.createTempFile("temp-file-name", ".tmp");
+        final FileSystemManager fileSystemManager = VFS.getManager();
 
         try (FileObject file = 
fileSystemManager.resolveFile(temp.getAbsolutePath())) {
             try (OutputStream outputStream = 
file.getContent().getOutputStream()) {
@@ -31,7 +31,7 @@ public class DefaultFileContentTest {
                 if (stream.markSupported()) {
                     for (int i = 0; i < 10; i++) {
                         stream.mark(0);
-                        byte[] data = new byte[100];
+                        final byte[] data = new byte[100];
                         stream.read(data, 0, 7);
                         Assert.assertEquals(expected, new String(data).trim());
                         stream.reset();
@@ -43,8 +43,8 @@ public class DefaultFileContentTest {
 
     @Test
     public void testMarkingWhenReadingEOS() throws Exception {
-        File temp = File.createTempFile("temp-file-name", ".tmp");
-        FileSystemManager fileSystemManager = VFS.getManager();
+        final File temp = File.createTempFile("temp-file-name", ".tmp");
+        final FileSystemManager fileSystemManager = VFS.getManager();
 
         try (FileObject file = 
fileSystemManager.resolveFile(temp.getAbsolutePath())) {
             try (OutputStream outputStream = 
file.getContent().getOutputStream()) {
@@ -56,7 +56,7 @@ public class DefaultFileContentTest {
                 if (stream.markSupported()) {
                     for (int i = 0; i < 10; i++) {
                         stream.mark(0);
-                        byte[] data = new byte[100];
+                        final byte[] data = new byte[100];
                         readCount = stream.read(data, 0, 7);
                         Assert.assertEquals(readCount, 7);
                         Assert.assertEquals(expected, new String(data).trim());


Reply via email to