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 2020/09/15 14:16:02 UTC

[commons-vfs] branch master updated: Javadoc and better param name.

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 75c45ec  Javadoc and better param name.
75c45ec is described below

commit 75c45ec8b78941d42e76b64fc7d13f91aeccae35
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Tue Sep 15 10:15:53 2020 -0400

    Javadoc and better param name.
---
 .../commons/vfs2/FileSystemConfigBuilder.java      | 228 +++++++++++----------
 .../vfs2/provider/temp/TemporaryFileProvider.java  |   8 +-
 2 files changed, 120 insertions(+), 116 deletions(-)

diff --git a/commons-vfs2/src/main/java/org/apache/commons/vfs2/FileSystemConfigBuilder.java b/commons-vfs2/src/main/java/org/apache/commons/vfs2/FileSystemConfigBuilder.java
index 757f0ac..3925bfa 100644
--- a/commons-vfs2/src/main/java/org/apache/commons/vfs2/FileSystemConfigBuilder.java
+++ b/commons-vfs2/src/main/java/org/apache/commons/vfs2/FileSystemConfigBuilder.java
@@ -53,21 +53,21 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets named option as boolean.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getBoolean(FileSystemOptions, String, Boolean)
      *
      * @since 2.0
      */
-    protected Boolean getBoolean(final FileSystemOptions opts, final String name) {
-        return getBoolean(opts, name, null);
+    protected Boolean getBoolean(final FileSystemOptions fileSystemOptions, final String name) {
+        return getBoolean(fileSystemOptions, name, null);
     }
 
     /**
      * Gets named option as boolean.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -75,14 +75,15 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected boolean getBoolean(final FileSystemOptions opts, final String name, final boolean defaultValue) {
-        return getBoolean(opts, name, Boolean.valueOf(defaultValue)).booleanValue();
+    protected boolean getBoolean(final FileSystemOptions fileSystemOptions, final String name,
+        final boolean defaultValue) {
+        return getBoolean(fileSystemOptions, name, Boolean.valueOf(defaultValue)).booleanValue();
     }
 
     /**
      * Gets named option as boolean.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -90,8 +91,9 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected Boolean getBoolean(final FileSystemOptions opts, final String name, final Boolean defaultValue) {
-        Boolean value = (Boolean) getParam(opts, name);
+    protected Boolean getBoolean(final FileSystemOptions fileSystemOptions, final String name,
+        final Boolean defaultValue) {
+        Boolean value = (Boolean) getParam(fileSystemOptions, name);
         if (value == null) {
             final String str = getProperty(name);
             if (str == null) {
@@ -105,21 +107,21 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets named option as byte.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getByte(FileSystemOptions, String, Byte)
      *
      * @since 2.0
      */
-    protected Byte getByte(final FileSystemOptions opts, final String name) {
-        return getByte(opts, name, null);
+    protected Byte getByte(final FileSystemOptions fileSystemOptions, final String name) {
+        return getByte(fileSystemOptions, name, null);
     }
 
     /**
      * Gets named option as byte.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -127,22 +129,22 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected byte getByte(final FileSystemOptions opts, final String name, final byte defaultValue) {
-        return getByte(opts, name, Byte.valueOf(defaultValue)).byteValue();
+    protected byte getByte(final FileSystemOptions fileSystemOptions, final String name, final byte defaultValue) {
+        return getByte(fileSystemOptions, name, Byte.valueOf(defaultValue)).byteValue();
     }
 
     /**
      * Gets named option as byte.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
      *
      * @since 2.0
      */
-    protected Byte getByte(final FileSystemOptions opts, final String name, final Byte defaultValue) {
-        Byte value = (Byte) getParam(opts, name);
+    protected Byte getByte(final FileSystemOptions fileSystemOptions, final String name, final Byte defaultValue) {
+        Byte value = (Byte) getParam(fileSystemOptions, name);
         if (value == null) {
             final String str = getProperty(name);
             if (str == null) {
@@ -156,21 +158,21 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets named option as character.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getCharacter(FileSystemOptions, String, Character)
      *
      * @since 2.0
      */
-    protected Character getCharacter(final FileSystemOptions opts, final String name) {
-        return getCharacter(opts, name, null);
+    protected Character getCharacter(final FileSystemOptions fileSystemOptions, final String name) {
+        return getCharacter(fileSystemOptions, name, null);
     }
 
     /**
      * Gets named option as character.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -178,22 +180,23 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected char getCharacter(final FileSystemOptions opts, final String name, final char defaultValue) {
-        return getCharacter(opts, name, new Character(defaultValue)).charValue();
+    protected char getCharacter(final FileSystemOptions fileSystemOptions, final String name, final char defaultValue) {
+        return getCharacter(fileSystemOptions, name, new Character(defaultValue)).charValue();
     }
 
     /**
      * Gets named option as character.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
      *
      * @since 2.0
      */
-    protected Character getCharacter(final FileSystemOptions opts, final String name, final Character defaultValue) {
-        Character value = (Character) getParam(opts, name);
+    protected Character getCharacter(final FileSystemOptions fileSystemOptions, final String name,
+        final Character defaultValue) {
+        Character value = (Character) getParam(fileSystemOptions, name);
         if (value == null) {
             final String str = getProperty(name);
             if (str == null || str.length() <= 0) {
@@ -216,21 +219,21 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets named option as double.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getDouble(FileSystemOptions, String, Double)
      *
      * @since 2.0
      */
-    protected Double getDouble(final FileSystemOptions opts, final String name) {
-        return getDouble(opts, name, null);
+    protected Double getDouble(final FileSystemOptions fileSystemOptions, final String name) {
+        return getDouble(fileSystemOptions, name, null);
     }
 
     /**
      * Gets named option as double.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -238,22 +241,24 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected double getDouble(final FileSystemOptions opts, final String name, final double defaultValue) {
-        return getDouble(opts, name, new Double(defaultValue)).doubleValue();
+    protected double getDouble(final FileSystemOptions fileSystemOptions, final String name,
+        final double defaultValue) {
+        return getDouble(fileSystemOptions, name, new Double(defaultValue)).doubleValue();
     }
 
     /**
      * Gets named option as double.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
      *
      * @since 2.0
      */
-    protected Double getDouble(final FileSystemOptions opts, final String name, final Double defaultValue) {
-        Double value = (Double) getParam(opts, name);
+    protected Double getDouble(final FileSystemOptions fileSystemOptions, final String name,
+        final Double defaultValue) {
+        Double value = (Double) getParam(fileSystemOptions, name);
         if (value == null) {
             final String str = getProperty(name);
             if (str == null || str.length() <= 0) {
@@ -269,7 +274,7 @@ public abstract class FileSystemConfigBuilder {
      *
      * @param <E> enumeration type
      * @param enumClass class of enumeration type
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name *
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getEnum(Class, FileSystemOptions, String, Enum)
@@ -277,8 +282,9 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.1
      */
-    protected <E extends Enum<E>> E getEnum(final Class<E> enumClass, final FileSystemOptions opts, final String name) {
-        return this.<E>getEnum(enumClass, opts, name, null);
+    protected <E extends Enum<E>> E getEnum(final Class<E> enumClass, final FileSystemOptions fileSystemOptions,
+        final String name) {
+        return this.<E>getEnum(enumClass, fileSystemOptions, name, null);
     }
 
     /**
@@ -286,7 +292,7 @@ public abstract class FileSystemConfigBuilder {
      *
      * @param <E> enumeration type
      * @param enumClass class of enumeration type
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -295,10 +301,10 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.1
      */
-    protected <E extends Enum<E>> E getEnum(final Class<E> enumClass, final FileSystemOptions opts, final String name,
-            final E defaultValue) {
+    protected <E extends Enum<E>> E getEnum(final Class<E> enumClass, final FileSystemOptions fileSystemOptions,
+        final String name, final E defaultValue) {
         @SuppressWarnings("unchecked")
-        E value = (E) getParam(opts, name);
+        E value = (E) getParam(fileSystemOptions, name);
         if (value == null) {
             final String str = getProperty(name);
             if (str == null) {
@@ -312,7 +318,7 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets named option as float.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getFloat(FileSystemOptions, String, Float)
@@ -320,14 +326,14 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected Float getFloat(final FileSystemOptions opts, final String name) {
-        return getFloat(opts, name, null);
+    protected Float getFloat(final FileSystemOptions fileSystemOptions, final String name) {
+        return getFloat(fileSystemOptions, name, null);
     }
 
     /**
      * Gets named option as float.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -336,14 +342,14 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected float getFloat(final FileSystemOptions opts, final String name, final float defaultValue) {
-        return getFloat(opts, name, new Float(defaultValue)).floatValue();
+    protected float getFloat(final FileSystemOptions fileSystemOptions, final String name, final float defaultValue) {
+        return getFloat(fileSystemOptions, name, new Float(defaultValue)).floatValue();
     }
 
     /**
      * Gets named option as float.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -351,8 +357,8 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected Float getFloat(final FileSystemOptions opts, final String name, final Float defaultValue) {
-        Float value = (Float) getParam(opts, name);
+    protected Float getFloat(final FileSystemOptions fileSystemOptions, final String name, final Float defaultValue) {
+        Float value = (Float) getParam(fileSystemOptions, name);
         if (value == null) {
             final String str = getProperty(name);
             if (str == null || str.length() <= 0) {
@@ -366,7 +372,7 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets named option as integer.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getInteger(FileSystemOptions, String, Integer)
@@ -374,14 +380,14 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected Integer getInteger(final FileSystemOptions opts, final String name) {
-        return getInteger(opts, name, null);
+    protected Integer getInteger(final FileSystemOptions fileSystemOptions, final String name) {
+        return getInteger(fileSystemOptions, name, null);
     }
 
     /**
      * Gets named option as integer.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -390,14 +396,14 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected int getInteger(final FileSystemOptions opts, final String name, final int defaultValue) {
-        return getInteger(opts, name, Integer.valueOf(defaultValue)).intValue();
+    protected int getInteger(final FileSystemOptions fileSystemOptions, final String name, final int defaultValue) {
+        return getInteger(fileSystemOptions, name, Integer.valueOf(defaultValue)).intValue();
     }
 
     /**
      * Gets named option as integer.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -405,8 +411,9 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected Integer getInteger(final FileSystemOptions opts, final String name, final Integer defaultValue) {
-        Integer value = (Integer) getParam(opts, name);
+    protected Integer getInteger(final FileSystemOptions fileSystemOptions, final String name,
+        final Integer defaultValue) {
+        Integer value = (Integer) getParam(fileSystemOptions, name);
         if (value == null) {
             final String str = getProperty(name);
             if (str == null) {
@@ -420,7 +427,7 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets named option as long.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getLong(FileSystemOptions, String, Long)
@@ -428,14 +435,14 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected Long getLong(final FileSystemOptions opts, final String name) {
-        return getLong(opts, name, null);
+    protected Long getLong(final FileSystemOptions fileSystemOptions, final String name) {
+        return getLong(fileSystemOptions, name, null);
     }
 
     /**
      * Gets named option as long.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -444,14 +451,14 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected long getLong(final FileSystemOptions opts, final String name, final long defaultValue) {
-        return getLong(opts, name, Long.valueOf(defaultValue)).longValue();
+    protected long getLong(final FileSystemOptions fileSystemOptions, final String name, final long defaultValue) {
+        return getLong(fileSystemOptions, name, Long.valueOf(defaultValue)).longValue();
     }
 
     /**
      * Gets named option as long.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -459,8 +466,8 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected Long getLong(final FileSystemOptions opts, final String name, final Long defaultValue) {
-        Long value = (Long) getParam(opts, name);
+    protected Long getLong(final FileSystemOptions fileSystemOptions, final String name, final Long defaultValue) {
+        Long value = (Long) getParam(fileSystemOptions, name);
         if (value == null) {
             final String str = getProperty(name);
             if (str == null) {
@@ -474,18 +481,14 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets named parameter.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name get option with this name
      * @return the named option or null
      *
      * @since 1.0
      */
-    protected Object getParam(final FileSystemOptions opts, final String name) {
-        if (opts == null) {
-            return null;
-        }
-
-        return opts.getOption(getConfigClass(), name);
+    protected Object getParam(final FileSystemOptions fileSystemOptions, final String name) {
+        return fileSystemOptions == null ? null : fileSystemOptions.getOption(getConfigClass(), name);
     }
 
     /**
@@ -503,19 +506,19 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets the root URI of the file system.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @return The root URI
      *
      * @since 2.0
      */
-    public String getRootURI(final FileSystemOptions opts) {
-        return getString(opts, ROOTURI);
+    public String getRootURI(final FileSystemOptions fileSystemOptions) {
+        return getString(fileSystemOptions, ROOTURI);
     }
 
     /**
      * Gets named option as short.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getShort(FileSystemOptions, String, Short)
@@ -523,14 +526,14 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected Short getShort(final FileSystemOptions opts, final String name) {
-        return getShort(opts, name, null);
+    protected Short getShort(final FileSystemOptions fileSystemOptions, final String name) {
+        return getShort(fileSystemOptions, name, null);
     }
 
     /**
      * Gets named option as short.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -539,14 +542,14 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected short getShort(final FileSystemOptions opts, final String name, final short defaultValue) {
-        return getShort(opts, name, Short.valueOf(defaultValue)).shortValue();
+    protected short getShort(final FileSystemOptions fileSystemOptions, final String name, final short defaultValue) {
+        return getShort(fileSystemOptions, name, Short.valueOf(defaultValue)).shortValue();
     }
 
     /**
      * Gets named option as short.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
@@ -554,8 +557,8 @@ public abstract class FileSystemConfigBuilder {
      *
      * @since 2.0
      */
-    protected Short getShort(final FileSystemOptions opts, final String name, final Short defaultValue) {
-        Short value = (Short) getParam(opts, name);
+    protected Short getShort(final FileSystemOptions fileSystemOptions, final String name, final Short defaultValue) {
+        Short value = (Short) getParam(fileSystemOptions, name);
         if (value == null) {
             final String str = getProperty(name);
             if (str == null) {
@@ -569,29 +572,30 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Gets named option as String.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @return the option in {@code opts} or system properties, otherwise null
      * @see #getString(FileSystemOptions, String, String)
      *
      * @since 2.0
      */
-    protected String getString(final FileSystemOptions opts, final String name) {
-        return getString(opts, name, null);
+    protected String getString(final FileSystemOptions fileSystemOptions, final String name) {
+        return getString(fileSystemOptions, name, null);
     }
 
     /**
      * Gets named option as String.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option name
      * @param defaultValue value to return if option is not present
      * @return the option in {@code opts} or system properties, otherwise {@code defaultValue}
      *
      * @since 2.0
      */
-    protected String getString(final FileSystemOptions opts, final String name, final String defaultValue) {
-        String value = (String) getParam(opts, name);
+    protected String getString(final FileSystemOptions fileSystemOptions, final String name,
+        final String defaultValue) {
+        String value = (String) getParam(fileSystemOptions, name);
         if (value == null) {
             value = getProperty(name);
             if (value == null) {
@@ -604,65 +608,65 @@ public abstract class FileSystemConfigBuilder {
     /**
      * Checks the named setting specified.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the option to check in {@code opts} or system properties
      * @return true if option exists
      *
      * @since 2.0
      */
-    protected boolean hasObject(final FileSystemOptions opts, final String name) {
-        return hasParam(opts, name) || System.getProperties().containsKey(toPropertyKey(name));
+    protected boolean hasObject(final FileSystemOptions fileSystemOptions, final String name) {
+        return hasParam(fileSystemOptions, name) || System.getProperties().containsKey(toPropertyKey(name));
     }
 
     /**
      * Checks if option exists.
      *
-     * @param opts file system options to work with
+     * @param fileSystemOptions file system options to query, may be null.
      * @param name the name to look up in {@code opts}
      * @return true if opts have the named parameter
      *
      * @since 1.0
      */
-    protected boolean hasParam(final FileSystemOptions opts, final String name) {
-        return opts != null && opts.hasOption(getConfigClass(), name);
+    protected boolean hasParam(final FileSystemOptions fileSystemOptions, final String name) {
+        return fileSystemOptions != null && fileSystemOptions.hasOption(getConfigClass(), name);
     }
 
     /**
-     * Sets named parameter.
+     * Sets the named parameter.
      *
-     * @param opts the file system options to modify
+     * @param fileSystemOptions the file system options to modify
      * @param name set option with this name
      * @param value boolean value to set
      *
      * @since 2.1
      */
-    protected void setParam(final FileSystemOptions opts, final String name, final boolean value) {
-        setParam(opts, name, Boolean.valueOf(value));
+    protected void setParam(final FileSystemOptions fileSystemOptions, final String name, final boolean value) {
+        setParam(fileSystemOptions, name, Boolean.valueOf(value));
     }
 
     /**
-     * Sets named parameter.
+     * Sets the named parameter.
      *
-     * @param opts the file system options to modify
+     * @param fileSystemOptions the file system options to modify
      * @param name set option with this name
      * @param value object value to set
      *
      * @since 1.0
      */
-    protected void setParam(final FileSystemOptions opts, final String name, final Object value) {
-        opts.setOption(getConfigClass(), name, value);
+    protected void setParam(final FileSystemOptions fileSystemOptions, final String name, final Object value) {
+        fileSystemOptions.setOption(getConfigClass(), name, value);
     }
 
     /**
      * Sets the root URI of the file system.
      *
-     * @param opts the file system options to modify
+     * @param fileSystemOptions the file system options to modify
      * @param rootURI The creator name to be associated with the file.
      *
      * @since 2.0
      */
-    public void setRootURI(final FileSystemOptions opts, final String rootURI) {
-        setParam(opts, ROOTURI, rootURI);
+    public void setRootURI(final FileSystemOptions fileSystemOptions, final String rootURI) {
+        setParam(fileSystemOptions, ROOTURI, rootURI);
     }
 
     /**
diff --git a/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/temp/TemporaryFileProvider.java b/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/temp/TemporaryFileProvider.java
index eb29457..f03ae01 100644
--- a/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/temp/TemporaryFileProvider.java
+++ b/commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/temp/TemporaryFileProvider.java
@@ -73,13 +73,13 @@ public class TemporaryFileProvider extends AbstractFileProvider implements Compa
      *
      * @param baseFile The base FileObject.
      * @param uri The URI of the file to be located.
-     * @param properties FileSystemOptions to use to locate or create the file.
+     * @param fileSystemOptions FileSystemOptions to use to locate or create the file.
      * @return The FileObject.
      * @throws FileSystemException if an error occurs.
      */
     @Override
     public synchronized FileObject findFile(final FileObject baseFile, final String uri,
-            final FileSystemOptions properties) throws FileSystemException {
+            final FileSystemOptions fileSystemOptions) throws FileSystemException {
         // Parse the name
         final StringBuilder buffer = new StringBuilder(uri);
         final String scheme = UriParser.extractScheme(getContext().getFileSystemManager().getSchemes(), uri, buffer);
@@ -89,7 +89,7 @@ public class TemporaryFileProvider extends AbstractFileProvider implements Compa
 
         // Create the temp file system if it does not exist
         // FileSystem filesystem = findFileSystem( this, (Properties) null);
-        FileSystem filesystem = findFileSystem(this, properties);
+        FileSystem filesystem = findFileSystem(this, fileSystemOptions);
         if (filesystem == null) {
             if (rootFile == null) {
                 rootFile = getContext().getTemporaryFileStore().allocateFile("tempfs");
@@ -97,7 +97,7 @@ public class TemporaryFileProvider extends AbstractFileProvider implements Compa
             final FileName rootName = getContext().parseURI(scheme + ":" + FileName.ROOT_PATH);
             // final FileName rootName =
             // new LocalFileName(scheme, scheme + ":", FileName.ROOT_PATH);
-            filesystem = new LocalFileSystem(rootName, rootFile.getAbsolutePath(), properties);
+            filesystem = new LocalFileSystem(rootName, rootFile.getAbsolutePath(), fileSystemOptions);
             addFileSystem(this, filesystem);
         }