[java-support] 04/10: JPAR-85 - Add missing final parameters to constructors.

Tom Zeller tzeller at dragonacea.biz
Mon May 16 21:19:39 EDT 2016


This is an automated email from the git hooks/post-receive script.

tzeller pushed a commit to branch JPAR-85
in repository java-support.

commit 6c803009bfdd9721f83475fe150843f24fe7978c
Author: Tom Zeller <tzeller at dragonacea.biz>
AuthorDate: Mon May 16 19:58:42 2016 -0500

    JPAR-85 - Add missing final parameters to constructors.
---
 .../utilities/java/support/collection/IndexingObjectStore.java      | 2 +-
 .../java/net/shibboleth/utilities/java/support/collection/Pair.java | 2 +-
 .../java/support/httpclient/FileCachingHttpClientBuilder.java       | 2 +-
 .../utilities/java/support/httpclient/HttpClientBuilder.java        | 2 +-
 .../utilities/java/support/logic/TransformAndCheckFunction.java     | 4 ++--
 .../net/shibboleth/utilities/java/support/resolver/CriteriaSet.java | 2 +-
 .../utilities/java/support/resolver/ResolverException.java          | 6 +++---
 .../java/support/security/BasicKeystoreKeyStrategyTool.java         | 2 +-
 .../support/security/SecureRandomIdentifierGenerationStrategy.java  | 2 +-
 .../java/support/security/SelfSignedCertificateGenerator.java       | 2 +-
 .../net/shibboleth/utilities/java/support/velocity/Template.java    | 4 ++--
 .../shibboleth/utilities/java/support/xml/ClasspathResolver.java    | 2 +-
 .../net/shibboleth/utilities/java/support/xml/SchemaBuilder.java    | 2 +-
 13 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/src/main/java/net/shibboleth/utilities/java/support/collection/IndexingObjectStore.java b/src/main/java/net/shibboleth/utilities/java/support/collection/IndexingObjectStore.java
