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 2016/10/23 17:55:19 UTC
[2/2] [lang] Add final modifier to method parameters.
Add final modifier to method parameters.
Project: http://git-wip-us.apache.org/repos/asf/commons-lang/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-lang/commit/aadb9a31
Tree: http://git-wip-us.apache.org/repos/asf/commons-lang/tree/aadb9a31
Diff: http://git-wip-us.apache.org/repos/asf/commons-lang/diff/aadb9a31
Branch: refs/heads/master
Commit: aadb9a31ed969fc72cc87d486ec02383faa5bd06
Parents: 607e344
Author: Gary Gregory <gg...@apache.org>
Authored: Sun Oct 23 10:55:14 2016 -0700
Committer: Gary Gregory <gg...@apache.org>
Committed: Sun Oct 23 10:55:14 2016 -0700
----------------------------------------------------------------------
.../org/apache/commons/lang3/ArrayUtils.java | 50 +++++++++----------
.../org/apache/commons/lang3/BooleanUtils.java | 2 +-
.../org/apache/commons/lang3/CharUtils.java | 2 +-
.../org/apache/commons/lang3/ClassUtils.java | 4 +-
.../org/apache/commons/lang3/StringUtils.java | 22 ++++-----
.../MultilineRecursiveToStringStyle.java | 4 +-
.../commons/lang3/builder/ToStringStyle.java | 52 ++++++++++----------
.../concurrent/AbstractCircuitBreaker.java | 8 +--
.../concurrent/CircuitBreakingException.java | 6 +--
.../concurrent/EventCountCircuitBreaker.java | 48 +++++++++---------
.../concurrent/ThresholdCircuitBreaker.java | 4 +-
.../lang3/event/EventListenerSupport.java | 2 +-
.../commons/lang3/exception/ExceptionUtils.java | 8 +--
.../apache/commons/lang3/math/NumberUtils.java | 8 +--
.../commons/lang3/reflect/MemberUtils.java | 14 +++---
.../commons/lang3/reflect/MethodUtils.java | 8 +--
.../apache/commons/lang3/text/StrLookup.java | 2 +-
.../apache/commons/lang3/text/WordUtils.java | 4 +-
.../commons/lang3/time/CalendarReflection.java | 6 +--
.../commons/lang3/time/FastDateParser.java | 48 +++++++++---------
.../commons/lang3/time/FastDatePrinter.java | 6 +--
.../apache/commons/lang3/ArrayUtilsTest.java | 4 +-
.../commons/lang3/CharSequenceUtilsTest.java | 10 ++--
.../lang3/builder/EqualsBuilderTest.java | 2 +-
.../lang3/builder/HashCodeBuilderTest.java | 4 +-
.../MultilineRecursiveToStringStyleTest.java | 10 ++--
.../EventCountCircuitBreakerTest.java | 14 +++---
.../lang3/concurrent/TimedSemaphoreTest.java | 2 +-
.../lang3/reflect/ConstructorUtilsTest.java | 4 +-
.../commons/lang3/reflect/MethodUtilsTest.java | 50 +++++++++----------
.../lang3/test/NotVisibleExceptionFactory.java | 2 +-
.../lang3/test/SystemDefaultsSwitch.java | 6 +--
.../commons/lang3/time/DateFormatUtilsTest.java | 6 +--
.../commons/lang3/time/FastDateFormatTest.java | 8 +--
.../lang3/time/FastDateParserSDFTest.java | 2 +-
.../commons/lang3/time/FastDateParserTest.java | 4 +-
.../commons/lang3/time/FastDatePrinterTest.java | 4 +-
.../time/FastDatePrinterTimeZonesTest.java | 2 +-
.../commons/lang3/time/StopWatchTest.java | 2 +-
.../apache/commons/lang3/time/WeekYearTest.java | 2 +-
40 files changed, 223 insertions(+), 223 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/ArrayUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/ArrayUtils.java b/src/main/java/org/apache/commons/lang3/ArrayUtils.java
index 3ba98e5..15e597b 100644
--- a/src/main/java/org/apache/commons/lang3/ArrayUtils.java
+++ b/src/main/java/org/apache/commons/lang3/ArrayUtils.java
@@ -1852,7 +1852,7 @@ public class ArrayUtils {
* @param offset2 the index of the second element to swap
* @since 3.5
*/
- public static void swap(final Object[] array, int offset1, int offset2) {
+ public static void swap(final Object[] array, final int offset1, final int offset2) {
if (array == null || array.length == 0) {
return;
}
@@ -1881,7 +1881,7 @@ public class ArrayUtils {
* @param offset2 the index of the second element to swap
* @since 3.5
*/
- public static void swap(final long[] array, int offset1, int offset2) {
+ public static void swap(final long[] array, final int offset1, final int offset2) {
if (array == null || array.length == 0) {
return;
}
@@ -1909,7 +1909,7 @@ public class ArrayUtils {
* @param offset2 the index of the second element to swap
* @since 3.5
*/
- public static void swap(final int[] array, int offset1, int offset2) {
+ public static void swap(final int[] array, final int offset1, final int offset2) {
if (array == null || array.length == 0) {
return;
}
@@ -1937,7 +1937,7 @@ public class ArrayUtils {
* @param offset2 the index of the second element to swap
* @since 3.5
*/
- public static void swap(final short[] array, int offset1, int offset2) {
+ public static void swap(final short[] array, final int offset1, final int offset2) {
if (array == null || array.length == 0) {
return;
}
@@ -1965,7 +1965,7 @@ public class ArrayUtils {
* @param offset2 the index of the second element to swap
* @since 3.5
*/
- public static void swap(final char[] array, int offset1, int offset2) {
+ public static void swap(final char[] array, final int offset1, final int offset2) {
if (array == null || array.length == 0) {
return;
}
@@ -1993,7 +1993,7 @@ public class ArrayUtils {
* @param offset2 the index of the second element to swap
* @since 3.5
*/
- public static void swap(final byte[] array, int offset1, int offset2) {
+ public static void swap(final byte[] array, final int offset1, final int offset2) {
if (array == null || array.length == 0) {
return;
}
@@ -2021,7 +2021,7 @@ public class ArrayUtils {
* @param offset2 the index of the second element to swap
* @since 3.5
*/
- public static void swap(final double[] array, int offset1, int offset2) {
+ public static void swap(final double[] array, final int offset1, final int offset2) {
if (array == null || array.length == 0) {
return;
}
@@ -2049,7 +2049,7 @@ public class ArrayUtils {
* @param offset2 the index of the second element to swap
* @since 3.5
*/
- public static void swap(final float[] array, int offset1, int offset2) {
+ public static void swap(final float[] array, final int offset1, final int offset2) {
if (array == null || array.length == 0) {
return;
}
@@ -2077,7 +2077,7 @@ public class ArrayUtils {
* @param offset2 the index of the second element to swap
* @since 3.5
*/
- public static void swap(final boolean[] array, int offset1, int offset2) {
+ public static void swap(final boolean[] array, final int offset1, final int offset2) {
if (array == null || array.length == 0) {
return;
}
@@ -2480,7 +2480,7 @@ public class ArrayUtils {
* rotate, than the effective offset is modulo the number of elements to rotate.
* @since 3.5
*/
- public static void shift(final Object[] array, int offset) {
+ public static void shift(final Object[] array, final int offset) {
if (array == null) {
return;
}
@@ -2499,7 +2499,7 @@ public class ArrayUtils {
* rotate, than the effective offset is modulo the number of elements to rotate.
* @since 3.5
*/
- public static void shift(final long[] array, int offset) {
+ public static void shift(final long[] array, final int offset) {
if (array == null) {
return;
}
@@ -2518,7 +2518,7 @@ public class ArrayUtils {
* rotate, than the effective offset is modulo the number of elements to rotate.
* @since 3.5
*/
- public static void shift(final int[] array, int offset) {
+ public static void shift(final int[] array, final int offset) {
if (array == null) {
return;
}
@@ -2537,7 +2537,7 @@ public class ArrayUtils {
* rotate, than the effective offset is modulo the number of elements to rotate.
* @since 3.5
*/
- public static void shift(final short[] array, int offset) {
+ public static void shift(final short[] array, final int offset) {
if (array == null) {
return;
}
@@ -2556,7 +2556,7 @@ public class ArrayUtils {
* rotate, than the effective offset is modulo the number of elements to rotate.
* @since 3.5
*/
- public static void shift(final char[] array, int offset) {
+ public static void shift(final char[] array, final int offset) {
if (array == null) {
return;
}
@@ -2575,7 +2575,7 @@ public class ArrayUtils {
* rotate, than the effective offset is modulo the number of elements to rotate.
* @since 3.5
*/
- public static void shift(final byte[] array, int offset) {
+ public static void shift(final byte[] array, final int offset) {
if (array == null) {
return;
}
@@ -2594,7 +2594,7 @@ public class ArrayUtils {
* rotate, than the effective offset is modulo the number of elements to rotate.
* @since 3.5
*/
- public static void shift(final double[] array, int offset) {
+ public static void shift(final double[] array, final int offset) {
if (array == null) {
return;
}
@@ -2613,7 +2613,7 @@ public class ArrayUtils {
* rotate, than the effective offset is modulo the number of elements to rotate.
* @since 3.5
*/
- public static void shift(final float[] array, int offset) {
+ public static void shift(final float[] array, final int offset) {
if (array == null) {
return;
}
@@ -2632,7 +2632,7 @@ public class ArrayUtils {
* rotate, than the effective offset is modulo the number of elements to rotate.
* @since 3.5
*/
- public static void shift(final boolean[] array, int offset) {
+ public static void shift(final boolean[] array, final int offset) {
if (array == null) {
return;
}
@@ -7440,7 +7440,7 @@ public class ArrayUtils {
public static <T extends Comparable<? super T>> boolean isSorted(final T[] array) {
return isSorted(array, new Comparator<T>() {
@Override
- public int compare(T o1, T o2) {
+ public int compare(final T o1, final T o2) {
return o1.compareTo(o2);
}
});
@@ -7485,7 +7485,7 @@ public class ArrayUtils {
* @return whether the array is sorted according to natural ordering
* @since 3.4
*/
- public static boolean isSorted(int[] array) {
+ public static boolean isSorted(final int[] array) {
if (array == null || array.length < 2) {
return true;
}
@@ -7510,7 +7510,7 @@ public class ArrayUtils {
* @return whether the array is sorted according to natural ordering
* @since 3.4
*/
- public static boolean isSorted(long[] array) {
+ public static boolean isSorted(final long[] array) {
if (array == null || array.length < 2) {
return true;
}
@@ -7535,7 +7535,7 @@ public class ArrayUtils {
* @return whether the array is sorted according to natural ordering
* @since 3.4
*/
- public static boolean isSorted(short[] array) {
+ public static boolean isSorted(final short[] array) {
if (array == null || array.length < 2) {
return true;
}
@@ -7610,7 +7610,7 @@ public class ArrayUtils {
* @return whether the array is sorted according to natural ordering
* @since 3.4
*/
- public static boolean isSorted(byte[] array) {
+ public static boolean isSorted(final byte[] array) {
if (array == null || array.length < 2) {
return true;
}
@@ -7635,7 +7635,7 @@ public class ArrayUtils {
* @return whether the array is sorted according to natural ordering
* @since 3.4
*/
- public static boolean isSorted(char[] array) {
+ public static boolean isSorted(final char[] array) {
if (array == null || array.length < 2) {
return true;
}
@@ -7661,7 +7661,7 @@ public class ArrayUtils {
* @return whether the array is sorted according to natural ordering
* @since 3.4
*/
- public static boolean isSorted(boolean[] array) {
+ public static boolean isSorted(final boolean[] array) {
if (array == null || array.length < 2) {
return true;
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/BooleanUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/BooleanUtils.java b/src/main/java/org/apache/commons/lang3/BooleanUtils.java
index 8238ebb..9a21289 100644
--- a/src/main/java/org/apache/commons/lang3/BooleanUtils.java
+++ b/src/main/java/org/apache/commons/lang3/BooleanUtils.java
@@ -1091,7 +1091,7 @@ public class BooleanUtils {
* a value greater than {@code 0} if {@code x && !y}
* @since 3.4
*/
- public static int compare(boolean x, boolean y) {
+ public static int compare(final boolean x, final boolean y) {
if (x == y) {
return 0;
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/CharUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/CharUtils.java b/src/main/java/org/apache/commons/lang3/CharUtils.java
index 3e95741..3e7e8e6 100644
--- a/src/main/java/org/apache/commons/lang3/CharUtils.java
+++ b/src/main/java/org/apache/commons/lang3/CharUtils.java
@@ -546,7 +546,7 @@ public class CharUtils {
* a value greater than {@code 0} if {@code x > y}
* @since 3.4
*/
- public static int compare(char x, char y) {
+ public static int compare(final char x, final char y) {
return x-y;
}
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/ClassUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/ClassUtils.java b/src/main/java/org/apache/commons/lang3/ClassUtils.java
index 465990e..9e3b976 100644
--- a/src/main/java/org/apache/commons/lang3/ClassUtils.java
+++ b/src/main/java/org/apache/commons/lang3/ClassUtils.java
@@ -341,7 +341,7 @@ public class ClassUtils {
* @see #getAbbreviatedName(String, int)
* @since 3.4
*/
- public static String getAbbreviatedName(final Class<?> cls, int len) {
+ public static String getAbbreviatedName(final Class<?> cls, final int len) {
if (cls == null) {
return StringUtils.EMPTY;
}
@@ -373,7 +373,7 @@ public class ClassUtils {
* @throws IllegalArgumentException if len <= 0
* @since 3.4
*/
- public static String getAbbreviatedName(String className, int len) {
+ public static String getAbbreviatedName(final String className, final int len) {
if (len <= 0) {
throw new IllegalArgumentException("len must be > 0");
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/StringUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/StringUtils.java b/src/main/java/org/apache/commons/lang3/StringUtils.java
index b911af1..76f419c 100644
--- a/src/main/java/org/apache/commons/lang3/StringUtils.java
+++ b/src/main/java/org/apache/commons/lang3/StringUtils.java
@@ -551,7 +551,7 @@ public class StringUtils {
* @return truncated String, {@code null} if null String input
* @since 3.5
*/
- public static String truncate(final String str, int maxWidth) {
+ public static String truncate(final String str, final int maxWidth) {
return truncate(str, 0, maxWidth);
}
@@ -614,7 +614,7 @@ public class StringUtils {
* @return truncated String, {@code null} if null String input
* @since 3.5
*/
- public static String truncate(final String str, int offset, int maxWidth) {
+ public static String truncate(final String str, final int offset, final int maxWidth) {
if (offset < 0) {
throw new IllegalArgumentException("offset cannot be negative");
}
@@ -933,7 +933,7 @@ public class StringUtils {
return pattern.matcher(decomposed).replaceAll(StringUtils.EMPTY);
}
- private static void convertRemainingAccentCharacters(StringBuilder decomposed) {
+ private static void convertRemainingAccentCharacters(final StringBuilder decomposed) {
for (int i = 0; i < decomposed.length(); i++) {
if (decomposed.charAt(i) == '\u0141') {
decomposed.deleteCharAt(i);
@@ -2147,7 +2147,7 @@ public class StringUtils {
* @return {@code true} if any of the search CharSequences are found, {@code false} otherwise
* @since 3.4
*/
- public static boolean containsAny(CharSequence cs, CharSequence... searchCharSequences) {
+ public static boolean containsAny(final CharSequence cs, final CharSequence... searchCharSequences) {
if (isEmpty(cs) || ArrayUtils.isEmpty(searchCharSequences)) {
return false;
}
@@ -4912,7 +4912,7 @@ public class StringUtils {
* null String input
* @since 3.5
*/
- public static String removeIgnoreCase(String str, String remove) {
+ public static String removeIgnoreCase(final String str, final String remove) {
if (isEmpty(str) || isEmpty(remove)) {
return str;
}
@@ -5099,7 +5099,7 @@ public class StringUtils {
* {@code null} if null String input
* @since 3.5
*/
- public static String replaceOnceIgnoreCase(String text, String searchString, String replacement) {
+ public static String replaceOnceIgnoreCase(final String text, final String searchString, final String replacement) {
return replaceIgnoreCase(text, searchString, replacement, 1);
}
@@ -5342,7 +5342,7 @@ public class StringUtils {
* {@code null} if null String input
* @since 3.5
*/
- public static String replaceIgnoreCase(String text, String searchString, String replacement) {
+ public static String replaceIgnoreCase(final String text, final String searchString, final String replacement) {
return replaceIgnoreCase(text, searchString, replacement, -1);
}
@@ -5374,7 +5374,7 @@ public class StringUtils {
* @return the text with any replacements processed,
* {@code null} if null String input
*/
- public static String replace(final String text, final String searchString, final String replacement, int max) {
+ public static String replace(final String text, final String searchString, final String replacement, final int max) {
return replace(text, searchString, replacement, max, false);
}
@@ -5409,7 +5409,7 @@ public class StringUtils {
* @return the text with any replacements processed,
* {@code null} if null String input
*/
- private static String replace(String text, String searchString, String replacement, int max, boolean ignoreCase) {
+ private static String replace(final String text, String searchString, final String replacement, int max, final boolean ignoreCase) {
if (isEmpty(text) || isEmpty(searchString) || replacement == null || max == 0) {
return text;
}
@@ -5469,7 +5469,7 @@ public class StringUtils {
* {@code null} if null String input
* @since 3.5
*/
- public static String replaceIgnoreCase(String text, String searchString, String replacement, int max) {
+ public static String replaceIgnoreCase(final String text, final String searchString, final String replacement, final int max) {
return replace(text, searchString, replacement, max, true);
}
@@ -7275,7 +7275,7 @@ public class StringUtils {
* or {@code null} if null String input
* @since 3.5
*/
- public static String rotate(String str, int shift) {
+ public static String rotate(final String str, final int shift) {
if (str == null) {
return null;
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyle.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyle.java b/src/main/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyle.java
index a2e4d4d..db00b3d 100644
--- a/src/main/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyle.java
+++ b/src/main/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyle.java
@@ -104,7 +104,7 @@ public class MultilineRecursiveToStringStyle extends RecursiveToStringStyle {
* @param spaces how far to indent
* @return a StringBuilder with {spaces} leading space characters.
*/
- private StringBuilder spacer(int spaces) {
+ private StringBuilder spacer(final int spaces) {
StringBuilder sb = new StringBuilder();
for (int i = 0; i < spaces; i++) {
sb.append(" ");
@@ -113,7 +113,7 @@ public class MultilineRecursiveToStringStyle extends RecursiveToStringStyle {
}
@Override
- public void appendDetail(StringBuffer buffer, String fieldName, Object value) {
+ public void appendDetail(final StringBuffer buffer, final String fieldName, final Object value) {
if (!ClassUtils.isPrimitiveWrapper(value.getClass()) && !String.class.equals(value.getClass())
&& accept(value.getClass())) {
spaces += indent;
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java b/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java
index 4e43e52..2612dc6 100644
--- a/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java
+++ b/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java
@@ -2403,8 +2403,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName,
- Object[] array, Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName,
+ final Object[] array, final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2419,8 +2419,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName, long[] array,
- Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName, final long[] array,
+ final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2435,8 +2435,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName, int[] array,
- Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName, final int[] array,
+ final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2451,8 +2451,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName,
- short[] array, Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName,
+ final short[] array, final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2467,8 +2467,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName, byte[] array,
- Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName, final byte[] array,
+ final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2483,8 +2483,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName, char[] array,
- Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName, final char[] array,
+ final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2499,8 +2499,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName,
- double[] array, Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName,
+ final double[] array, final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2515,8 +2515,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName,
- float[] array, Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName,
+ final float[] array, final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2531,8 +2531,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName,
- boolean[] array, Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName,
+ final boolean[] array, final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2547,8 +2547,8 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- public void append(StringBuffer buffer, String fieldName, Object value,
- Boolean fullDetail) {
+ public void append(final StringBuffer buffer, final String fieldName, final Object value,
+ final Boolean fullDetail) {
if (fieldName == null) {
throw new UnsupportedOperationException(
@@ -2563,12 +2563,12 @@ public abstract class ToStringStyle implements Serializable {
}
@Override
- protected void appendDetail(StringBuffer buffer, String fieldName, char value) {
+ protected void appendDetail(final StringBuffer buffer, final String fieldName, final char value) {
appendValueAsString(buffer, String.valueOf(value));
}
@Override
- protected void appendDetail(StringBuffer buffer, String fieldName, Object value) {
+ protected void appendDetail(final StringBuffer buffer, final String fieldName, final Object value) {
if (value == null) {
appendNullText(buffer, fieldName);
@@ -2594,12 +2594,12 @@ public abstract class ToStringStyle implements Serializable {
appendDetail(buffer, fieldName, valueAsString);
}
- private boolean isJsonArray(String valueAsString) {
+ private boolean isJsonArray(final String valueAsString) {
return valueAsString.startsWith(getArrayStart())
&& valueAsString.startsWith(getArrayEnd());
}
- private boolean isJsonObject(String valueAsString) {
+ private boolean isJsonObject(final String valueAsString) {
return valueAsString.startsWith(getContentStart())
&& valueAsString.endsWith(getContentEnd());
}
@@ -2610,12 +2610,12 @@ public abstract class ToStringStyle implements Serializable {
* @param buffer the StringBuffer to append the value to.
* @param value the value to append.
*/
- private void appendValueAsString(StringBuffer buffer, String value) {
+ private void appendValueAsString(final StringBuffer buffer, final String value) {
buffer.append("\"" + value + "\"");
}
@Override
- protected void appendFieldStart(StringBuffer buffer, String fieldName) {
+ protected void appendFieldStart(final StringBuffer buffer, final String fieldName) {
if (fieldName == null) {
throw new UnsupportedOperationException(
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/concurrent/AbstractCircuitBreaker.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/concurrent/AbstractCircuitBreaker.java b/src/main/java/org/apache/commons/lang3/concurrent/AbstractCircuitBreaker.java
index 931c565..bcafeb6 100644
--- a/src/main/java/org/apache/commons/lang3/concurrent/AbstractCircuitBreaker.java
+++ b/src/main/java/org/apache/commons/lang3/concurrent/AbstractCircuitBreaker.java
@@ -90,7 +90,7 @@ public abstract class AbstractCircuitBreaker<T> implements CircuitBreaker<T> {
* @param state the state to be converted
* @return the boolean open flag
*/
- protected static boolean isOpen(State state) {
+ protected static boolean isOpen(final State state) {
return state == State.OPEN;
}
@@ -100,7 +100,7 @@ public abstract class AbstractCircuitBreaker<T> implements CircuitBreaker<T> {
*
* @param newState the new state to be set
*/
- protected void changeState(State newState) {
+ protected void changeState(final State newState) {
if (state.compareAndSet(newState.oppositeState(), newState)) {
changeSupport.firePropertyChange(PROPERTY_NAME, !isOpen(newState), isOpen(newState));
}
@@ -113,7 +113,7 @@ public abstract class AbstractCircuitBreaker<T> implements CircuitBreaker<T> {
*
* @param listener the listener to be added
*/
- public void addChangeListener(PropertyChangeListener listener) {
+ public void addChangeListener(final PropertyChangeListener listener) {
changeSupport.addPropertyChangeListener(listener);
}
@@ -122,7 +122,7 @@ public abstract class AbstractCircuitBreaker<T> implements CircuitBreaker<T> {
*
* @param listener the listener to be removed
*/
- public void removeChangeListener(PropertyChangeListener listener) {
+ public void removeChangeListener(final PropertyChangeListener listener) {
changeSupport.removePropertyChangeListener(listener);
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/concurrent/CircuitBreakingException.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/concurrent/CircuitBreakingException.java b/src/main/java/org/apache/commons/lang3/concurrent/CircuitBreakingException.java
index 5a37283..a3b6fdc 100644
--- a/src/main/java/org/apache/commons/lang3/concurrent/CircuitBreakingException.java
+++ b/src/main/java/org/apache/commons/lang3/concurrent/CircuitBreakingException.java
@@ -42,7 +42,7 @@ public class CircuitBreakingException extends RuntimeException {
* @param message the error message
* @param cause the cause of this exception
*/
- public CircuitBreakingException(String message, Throwable cause) {
+ public CircuitBreakingException(final String message, final Throwable cause) {
super(message, cause);
}
@@ -51,7 +51,7 @@ public class CircuitBreakingException extends RuntimeException {
*
* @param message the error message
*/
- public CircuitBreakingException(String message) {
+ public CircuitBreakingException(final String message) {
super(message);
}
@@ -60,7 +60,7 @@ public class CircuitBreakingException extends RuntimeException {
*
* @param cause the cause of this exception
*/
- public CircuitBreakingException(Throwable cause) {
+ public CircuitBreakingException(final Throwable cause) {
super(cause);
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreaker.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreaker.java b/src/main/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreaker.java
index 504ac36..b0d22ad 100644
--- a/src/main/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreaker.java
+++ b/src/main/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreaker.java
@@ -171,9 +171,9 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* @param closingInterval the interval for closing the circuit breaker
* @param closingUnit the {@code TimeUnit} defining the closing interval
*/
- public EventCountCircuitBreaker(int openingThreshold, long openingInterval,
- TimeUnit openingUnit, int closingThreshold, long closingInterval,
- TimeUnit closingUnit) {
+ public EventCountCircuitBreaker(final int openingThreshold, final long openingInterval,
+ final TimeUnit openingUnit, final int closingThreshold, final long closingInterval,
+ final TimeUnit closingUnit) {
super();
checkIntervalData = new AtomicReference<>(new CheckIntervalData(0, 0));
this.openingThreshold = openingThreshold;
@@ -195,8 +195,8 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* number of events received in the time span determined by the check interval goes
* below this threshold, the circuit breaker is closed again
*/
- public EventCountCircuitBreaker(int openingThreshold, long checkInterval, TimeUnit checkUnit,
- int closingThreshold) {
+ public EventCountCircuitBreaker(final int openingThreshold, final long checkInterval, final TimeUnit checkUnit,
+ final int closingThreshold) {
this(openingThreshold, checkInterval, checkUnit, closingThreshold, checkInterval,
checkUnit);
}
@@ -211,7 +211,7 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* @param checkInterval the check interval for opening or closing the circuit breaker
* @param checkUnit the {@code TimeUnit} defining the check interval
*/
- public EventCountCircuitBreaker(int threshold, long checkInterval, TimeUnit checkUnit) {
+ public EventCountCircuitBreaker(final int threshold, final long checkInterval, final TimeUnit checkUnit) {
this(threshold, checkInterval, checkUnit, threshold);
}
@@ -269,7 +269,7 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* {@inheritDoc}
*/
@Override
- public boolean incrementAndCheckState(Integer increment)
+ public boolean incrementAndCheckState(final Integer increment)
throws CircuitBreakingException {
return performStateCheck(1);
}
@@ -317,7 +317,7 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* @param increment the increment for the internal counter
* @return a flag whether the circuit breaker is now closed
*/
- private boolean performStateCheck(int increment) {
+ private boolean performStateCheck(final int increment) {
CheckIntervalData currentData;
CheckIntervalData nextData;
State currentState;
@@ -348,8 +348,8 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* @param nextData the replacing check data object
* @return a flag whether the update was successful
*/
- private boolean updateCheckIntervalData(CheckIntervalData currentData,
- CheckIntervalData nextData) {
+ private boolean updateCheckIntervalData(final CheckIntervalData currentData,
+ final CheckIntervalData nextData) {
return currentData == nextData
|| checkIntervalData.compareAndSet(currentData, nextData);
}
@@ -360,7 +360,7 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
*
* @param newState the new state to be set
*/
- private void changeStateAndStartNewCheckInterval(State newState) {
+ private void changeStateAndStartNewCheckInterval(final State newState) {
changeState(newState);
checkIntervalData.set(new CheckIntervalData(0, now()));
}
@@ -376,8 +376,8 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* @param time the current time
* @return the updated {@code CheckIntervalData} object
*/
- private CheckIntervalData nextCheckIntervalData(int increment,
- CheckIntervalData currentData, State currentState, long time) {
+ private CheckIntervalData nextCheckIntervalData(final int increment,
+ final CheckIntervalData currentData, final State currentState, final long time) {
CheckIntervalData nextData;
if (stateStrategy(currentState).isCheckIntervalFinished(this, currentData, time)) {
nextData = new CheckIntervalData(increment, time);
@@ -404,7 +404,7 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* @return the corresponding {@code StateStrategy}
* @throws CircuitBreakingException if the strategy cannot be resolved
*/
- private static StateStrategy stateStrategy(State state) {
+ private static StateStrategy stateStrategy(final State state) {
StateStrategy strategy = STRATEGY_MAP.get(state);
return strategy;
}
@@ -440,7 +440,7 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* @param count the current count value
* @param intervalStart the start time of the check interval
*/
- public CheckIntervalData(int count, long intervalStart) {
+ public CheckIntervalData(final int count, final long intervalStart) {
eventCount = count;
checkIntervalStart = intervalStart;
}
@@ -470,7 +470,7 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* @param delta the delta
* @return the updated instance
*/
- public CheckIntervalData increment(int delta) {
+ public CheckIntervalData increment(final int delta) {
return (delta != 0) ? new CheckIntervalData(getEventCount() + delta,
getCheckIntervalStart()) : this;
}
@@ -490,8 +490,8 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* @param now the current time
* @return a flag whether the end of the current check interval is reached
*/
- public boolean isCheckIntervalFinished(EventCountCircuitBreaker breaker,
- CheckIntervalData currentData, long now) {
+ public boolean isCheckIntervalFinished(final EventCountCircuitBreaker breaker,
+ final CheckIntervalData currentData, final long now) {
return now - currentData.getCheckIntervalStart() > fetchCheckInterval(breaker);
}
@@ -527,8 +527,8 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* {@inheritDoc}
*/
@Override
- public boolean isStateTransition(EventCountCircuitBreaker breaker,
- CheckIntervalData currentData, CheckIntervalData nextData) {
+ public boolean isStateTransition(final EventCountCircuitBreaker breaker,
+ final CheckIntervalData currentData, final CheckIntervalData nextData) {
return nextData.getEventCount() > breaker.getOpeningThreshold();
}
@@ -536,7 +536,7 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* {@inheritDoc}
*/
@Override
- protected long fetchCheckInterval(EventCountCircuitBreaker breaker) {
+ protected long fetchCheckInterval(final EventCountCircuitBreaker breaker) {
return breaker.getOpeningInterval();
}
}
@@ -549,8 +549,8 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* {@inheritDoc}
*/
@Override
- public boolean isStateTransition(EventCountCircuitBreaker breaker,
- CheckIntervalData currentData, CheckIntervalData nextData) {
+ public boolean isStateTransition(final EventCountCircuitBreaker breaker,
+ final CheckIntervalData currentData, final CheckIntervalData nextData) {
return nextData.getCheckIntervalStart() != currentData
.getCheckIntervalStart()
&& currentData.getEventCount() < breaker.getClosingThreshold();
@@ -560,7 +560,7 @@ public class EventCountCircuitBreaker extends AbstractCircuitBreaker<Integer> {
* {@inheritDoc}
*/
@Override
- protected long fetchCheckInterval(EventCountCircuitBreaker breaker) {
+ protected long fetchCheckInterval(final EventCountCircuitBreaker breaker) {
return breaker.getClosingInterval();
}
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/concurrent/ThresholdCircuitBreaker.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/concurrent/ThresholdCircuitBreaker.java b/src/main/java/org/apache/commons/lang3/concurrent/ThresholdCircuitBreaker.java
index b225d2b..c00e93d 100644
--- a/src/main/java/org/apache/commons/lang3/concurrent/ThresholdCircuitBreaker.java
+++ b/src/main/java/org/apache/commons/lang3/concurrent/ThresholdCircuitBreaker.java
@@ -72,7 +72,7 @@ public class ThresholdCircuitBreaker extends AbstractCircuitBreaker<Long> {
*
* @param threshold the threshold.
*/
- public ThresholdCircuitBreaker(long threshold) {
+ public ThresholdCircuitBreaker(final long threshold) {
super();
this.used = new AtomicLong(INITIAL_COUNT);
this.threshold = threshold;
@@ -112,7 +112,7 @@ public class ThresholdCircuitBreaker extends AbstractCircuitBreaker<Long> {
* <p>If the threshold is zero, the circuit breaker will be in a permanent <em>open</em> state.</p>
*/
@Override
- public boolean incrementAndCheckState(Long increment) throws CircuitBreakingException {
+ public boolean incrementAndCheckState(final Long increment) throws CircuitBreakingException {
if (threshold == 0) {
open();
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/event/EventListenerSupport.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/event/EventListenerSupport.java b/src/main/java/org/apache/commons/lang3/event/EventListenerSupport.java
index 5d3ed6b..5e9a784 100644
--- a/src/main/java/org/apache/commons/lang3/event/EventListenerSupport.java
+++ b/src/main/java/org/apache/commons/lang3/event/EventListenerSupport.java
@@ -193,7 +193,7 @@ public class EventListenerSupport<L> implements Serializable {
* @throws NullPointerException if <code>listener</code> is <code>null</code>.
* @since 3.5
*/
- public void addListener(final L listener, boolean allowDuplicate) {
+ public void addListener(final L listener, final boolean allowDuplicate) {
Validate.notNull(listener, "Listener object cannot be null.");
if (allowDuplicate) {
listeners.add(listener);
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/exception/ExceptionUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/exception/ExceptionUtils.java b/src/main/java/org/apache/commons/lang3/exception/ExceptionUtils.java
index f26a07a..7a1d810 100644
--- a/src/main/java/org/apache/commons/lang3/exception/ExceptionUtils.java
+++ b/src/main/java/org/apache/commons/lang3/exception/ExceptionUtils.java
@@ -754,7 +754,7 @@ public class ExceptionUtils {
* @since 3.5
* @see #wrapAndThrow(Throwable)
*/
- public static <R> R rethrow(Throwable throwable) {
+ public static <R> R rethrow(final Throwable throwable) {
// claim that the typeErasure invocation throws a RuntimeException
return ExceptionUtils.<R, RuntimeException> typeErasure(throwable);
}
@@ -766,7 +766,7 @@ public class ExceptionUtils {
* clause.
*/
@SuppressWarnings("unchecked")
- private static <R, T extends Throwable> R typeErasure(Throwable throwable) throws T {
+ private static <R, T extends Throwable> R typeErasure(final Throwable throwable) throws T {
throw (T) throwable;
}
@@ -791,7 +791,7 @@ public class ExceptionUtils {
* @see #rethrow(Throwable)
* @see #hasCause(Throwable, Class)
*/
- public static <R> R wrapAndThrow(Throwable throwable) {
+ public static <R> R wrapAndThrow(final Throwable throwable) {
if (throwable instanceof RuntimeException) {
throw (RuntimeException) throwable;
}
@@ -815,7 +815,7 @@ public class ExceptionUtils {
* @see #wrapAndThrow(Throwable)
*/
public static boolean hasCause(Throwable chain,
- Class<? extends Throwable> type) {
+ final Class<? extends Throwable> type) {
if (chain instanceof UndeclaredThrowableException) {
chain = chain.getCause();
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/math/NumberUtils.java b/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
index d75f783..335dd8d 100644
--- a/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
+++ b/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
@@ -1576,7 +1576,7 @@ public class NumberUtils {
* a value greater than {@code 0} if {@code x > y}
* @since 3.4
*/
- public static int compare(int x, int y) {
+ public static int compare(final int x, final int y) {
if (x == y) {
return 0;
}
@@ -1593,7 +1593,7 @@ public class NumberUtils {
* a value greater than {@code 0} if {@code x > y}
* @since 3.4
*/
- public static int compare(long x, long y) {
+ public static int compare(final long x, final long y) {
if (x == y) {
return 0;
}
@@ -1610,7 +1610,7 @@ public class NumberUtils {
* a value greater than {@code 0} if {@code x > y}
* @since 3.4
*/
- public static int compare(short x, short y) {
+ public static int compare(final short x, final short y) {
if (x == y) {
return 0;
}
@@ -1627,7 +1627,7 @@ public class NumberUtils {
* a value greater than {@code 0} if {@code x > y}
* @since 3.4
*/
- public static int compare(byte x, byte y) {
+ public static int compare(final byte x, final byte y) {
return x - y;
}
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java b/src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java
index c48a0a0..a61b051 100644
--- a/src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java
+++ b/src/main/java/org/apache/commons/lang3/reflect/MemberUtils.java
@@ -247,15 +247,15 @@ abstract class MemberUtils {
return cost;
}
- static boolean isMatchingMethod(Method method, Class<?>[] parameterTypes) {
+ static boolean isMatchingMethod(final Method method, final Class<?>[] parameterTypes) {
return MemberUtils.isMatchingExecutable(Executable.of(method), parameterTypes);
}
- static boolean isMatchingConstructor(Constructor<?> method, Class<?>[] parameterTypes) {
+ static boolean isMatchingConstructor(final Constructor<?> method, final Class<?>[] parameterTypes) {
return MemberUtils.isMatchingExecutable(Executable.of(method), parameterTypes);
}
- private static boolean isMatchingExecutable(Executable method, Class<?>[] parameterTypes) {
+ private static boolean isMatchingExecutable(final Executable method, final Class<?>[] parameterTypes) {
final Class<?>[] methodParameterTypes = method.getParameterTypes();
if (method.isVarArgs()) {
int i;
@@ -283,15 +283,15 @@ abstract class MemberUtils {
private final Class<?>[] parameterTypes;
private final boolean isVarArgs;
- private static Executable of(Method method) { return new Executable(method); }
- private static Executable of(Constructor<?> constructor) { return new Executable(constructor); }
+ private static Executable of(final Method method) { return new Executable(method); }
+ private static Executable of(final Constructor<?> constructor) { return new Executable(constructor); }
- private Executable(Method method) {
+ private Executable(final Method method) {
parameterTypes = method.getParameterTypes();
isVarArgs = method.isVarArgs();
}
- private Executable(Constructor<?> constructor) {
+ private Executable(final Constructor<?> constructor) {
parameterTypes = constructor.getParameterTypes();
isVarArgs = constructor.isVarArgs();
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java b/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java
index cc0a0e4..63029f2 100644
--- a/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java
+++ b/src/main/java/org/apache/commons/lang3/reflect/MethodUtils.java
@@ -251,7 +251,7 @@ public class MethodUtils {
* @throws IllegalAccessException if the requested method is not accessible via reflection
*/
public static Object invokeMethod(final Object object, final String methodName,
- Object[] args, Class<?>[] parameterTypes)
+ final Object[] args, final Class<?>[] parameterTypes)
throws NoSuchMethodException, IllegalAccessException,
InvocationTargetException {
return invokeMethod(object, false, methodName, args, parameterTypes);
@@ -444,7 +444,7 @@ public class MethodUtils {
return method.invoke(null, args);
}
- private static Object[] toVarArgs(Method method, Object[] args) {
+ private static Object[] toVarArgs(final Method method, Object[] args) {
if (method.isVarArgs()) {
Class<?>[] methodParameterTypes = method.getParameterTypes();
args = getVarArgs(args, methodParameterTypes);
@@ -462,7 +462,7 @@ public class MethodUtils {
* @return an array of the variadic arguments passed to the method
* @since 3.5
*/
- static Object[] getVarArgs(Object[] args, Class<?>[] methodParameterTypes) {
+ static Object[] getVarArgs(final Object[] args, final Class<?>[] methodParameterTypes) {
if (args.length == methodParameterTypes.length
&& args[args.length - 1].getClass().equals(methodParameterTypes[methodParameterTypes.length - 1])) {
// The args array is already in the canonical form for the method.
@@ -750,7 +750,7 @@ public class MethodUtils {
* @param toClassArray
* @return the aggregate number of inheritance hops between assignable argument class types.
*/
- private static int distance(Class<?>[] classArray, Class<?>[] toClassArray) {
+ private static int distance(final Class<?>[] classArray, final Class<?>[] toClassArray) {
int answer = 0;
if (!ClassUtils.isAssignable(classArray, toClassArray, true)) {
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/text/StrLookup.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/StrLookup.java b/src/main/java/org/apache/commons/lang3/text/StrLookup.java
index 01e87f2..4e2569a 100644
--- a/src/main/java/org/apache/commons/lang3/text/StrLookup.java
+++ b/src/main/java/org/apache/commons/lang3/text/StrLookup.java
@@ -166,7 +166,7 @@ public abstract class StrLookup<V> {
* {@inheritDoc} This implementation directly accesses system properties.
*/
@Override
- public String lookup(String key) {
+ public String lookup(final String key) {
if (key.length() > 0) {
try {
return System.getProperty(key);
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/text/WordUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/text/WordUtils.java b/src/main/java/org/apache/commons/lang3/text/WordUtils.java
index 69f5a7e..7fff981 100644
--- a/src/main/java/org/apache/commons/lang3/text/WordUtils.java
+++ b/src/main/java/org/apache/commons/lang3/text/WordUtils.java
@@ -172,7 +172,7 @@ public class WordUtils {
* @param wrapLongWords true if long words (such as URLs) should be wrapped
* @return a line with newlines inserted, <code>null</code> if null input
*/
- public static String wrap(final String str, int wrapLength, String newLineStr, final boolean wrapLongWords) {
+ public static String wrap(final String str, final int wrapLength, final String newLineStr, final boolean wrapLongWords) {
return wrap(str, wrapLength, newLineStr, wrapLongWords, " ");
}
@@ -698,7 +698,7 @@ public class WordUtils {
* @return {@code true} if all search words are found, {@code false} otherwise
* @since 3.5
*/
- public static boolean containsAllWords(CharSequence word, CharSequence... words) {
+ public static boolean containsAllWords(final CharSequence word, final CharSequence... words) {
if (StringUtils.isEmpty(word) || ArrayUtils.isEmpty(words)) {
return false;
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/time/CalendarReflection.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/time/CalendarReflection.java b/src/main/java/org/apache/commons/lang3/time/CalendarReflection.java
index 79ebb3f..933e50f 100644
--- a/src/main/java/org/apache/commons/lang3/time/CalendarReflection.java
+++ b/src/main/java/org/apache/commons/lang3/time/CalendarReflection.java
@@ -30,7 +30,7 @@ class CalendarReflection {
private static final Method IS_WEEK_DATE_SUPPORTED = getCalendarMethod("isWeekDateSupported");
private static final Method GET_WEEK_YEAR = getCalendarMethod("getWeekYear");
- private static Method getCalendarMethod(String methodName, Class<?>... argTypes) {
+ private static Method getCalendarMethod(final String methodName, final Class<?>... argTypes) {
try {
Method m = Calendar.class.getMethod(methodName, argTypes);
return m;
@@ -44,7 +44,7 @@ class CalendarReflection {
* @param calendar The calendar instance.
* @return false, if runtime is less than java 1.7; otherwise, the result of calendar.isWeekDateSupported().
*/
- static boolean isWeekDateSupported(Calendar calendar) {
+ static boolean isWeekDateSupported(final Calendar calendar) {
try {
return IS_WEEK_DATE_SUPPORTED!=null && ((Boolean)IS_WEEK_DATE_SUPPORTED.invoke(calendar)).booleanValue();
} catch (Exception e) {
@@ -68,7 +68,7 @@ class CalendarReflection {
* @param calendar The calendar instance.
* @return the week year or year value.
*/
- public static int getWeekYear(Calendar calendar) {
+ public static int getWeekYear(final Calendar calendar) {
try {
if (isWeekDateSupported(calendar)) {
return (Integer) GET_WEEK_YEAR.invoke(calendar);
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/time/FastDateParser.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/time/FastDateParser.java b/src/main/java/org/apache/commons/lang3/time/FastDateParser.java
index f6caf10..b4afa8a 100644
--- a/src/main/java/org/apache/commons/lang3/time/FastDateParser.java
+++ b/src/main/java/org/apache/commons/lang3/time/FastDateParser.java
@@ -96,7 +96,7 @@ public class FastDateParser implements DateParser, Serializable {
// all entries must be lowercase by locale.
private static final Comparator<String> LONGER_FIRST_LOWERCASE = new Comparator<String>() {
@Override
- public int compare(String left, String right) {
+ public int compare(final String left, final String right) {
return right.compareTo(left);
}
};
@@ -182,12 +182,12 @@ public class FastDateParser implements DateParser, Serializable {
final Strategy strategy;
final int width;
- StrategyAndWidth(Strategy strategy, int width) {
+ StrategyAndWidth(final Strategy strategy, final int width) {
this.strategy = strategy;
this.width = width;
}
- int getMaxWidth(ListIterator<StrategyAndWidth> lt) {
+ int getMaxWidth(final ListIterator<StrategyAndWidth> lt) {
if(!strategy.isNumber() || !lt.hasNext()) {
return 0;
}
@@ -205,7 +205,7 @@ public class FastDateParser implements DateParser, Serializable {
final private Calendar definingCalendar;
private int currentIdx;
- StrategyParser(String pattern, Calendar definingCalendar) {
+ StrategyParser(final String pattern, final Calendar definingCalendar) {
this.pattern = pattern;
this.definingCalendar = definingCalendar;
}
@@ -222,7 +222,7 @@ public class FastDateParser implements DateParser, Serializable {
return literal();
}
- private StrategyAndWidth letterPattern(char c) {
+ private StrategyAndWidth letterPattern(final char c) {
int begin = currentIdx;
while (++currentIdx < pattern.length()) {
if (pattern.charAt(currentIdx) != c) {
@@ -259,7 +259,7 @@ public class FastDateParser implements DateParser, Serializable {
}
}
- private static boolean isFormatLetter(char c) {
+ private static boolean isFormatLetter(final char c) {
return c >= 'A' && c <= 'Z' || c >= 'a' && c <= 'z';
}
@@ -463,7 +463,7 @@ public class FastDateParser implements DateParser, Serializable {
* @param regex The regular expression to build
* @return The map of string display names to field values
*/
- private static Map<String, Integer> appendDisplayNames(Calendar cal, Locale locale, int field, StringBuilder regex) {
+ private static Map<String, Integer> appendDisplayNames(final Calendar cal, final Locale locale, final int field, final StringBuilder regex) {
Map<String, Integer> values = new HashMap<>();
Map<String, Integer> displayNames = cal.getDisplayNames(field, Calendar.ALL_STYLES, locale);
@@ -514,11 +514,11 @@ public class FastDateParser implements DateParser, Serializable {
private Pattern pattern;
- void createPattern(StringBuilder regex) {
+ void createPattern(final StringBuilder regex) {
createPattern(regex.toString());
}
- void createPattern(String regex) {
+ void createPattern(final String regex) {
this.pattern = Pattern.compile(regex);
}
@@ -534,7 +534,7 @@ public class FastDateParser implements DateParser, Serializable {
}
@Override
- boolean parse(FastDateParser parser, Calendar calendar, String source, ParsePosition pos, int maxWidth) {
+ boolean parse(final FastDateParser parser, final Calendar calendar, final String source, final ParsePosition pos, final int maxWidth) {
Matcher matcher = pattern.matcher(source.substring(pos.getIndex()));
if (!matcher.lookingAt()) {
pos.setErrorIndex(pos.getIndex());
@@ -554,7 +554,7 @@ public class FastDateParser implements DateParser, Serializable {
* @param definingCalendar The calendar to obtain the short and long values
* @return The Strategy that will handle parsing for the field
*/
- private Strategy getStrategy(char f, int width, final Calendar definingCalendar) {
+ private Strategy getStrategy(final char f, final int width, final Calendar definingCalendar) {
switch(f) {
default:
throw new IllegalArgumentException("Format '"+f+"' not supported");
@@ -669,7 +669,7 @@ public class FastDateParser implements DateParser, Serializable {
}
@Override
- boolean parse(FastDateParser parser, Calendar calendar, String source, ParsePosition pos, int maxWidth) {
+ boolean parse(final FastDateParser parser, final Calendar calendar, final String source, final ParsePosition pos, final int maxWidth) {
for (int idx = 0; idx < formatField.length(); ++idx) {
int sIdx = idx + pos.getIndex();
if (sIdx == source.length()) {
@@ -716,7 +716,7 @@ public class FastDateParser implements DateParser, Serializable {
* {@inheritDoc}
*/
@Override
- void setCalendar(FastDateParser parser, Calendar cal, String value) {
+ void setCalendar(final FastDateParser parser, final Calendar cal, final String value) {
final Integer iVal = lKeyValues.get(value.toLowerCase(locale));
cal.set(field, iVal.intValue());
}
@@ -746,7 +746,7 @@ public class FastDateParser implements DateParser, Serializable {
}
@Override
- boolean parse(FastDateParser parser, Calendar calendar, String source, ParsePosition pos, int maxWidth) {
+ boolean parse(final FastDateParser parser, final Calendar calendar, final String source, final ParsePosition pos, final int maxWidth) {
int idx = pos.getIndex();
int last = source.length();
@@ -791,7 +791,7 @@ public class FastDateParser implements DateParser, Serializable {
* @param iValue The parsed integer
* @return The modified value
*/
- int modify(FastDateParser parser, final int iValue) {
+ int modify(final FastDateParser parser, final int iValue) {
return iValue;
}
@@ -802,7 +802,7 @@ public class FastDateParser implements DateParser, Serializable {
* {@inheritDoc}
*/
@Override
- int modify(FastDateParser parser, final int iValue) {
+ int modify(final FastDateParser parser, final int iValue) {
return iValue < 100 ? parser.adjustYear(iValue) : iValue;
}
};
@@ -821,7 +821,7 @@ public class FastDateParser implements DateParser, Serializable {
TimeZone zone;
int dstOffset;
- TzInfo(TimeZone tz, boolean useDst) {
+ TzInfo(final TimeZone tz, final boolean useDst) {
zone = tz;
dstOffset = useDst ?tz.getDSTSavings() :0;
}
@@ -909,7 +909,7 @@ public class FastDateParser implements DateParser, Serializable {
* Construct a Strategy that parses a TimeZone
* @param pattern The Pattern
*/
- ISO8601TimeZoneStrategy(String pattern) {
+ ISO8601TimeZoneStrategy(final String pattern) {
createPattern(pattern);
}
@@ -917,7 +917,7 @@ public class FastDateParser implements DateParser, Serializable {
* {@inheritDoc}
*/
@Override
- void setCalendar(FastDateParser parser, Calendar cal, String value) {
+ void setCalendar(final FastDateParser parser, final Calendar cal, final String value) {
if (value.equals("Z")) {
cal.setTimeZone(TimeZone.getTimeZone("UTC"));
} else {
@@ -936,7 +936,7 @@ public class FastDateParser implements DateParser, Serializable {
* @return a ISO8601TimeZoneStrategy that can format TimeZone String of length {@code tokenLen}. If no such
* strategy exists, an IllegalArgumentException will be thrown.
*/
- static Strategy getStrategy(int tokenLen) {
+ static Strategy getStrategy(final int tokenLen) {
switch(tokenLen) {
case 1:
return ISO_8601_1_STRATEGY;
@@ -952,7 +952,7 @@ public class FastDateParser implements DateParser, Serializable {
private static final Strategy NUMBER_MONTH_STRATEGY = new NumberStrategy(Calendar.MONTH) {
@Override
- int modify(FastDateParser parser, final int iValue) {
+ int modify(final FastDateParser parser, final int iValue) {
return iValue-1;
}
};
@@ -963,7 +963,7 @@ public class FastDateParser implements DateParser, Serializable {
private static final Strategy DAY_OF_MONTH_STRATEGY = new NumberStrategy(Calendar.DAY_OF_MONTH);
private static final Strategy DAY_OF_WEEK_STRATEGY = new NumberStrategy(Calendar.DAY_OF_WEEK) {
@Override
- int modify(FastDateParser parser, final int iValue) {
+ int modify(final FastDateParser parser, final int iValue) {
return iValue != 7 ? iValue + 1 : Calendar.SUNDAY;
}
};
@@ -971,13 +971,13 @@ public class FastDateParser implements DateParser, Serializable {
private static final Strategy HOUR_OF_DAY_STRATEGY = new NumberStrategy(Calendar.HOUR_OF_DAY);
private static final Strategy HOUR24_OF_DAY_STRATEGY = new NumberStrategy(Calendar.HOUR_OF_DAY) {
@Override
- int modify(FastDateParser parser, final int iValue) {
+ int modify(final FastDateParser parser, final int iValue) {
return iValue == 24 ? 0 : iValue;
}
};
private static final Strategy HOUR12_STRATEGY = new NumberStrategy(Calendar.HOUR) {
@Override
- int modify(FastDateParser parser, final int iValue) {
+ int modify(final FastDateParser parser, final int iValue) {
return iValue == 12 ? 0 : iValue;
}
};
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/main/java/org/apache/commons/lang3/time/FastDatePrinter.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/lang3/time/FastDatePrinter.java b/src/main/java/org/apache/commons/lang3/time/FastDatePrinter.java
index 8198063..f50010e 100644
--- a/src/main/java/org/apache/commons/lang3/time/FastDatePrinter.java
+++ b/src/main/java/org/apache/commons/lang3/time/FastDatePrinter.java
@@ -423,7 +423,7 @@ public class FastDatePrinter implements DatePrinter, Serializable {
* @param obj the object to format
* @return The formatted value.
*/
- String format(Object obj) {
+ String format(final Object obj) {
if (obj instanceof Date) {
return format((Date) obj);
} else if (obj instanceof Calendar) {
@@ -1031,7 +1031,7 @@ public class FastDatePrinter implements DatePrinter, Serializable {
* {@inheritDoc}
*/
@Override
- public final void appendTo(final Appendable buffer, int value) throws IOException {
+ public final void appendTo(final Appendable buffer, final int value) throws IOException {
appendFullDigits(buffer, value, mSize);
}
}
@@ -1449,7 +1449,7 @@ public class FastDatePrinter implements DatePrinter, Serializable {
* @return a Iso8601_Rule that can format TimeZone String of length {@code tokenLen}. If no such
* rule exists, an IllegalArgumentException will be thrown.
*/
- static Iso8601_Rule getRule(int tokenLen) {
+ static Iso8601_Rule getRule(final int tokenLen) {
switch(tokenLen) {
case 1:
return Iso8601_Rule.ISO8601_HOURS;
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java b/src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java
index 2d137c8..8c21dbc 100644
--- a/src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java
+++ b/src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java
@@ -2958,7 +2958,7 @@ public class ArrayUtilsTest {
public void testContains_LANG_1261() {
class LANG1261ParentObject {
@Override
- public boolean equals(Object o) {
+ public boolean equals(final Object o) {
return true;
}
}
@@ -4288,7 +4288,7 @@ public class ArrayUtilsTest {
@Test
public void testIsSortedComparator() {
Comparator<Integer> c = new Comparator<Integer>() {
- public int compare(Integer o1, Integer o2) {
+ public int compare(final Integer o1, final Integer o2) {
return o2.compareTo(o1);
}
};
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/test/java/org/apache/commons/lang3/CharSequenceUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/lang3/CharSequenceUtilsTest.java b/src/test/java/org/apache/commons/lang3/CharSequenceUtilsTest.java
index 8518a89..06396be 100644
--- a/src/test/java/org/apache/commons/lang3/CharSequenceUtilsTest.java
+++ b/src/test/java/org/apache/commons/lang3/CharSequenceUtilsTest.java
@@ -82,8 +82,8 @@ public class CharSequenceUtilsTest {
final int len;
final boolean expected;
final Class<?> throwable;
- TestData(String source, boolean ignoreCase, int toffset,
- String other, int ooffset, int len, boolean expected){
+ TestData(final String source, final boolean ignoreCase, final int toffset,
+ final String other, final int ooffset, final int len, final boolean expected){
this.source = source;
this.ignoreCase = ignoreCase;
this.toffset = toffset;
@@ -93,8 +93,8 @@ public class CharSequenceUtilsTest {
this.expected = expected;
this.throwable = null;
}
- TestData(String source, boolean ignoreCase, int toffset,
- String other, int ooffset, int len, Class<?> throwable){
+ TestData(final String source, final boolean ignoreCase, final int toffset,
+ final String other, final int ooffset, final int len, final Class<?> throwable){
this.source = source;
this.ignoreCase = ignoreCase;
this.toffset = toffset;
@@ -140,7 +140,7 @@ public class CharSequenceUtilsTest {
abstract boolean invoke();
- void run(TestData data, String id) {
+ void run(final TestData data, final String id) {
if (data.throwable != null) {
try {
invoke();
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/test/java/org/apache/commons/lang3/builder/EqualsBuilderTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/lang3/builder/EqualsBuilderTest.java b/src/test/java/org/apache/commons/lang3/builder/EqualsBuilderTest.java
index 1f29a57..a586049 100644
--- a/src/test/java/org/apache/commons/lang3/builder/EqualsBuilderTest.java
+++ b/src/test/java/org/apache/commons/lang3/builder/EqualsBuilderTest.java
@@ -1153,7 +1153,7 @@ public class EqualsBuilderTest {
private final int a;
private final int b;
- public TestObjectEqualsExclude(int a, int b) {
+ public TestObjectEqualsExclude(final int a, final int b) {
this.a = a;
this.b = b;
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/test/java/org/apache/commons/lang3/builder/HashCodeBuilderTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/lang3/builder/HashCodeBuilderTest.java b/src/test/java/org/apache/commons/lang3/builder/HashCodeBuilderTest.java
index 1ad2116..8ce0a04 100644
--- a/src/test/java/org/apache/commons/lang3/builder/HashCodeBuilderTest.java
+++ b/src/test/java/org/apache/commons/lang3/builder/HashCodeBuilderTest.java
@@ -568,7 +568,7 @@ public class HashCodeBuilderTest {
private final int a;
private final int b;
- public TestObjectHashCodeExclude(int a, int b) {
+ public TestObjectHashCodeExclude(final int a, final int b) {
this.a = a;
this.b = b;
}
@@ -588,7 +588,7 @@ public class HashCodeBuilderTest {
@HashCodeExclude
private final int b;
- public TestObjectHashCodeExclude2(int a, int b) {
+ public TestObjectHashCodeExclude2(final int a, final int b) {
this.a = a;
this.b = b;
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/test/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyleTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyleTest.java b/src/test/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyleTest.java
index e4e4284..4b9ebee 100644
--- a/src/test/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyleTest.java
+++ b/src/test/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyleTest.java
@@ -202,11 +202,11 @@ public class MultilineRecursiveToStringStyleTest {
assertEquals(exp, toString(wa));
}
- private String getClassPrefix(Object object) {
+ private String getClassPrefix(final Object object) {
return object.getClass().getName() + "@" + Integer.toHexString(System.identityHashCode(object));
}
- private String toString(Object object) {
+ private String toString(final Object object) {
return new ReflectionToStringBuilder(object, new MultilineRecursiveToStringStyle()).toString();
}
@@ -222,7 +222,7 @@ public class MultilineRecursiveToStringStyleTest {
static class Bank {
String name;
- public Bank(String name) {
+ public Bank(final String name) {
this.name = name;
}
}
@@ -232,7 +232,7 @@ public class MultilineRecursiveToStringStyleTest {
Bank bank;
List<Account> accounts;
- public Customer(String name) {
+ public Customer(final String name) {
this.name = name;
}
}
@@ -254,7 +254,7 @@ public class MultilineRecursiveToStringStyleTest {
double amount;
String date;
- public Transaction(String datum, double betrag) {
+ public Transaction(final String datum, final double betrag) {
this.date = datum;
this.amount = betrag;
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/test/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreakerTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreakerTest.java b/src/test/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreakerTest.java
index df0ef7d..d8515e7 100644
--- a/src/test/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreakerTest.java
+++ b/src/test/java/org/apache/commons/lang3/concurrent/EventCountCircuitBreakerTest.java
@@ -330,9 +330,9 @@ public class EventCountCircuitBreakerTest {
/** The current time in nanoseconds. */
private long currentTime;
- public EventCountCircuitBreakerTestImpl(int openingThreshold, long openingInterval,
- TimeUnit openingUnit, int closingThreshold, long closingInterval,
- TimeUnit closingUnit) {
+ public EventCountCircuitBreakerTestImpl(final int openingThreshold, final long openingInterval,
+ final TimeUnit openingUnit, final int closingThreshold, final long closingInterval,
+ final TimeUnit closingUnit) {
super(openingThreshold, openingInterval, openingUnit, closingThreshold,
closingInterval, closingUnit);
}
@@ -343,7 +343,7 @@ public class EventCountCircuitBreakerTest {
* @param time the time to set
* @return a reference to this object
*/
- public EventCountCircuitBreakerTestImpl at(long time) {
+ public EventCountCircuitBreakerTestImpl at(final long time) {
currentTime = time;
return this;
}
@@ -374,12 +374,12 @@ public class EventCountCircuitBreakerTest {
*
* @param source the expected event source
*/
- public ChangeListener(Object source) {
+ public ChangeListener(final Object source) {
expectedSource = source;
changedValues = new ArrayList<>();
}
- public void propertyChange(PropertyChangeEvent evt) {
+ public void propertyChange(final PropertyChangeEvent evt) {
assertEquals("Wrong event source", expectedSource, evt.getSource());
assertEquals("Wrong property name", "open", evt.getPropertyName());
Boolean newValue = (Boolean) evt.getNewValue();
@@ -393,7 +393,7 @@ public class EventCountCircuitBreakerTest {
*
* @param values the expected values
*/
- public void verify(Boolean... values) {
+ public void verify(final Boolean... values) {
assertArrayEquals(values,
changedValues.toArray(new Boolean[changedValues.size()]));
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java b/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java
index 3511021..15be2f4 100644
--- a/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java
+++ b/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java
@@ -546,7 +546,7 @@ public class TimedSemaphoreTest {
/** Flag whether a permit could be acquired. */
private boolean acquired;
- public TryAcquireThread(TimedSemaphore s, CountDownLatch l) {
+ public TryAcquireThread(final TimedSemaphore s, final CountDownLatch l) {
semaphore = s;
latch = l;
}
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/test/java/org/apache/commons/lang3/reflect/ConstructorUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/lang3/reflect/ConstructorUtilsTest.java b/src/test/java/org/apache/commons/lang3/reflect/ConstructorUtilsTest.java
index 42e1295..c019cbe 100644
--- a/src/test/java/org/apache/commons/lang3/reflect/ConstructorUtilsTest.java
+++ b/src/test/java/org/apache/commons/lang3/reflect/ConstructorUtilsTest.java
@@ -77,12 +77,12 @@ public class ConstructorUtilsTest {
varArgs = s;
}
- public TestBean(final Integer i, String... s) {
+ public TestBean(final Integer i, final String... s) {
toString = "(Integer, String...)";
varArgs = s;
}
- public TestBean(final Integer first, int... args) {
+ public TestBean(final Integer first, final int... args) {
toString = "(Integer, String...)";
varArgs = new String[args.length];
for(int i = 0; i< args.length; ++i) {
http://git-wip-us.apache.org/repos/asf/commons-lang/blob/aadb9a31/src/test/java/org/apache/commons/lang3/reflect/MethodUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/lang3/reflect/MethodUtilsTest.java b/src/test/java/org/apache/commons/lang3/reflect/MethodUtilsTest.java
index 5e74fc9..a823325 100644
--- a/src/test/java/org/apache/commons/lang3/reflect/MethodUtilsTest.java
+++ b/src/test/java/org/apache/commons/lang3/reflect/MethodUtilsTest.java
@@ -179,57 +179,57 @@ public class MethodUtilsTest {
return "foo(Object...)";
}
- public int[] unboxing(int... values) {
+ public int[] unboxing(final int... values) {
return values;
}
// This method is overloaded for the wrapper class for every primitive type, plus the common supertypes
// Number and Object. This is an acid test since it easily leads to ambiguous methods.
- public static String varOverload(Byte... args) { return "Byte..."; }
- public static String varOverload(Character... args) { return "Character..."; }
- public static String varOverload(Short... args) { return "Short..."; }
- public static String varOverload(Boolean... args) { return "Boolean..."; }
- public static String varOverload(Float... args) { return "Float..."; }
- public static String varOverload(Double... args) { return "Double..."; }
- public static String varOverload(Integer... args) { return "Integer..."; }
- public static String varOverload(Long... args) { return "Long..."; }
- public static String varOverload(Number... args) { return "Number..."; }
- public static String varOverload(Object... args) { return "Object..."; }
- public static String varOverload(String... args) { return "String..."; }
+ public static String varOverload(final Byte... args) { return "Byte..."; }
+ public static String varOverload(final Character... args) { return "Character..."; }
+ public static String varOverload(final Short... args) { return "Short..."; }
+ public static String varOverload(final Boolean... args) { return "Boolean..."; }
+ public static String varOverload(final Float... args) { return "Float..."; }
+ public static String varOverload(final Double... args) { return "Double..."; }
+ public static String varOverload(final Integer... args) { return "Integer..."; }
+ public static String varOverload(final Long... args) { return "Long..."; }
+ public static String varOverload(final Number... args) { return "Number..."; }
+ public static String varOverload(final Object... args) { return "Object..."; }
+ public static String varOverload(final String... args) { return "String..."; }
// This method is overloaded for the wrapper class for every numeric primitive type, plus the common
// supertype Number
- public static String numOverload(Byte... args) { return "Byte..."; }
- public static String numOverload(Short... args) { return "Short..."; }
- public static String numOverload(Float... args) { return "Float..."; }
- public static String numOverload(Double... args) { return "Double..."; }
- public static String numOverload(Integer... args) { return "Integer..."; }
- public static String numOverload(Long... args) { return "Long..."; }
- public static String numOverload(Number... args) { return "Number..."; }
+ public static String numOverload(final Byte... args) { return "Byte..."; }
+ public static String numOverload(final Short... args) { return "Short..."; }
+ public static String numOverload(final Float... args) { return "Float..."; }
+ public static String numOverload(final Double... args) { return "Double..."; }
+ public static String numOverload(final Integer... args) { return "Integer..."; }
+ public static String numOverload(final Long... args) { return "Long..."; }
+ public static String numOverload(final Number... args) { return "Number..."; }
// These varOverloadEcho and varOverloadEchoStatic methods are designed to verify that
// not only is the correct overloaded variant invoked, but that the varags arguments
// are also delivered correctly to the method.
- public ImmutablePair<String, Object[]> varOverloadEcho(String... args) {
+ public ImmutablePair<String, Object[]> varOverloadEcho(final String... args) {
return new ImmutablePair<String, Object[]>("String...", args);
}
- public ImmutablePair<String, Object[]> varOverloadEcho(Number... args) {
+ public ImmutablePair<String, Object[]> varOverloadEcho(final Number... args) {
return new ImmutablePair<String, Object[]>("Number...", args);
}
- public static ImmutablePair<String, Object[]> varOverloadEchoStatic(String... args) {
+ public static ImmutablePair<String, Object[]> varOverloadEchoStatic(final String... args) {
return new ImmutablePair<String, Object[]>("String...", args);
}
- public static ImmutablePair<String, Object[]> varOverloadEchoStatic(Number... args) {
+ public static ImmutablePair<String, Object[]> varOverloadEchoStatic(final Number... args) {
return new ImmutablePair<String, Object[]>("Number...", args);
}
- static void verify(ImmutablePair<String, Object[]> a, ImmutablePair<String, Object[]> b) {
+ static void verify(final ImmutablePair<String, Object[]> a, final ImmutablePair<String, Object[]> b) {
assertEquals(a.getLeft(), b.getLeft());
assertArrayEquals(a.getRight(), b.getRight());
}
- static void verify(ImmutablePair<String, Object[]> a, Object _b) {
+ static void verify(final ImmutablePair<String, Object[]> a, final Object _b) {
@SuppressWarnings("unchecked")
final ImmutablePair<String, Object[]> b = (ImmutablePair<String, Object[]>) _b;
verify(a, b);