index ee2688d..795cf62 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/collection/IndexingObjectStore.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/collection/IndexingObjectStore.java
@@ -280,7 +280,7 @@ public class IndexingObjectStore<T> {
          * 
          * @param wrappedObject the object being wrapped
          */
-        public StoredObjectWrapper(T wrappedObject) {
+        public StoredObjectWrapper(final T wrappedObject) {
             object = wrappedObject;
             referenceCount = 0;
         }
diff --git a/src/main/java/net/shibboleth/utilities/java/support/collection/Pair.java b/src/main/java/net/shibboleth/utilities/java/support/collection/Pair.java
index 0d6d3d1..800d4f1 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/collection/Pair.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/collection/Pair.java
@@ -60,7 +60,7 @@ public class Pair<T1, T2> {
      * 
      * @param pair pair to be copied
      */
-    public Pair(@Nonnull Pair<? extends T1, ? extends T2> pair) {
+    public Pair(@Nonnull final Pair<? extends T1, ? extends T2> pair) {
         Constraint.isNotNull(pair, "Pair to be copied can not be null");
         first = pair.getFirst();
         second = pair.getSecond();
diff --git a/src/main/java/net/shibboleth/utilities/java/support/httpclient/FileCachingHttpClientBuilder.java b/src/main/java/net/shibboleth/utilities/java/support/httpclient/FileCachingHttpClientBuilder.java
index 96f41a8..f1f021c 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/httpclient/FileCachingHttpClientBuilder.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/httpclient/FileCachingHttpClientBuilder.java
@@ -296,7 +296,7 @@ public class FileCachingHttpClientBuilder extends HttpClientBuilder {
          * @param taskInterval the interval at which storage maintenance should run
          */
         public StorageManagingHttpClient(@Nonnull final CloseableHttpClient wrappedClient, 
-                @Nonnull final ManagedHttpCacheStorage managedStorage, long taskInterval)  {
+                @Nonnull final ManagedHttpCacheStorage managedStorage, final long taskInterval)  {
            super(); 
            httpClient = Constraint.isNotNull(wrappedClient, "HttpClient was null");
            storage = Constraint.isNotNull(managedStorage, "ManagedHttpCacheStorage was null");
diff --git a/src/main/java/net/shibboleth/utilities/java/support/httpclient/HttpClientBuilder.java b/src/main/java/net/shibboleth/utilities/java/support/httpclient/HttpClientBuilder.java
index bf74ed3..5d35999 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/httpclient/HttpClientBuilder.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/httpclient/HttpClientBuilder.java
@@ -244,7 +244,7 @@ public class HttpClientBuilder {
      * 
      * @param builder the Apache HttpClientBuilder 4.3+ instance over which to layer this builder
      */
-    public HttpClientBuilder(@Nonnull org.apache.http.impl.client.HttpClientBuilder builder) {
+    public HttpClientBuilder(@Nonnull final org.apache.http.impl.client.HttpClientBuilder builder) {
         Constraint.isNotNull(builder, "Apache HttpClientBuilder may not be null");
         apacheBuilder = builder;
         resetDefaults();
diff --git a/src/main/java/net/shibboleth/utilities/java/support/logic/TransformAndCheckFunction.java b/src/main/java/net/shibboleth/utilities/java/support/logic/TransformAndCheckFunction.java
index 432842f..c4f4817 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/logic/TransformAndCheckFunction.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/logic/TransformAndCheckFunction.java
@@ -54,8 +54,8 @@ public class TransformAndCheckFunction<T> implements Function<T, Optional<? exte
      * @param failOnInputConstraintViolation whether input that does not meet the constraint should cause an error or
      *            just be ignored
      */
-    public TransformAndCheckFunction(@Nonnull Function<T, ? extends T> inputPreprocessor,
-            @Nonnull Predicate<T> inputConstraint, boolean failOnInputConstraintViolation) {
+    public TransformAndCheckFunction(@Nonnull final Function<T, ? extends T> inputPreprocessor,
+            @Nonnull final Predicate<T> inputConstraint, final boolean failOnInputConstraintViolation) {
         preprocessor = Constraint.isNotNull(inputPreprocessor, "Input preprocessor can not be null");
         constraint = Constraint.isNotNull(inputConstraint, "Input constraint can not be null");
         failOnConstraintViolation = failOnInputConstraintViolation;
diff --git a/src/main/java/net/shibboleth/utilities/java/support/resolver/CriteriaSet.java b/src/main/java/net/shibboleth/utilities/java/support/resolver/CriteriaSet.java
index e4c782a..c39aecd 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/resolver/CriteriaSet.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/resolver/CriteriaSet.java
@@ -34,7 +34,7 @@ public class CriteriaSet extends ClassIndexedSet<Criterion> implements Criterion
      * 
      * @param criteria criteria to add, may be null or contain null values
      */
-    public CriteriaSet(@Nullable Criterion... criteria) {
+    public CriteriaSet(@Nullable final Criterion... criteria) {
         super();
 
         if (criteria == null || criteria.length == 0) {
diff --git a/src/main/java/net/shibboleth/utilities/java/support/resolver/ResolverException.java b/src/main/java/net/shibboleth/utilities/java/support/resolver/ResolverException.java
index c814ae1..9389e83 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/resolver/ResolverException.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/resolver/ResolverException.java
@@ -39,7 +39,7 @@ public class ResolverException extends Exception {
      * 
      * @param message exception message
      */
-    public ResolverException(@Nullable String message) {
+    public ResolverException(@Nullable final String message) {
         super(message);
     }
 
@@ -48,7 +48,7 @@ public class ResolverException extends Exception {
      * 
      * @param wrappedException exception to be wrapped by this one
      */
-    public ResolverException(@Nullable Exception wrappedException) {
+    public ResolverException(@Nullable final Exception wrappedException) {
         super(wrappedException);
     }
 
@@ -58,7 +58,7 @@ public class ResolverException extends Exception {
      * @param message exception message
      * @param wrappedException exception to be wrapped by this one
      */
-    public ResolverException(@Nullable String message, @Nullable Exception wrappedException) {
+    public ResolverException(@Nullable final String message, @Nullable final Exception wrappedException) {
         super(message, wrappedException);
     }
 }
\ No newline at end of file
diff --git a/src/main/java/net/shibboleth/utilities/java/support/security/BasicKeystoreKeyStrategyTool.java b/src/main/java/net/shibboleth/utilities/java/support/security/BasicKeystoreKeyStrategyTool.java
index 90b6ab6..b78555a 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/security/BasicKeystoreKeyStrategyTool.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/security/BasicKeystoreKeyStrategyTool.java
@@ -235,7 +235,7 @@ public class BasicKeystoreKeyStrategyTool {
          *
          * @param optionName 
          */
-        public FileConverter(String optionName) {
+        public FileConverter(final String optionName) {
             super(optionName);
         }
 
diff --git a/src/main/java/net/shibboleth/utilities/java/support/security/SecureRandomIdentifierGenerationStrategy.java b/src/main/java/net/shibboleth/utilities/java/support/security/SecureRandomIdentifierGenerationStrategy.java
index 372678d..ebfdb4e 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/security/SecureRandomIdentifierGenerationStrategy.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/security/SecureRandomIdentifierGenerationStrategy.java
@@ -54,7 +54,7 @@ public class SecureRandomIdentifierGenerationStrategy extends RandomIdentifierGe
      * @param identifierSize number of random bytes in the identifier
      * @param identifierEncoder encoder used to convert random bytes to string identifier
      */
-    public SecureRandomIdentifierGenerationStrategy(@Nonnull final SecureRandom source, int identifierSize,
+    public SecureRandomIdentifierGenerationStrategy(@Nonnull final SecureRandom source, final int identifierSize,
             @Nonnull final BinaryEncoder identifierEncoder) {
         super(source, identifierSize, identifierEncoder);
     }
diff --git a/src/main/java/net/shibboleth/utilities/java/support/security/SelfSignedCertificateGenerator.java b/src/main/java/net/shibboleth/utilities/java/support/security/SelfSignedCertificateGenerator.java
index 257546e..dbd803b 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/security/SelfSignedCertificateGenerator.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/security/SelfSignedCertificateGenerator.java
@@ -384,7 +384,7 @@ public class SelfSignedCertificateGenerator {
          *
          * @param optionName 
          */
-        public FileConverter(String optionName) {
+        public FileConverter(final String optionName) {
             super(optionName);
         }
 
diff --git a/src/main/java/net/shibboleth/utilities/java/support/velocity/Template.java b/src/main/java/net/shibboleth/utilities/java/support/velocity/Template.java
index d8f2d0f..7930b6f 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/velocity/Template.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/velocity/Template.java
@@ -70,8 +70,8 @@ public final class Template {
      * @param velocityTemplateName name of the template to be evaluated
      * @param velocityTemplateEncoding encoding used by the template
      */
-    private Template(@Nonnull VelocityEngine velocityEngine, @Nonnull @NotEmpty String velocityTemplateName,
-            @Nonnull String velocityTemplateEncoding) {
+    private Template(@Nonnull final VelocityEngine velocityEngine, @Nonnull @NotEmpty final String velocityTemplateName,
+            @Nonnull final String velocityTemplateEncoding) {
         engine = Constraint.isNotNull(velocityEngine, "Velocity engine can not be null");
         templateName =
                 Constraint.isNotNull(StringSupport.trimOrNull(velocityTemplateName),
diff --git a/src/main/java/net/shibboleth/utilities/java/support/xml/ClasspathResolver.java b/src/main/java/net/shibboleth/utilities/java/support/xml/ClasspathResolver.java
index 80164fd..87bc775 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/xml/ClasspathResolver.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/xml/ClasspathResolver.java
@@ -124,7 +124,7 @@ public class ClasspathResolver implements EntityResolver, LSResourceResolver {
          * @param sysId system id of the resolved resource
          * @param input resolved resource
          */
-        public LSInputImpl(String pubId, String sysId, InputStream input) {
+        public LSInputImpl(final String pubId, final String sysId, final InputStream input) {
             publicId = pubId;
             systemId = sysId;
             buffInput = new BufferedInputStream(input);
diff --git a/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java b/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java
index 653d744..7361c4f 100644
--- a/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java
+++ b/src/main/java/net/shibboleth/utilities/java/support/xml/SchemaBuilder.java
@@ -71,7 +71,7 @@ public class SchemaBuilder {
          * 
          * @param uri schema factory identifier
          */
-        private SchemaLanguage(@Nonnull @NotEmpty String uri) {
+        private SchemaLanguage(@Nonnull @NotEmpty final String uri) {
             schemaFactoryURI = Constraint.isNotNull(StringSupport.trimOrNull(uri), "URI cannot be null or empty");
         }
 

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list