PMD: UselessParentheses
diff --git a/src/test/java/org/apache/commons/text/StrBuilderTest.java b/src/test/java/org/apache/commons/text/StrBuilderTest.java
index 2b06b8c..738fa93 100644
--- a/src/test/java/org/apache/commons/text/StrBuilderTest.java
+++ b/src/test/java/org/apache/commons/text/StrBuilderTest.java
@@ -678,7 +678,7 @@
@Test
public void testDeleteCharAtWithNegative() {
- assertThatExceptionOfType(StringIndexOutOfBoundsException.class).isThrownBy(() -> new StrBuilder().deleteCharAt((-1258)));
+ assertThatExceptionOfType(StringIndexOutOfBoundsException.class).isThrownBy(() -> new StrBuilder().deleteCharAt(-1258));
}
@Test
diff --git a/src/test/java/org/apache/commons/text/StrSubstitutorTest.java b/src/test/java/org/apache/commons/text/StrSubstitutorTest.java
index 7bd6486..858b43b 100644
--- a/src/test/java/org/apache/commons/text/StrSubstitutorTest.java
+++ b/src/test/java/org/apache/commons/text/StrSubstitutorTest.java
@@ -476,7 +476,7 @@
final StrSubstitutor strSubstitutor =
new StrSubstitutor(strLookupMapStrLookup, strMatcher, strMatcher, 'b', strMatcher);
- assertFalse(strSubstitutor.replaceIn((StringBuilder) null, 1315, (-1369)));
+ assertFalse(strSubstitutor.replaceIn((StringBuilder) null, 1315, -1369));
assertEquals('b', strSubstitutor.getEscapeChar());
assertFalse(strSubstitutor.isPreserveEscapes());
}
diff --git a/src/test/java/org/apache/commons/text/StringEscapeUtilsTest.java b/src/test/java/org/apache/commons/text/StringEscapeUtilsTest.java
index 21a73fe..4651a4e 100644
--- a/src/test/java/org/apache/commons/text/StringEscapeUtilsTest.java
+++ b/src/test/java/org/apache/commons/text/StringEscapeUtilsTest.java
@@ -68,7 +68,7 @@
private void assertEscapeJava(final String expected, final String original, String message) throws IOException {
final String converted = StringEscapeUtils.escapeJava(original);
- message = "escapeJava(String) failed" + (message == null ? "" : (": " + message));
+ message = "escapeJava(String) failed" + (message == null ? "" : ": " + message);
assertEquals(expected, converted, message);
final StringWriter writer = new StringWriter();
@@ -85,7 +85,7 @@
final String actual = StringEscapeUtils.unescapeJava(original);
assertEquals(unescaped, actual, "unescape(String) failed"
- + (message == null ? "" : (": " + message))
+ + (message == null ? "" : ": " + message)
+ ": expected '" + StringEscapeUtils.escapeJava(unescaped)
// we escape this so we can see it in the error message
+ "' actual '" + StringEscapeUtils.escapeJava(actual) + "'");
diff --git a/src/test/java/org/apache/commons/text/TextStringBuilderTest.java b/src/test/java/org/apache/commons/text/TextStringBuilderTest.java
index 3e36f7a..c600185 100644
--- a/src/test/java/org/apache/commons/text/TextStringBuilderTest.java
+++ b/src/test/java/org/apache/commons/text/TextStringBuilderTest.java
@@ -695,7 +695,7 @@
@Test
public void testDeleteCharAtWithNegative() {
- assertThatExceptionOfType(StringIndexOutOfBoundsException.class).isThrownBy(() -> new TextStringBuilder().deleteCharAt((-1258)));
+ assertThatExceptionOfType(StringIndexOutOfBoundsException.class).isThrownBy(() -> new TextStringBuilder().deleteCharAt(-1258));
}
@Test
diff --git a/src/test/java/org/apache/commons/text/WordUtilsTest.java b/src/test/java/org/apache/commons/text/WordUtilsTest.java
index 54e4969..6c8d97b 100644
--- a/src/test/java/org/apache/commons/text/WordUtilsTest.java
+++ b/src/test/java/org/apache/commons/text/WordUtilsTest.java
@@ -55,7 +55,7 @@
@Test
public void testAbbreviateForNullAndEmptyString() {
- assertThat((WordUtils.abbreviate(null, 1, -1, ""))).isNull();
+ assertThat(WordUtils.abbreviate(null, 1, -1, "")).isNull();
assertThat(WordUtils.abbreviate("", 1, -1, "")).isEqualTo(StringUtils.EMPTY);
assertThat(WordUtils.abbreviate("0123456790", 0, 0, "")).isEqualTo("");
assertThat(WordUtils.abbreviate(" 0123456790", 0, -1, "")).isEqualTo("");
diff --git a/src/test/java/org/apache/commons/text/jmh/DoubleFormatPerformance.java b/src/test/java/org/apache/commons/text/jmh/DoubleFormatPerformance.java
index bbae86a..4b58491 100644
--- a/src/test/java/org/apache/commons/text/jmh/DoubleFormatPerformance.java
+++ b/src/test/java/org/apache/commons/text/jmh/DoubleFormatPerformance.java
@@ -102,11 +102,11 @@
*/
private static double randomDouble(final int minExp, final int maxExp, final UniformRandomProvider rng) {
// Create random doubles using random bits in the sign bit and the mantissa.
- final long mask = ((1L << 52) - 1) | 1L << 63;
+ final long mask = (1L << 52) - 1 | 1L << 63;
final long bits = rng.nextLong() & mask;
// The exponent must be unsigned so + 1023 to the signed exponent
final long exp = rng.nextInt(maxExp - minExp + 1) + minExp + 1023;
- return Double.longBitsToDouble(bits | (exp << 52));
+ return Double.longBitsToDouble(bits | exp << 52);
}
/** Create an array with the given length containing random doubles with exponents in the range
diff --git a/src/test/java/org/apache/commons/text/numbers/DoubleFormatTest.java b/src/test/java/org/apache/commons/text/numbers/DoubleFormatTest.java
index 052ad59..402bb93 100644
--- a/src/test/java/org/apache/commons/text/numbers/DoubleFormatTest.java
+++ b/src/test/java/org/apache/commons/text/numbers/DoubleFormatTest.java
@@ -135,11 +135,11 @@
*/
private static double randomDouble(final int minExp, final int maxExp, final Random rnd) {
// Create random doubles using random bits in the sign bit and the mantissa.
- final long mask = ((1L << 52) - 1) | 1L << 63;
+ final long mask = (1L << 52) - 1 | 1L << 63;
final long bits = rnd.nextLong() & mask;
// The exponent must be unsigned so + 1023 to the signed exponent
final long exp = rnd.nextInt(maxExp - minExp + 1) + minExp + 1023;
- return Double.longBitsToDouble(bits | (exp << 52));
+ return Double.longBitsToDouble(bits | exp << 52);
}
/** Create a random double value using the full range of exponent values.
diff --git a/src/test/java/org/apache/commons/text/numbers/ParsedDecimalTest.java b/src/test/java/org/apache/commons/text/numbers/ParsedDecimalTest.java
index 2135458..76c2e16 100644
--- a/src/test/java/org/apache/commons/text/numbers/ParsedDecimalTest.java
+++ b/src/test/java/org/apache/commons/text/numbers/ParsedDecimalTest.java
@@ -202,10 +202,10 @@
}
private static double createRandomDouble(final UniformRandomProvider rng) {
- final long mask = ((1L << 52) - 1) | 1L << 63;
+ final long mask = (1L << 52) - 1 | 1L << 63;
final long bits = rng.nextLong() & mask;
final long exp = rng.nextInt(2045) + 1;
- return Double.longBitsToDouble(bits | (exp << 52));
+ return Double.longBitsToDouble(bits | exp << 52);
}
/** Get the raw digits in the given decimal as a string.
diff --git a/src/test/java/org/apache/commons/text/similarity/IntersectionResultTest.java b/src/test/java/org/apache/commons/text/similarity/IntersectionResultTest.java
index ea8520b..f9c53df 100644
--- a/src/test/java/org/apache/commons/text/similarity/IntersectionResultTest.java
+++ b/src/test/java/org/apache/commons/text/similarity/IntersectionResultTest.java
@@ -43,7 +43,7 @@
Assertions.assertNotEquals(results[i], something);
Assertions.assertNotEquals(null, results[i]);
for (int j = 0; j < results.length; j++) {
- Assertions.assertEquals(results[i].equals(results[j]), (i == j));
+ Assertions.assertEquals(results[i].equals(results[j]), i == j);
}
}
}
diff --git a/src/test/java/org/apache/commons/text/similarity/IntersectionSimilarityTest.java b/src/test/java/org/apache/commons/text/similarity/IntersectionSimilarityTest.java
index 52a837e..76b1d92 100644
--- a/src/test/java/org/apache/commons/text/similarity/IntersectionSimilarityTest.java
+++ b/src/test/java/org/apache/commons/text/similarity/IntersectionSimilarityTest.java
@@ -58,7 +58,7 @@
for (int i = 1; i < length; i++) {
final char ch1 = ch2;
ch2 = sequence.charAt(i);
- list.add(Integer.valueOf((ch1 << 16) | ch2));
+ list.add(Integer.valueOf(ch1 << 16 | ch2));
}
}
return list;
@@ -79,7 +79,7 @@
for (int i = 1; i < length; i++) {
final char ch1 = ch2;
ch2 = sequence.charAt(i);
- set.add(Integer.valueOf((ch1 << 16) | ch2));
+ set.add(Integer.valueOf(ch1 << 16 | ch2));
}
}
return set;
@@ -165,7 +165,7 @@
for (int i = 1; i < word.length(); i++) {
final char ch1 = ch2;
ch2 = Character.toUpperCase(word.charAt(i));
- set.add(Integer.valueOf((ch1 << 16) | ch2));
+ set.add(Integer.valueOf(ch1 << 16 | ch2));
}
}
}
diff --git a/src/test/java/org/apache/commons/text/similarity/JaccardDistanceTest.java b/src/test/java/org/apache/commons/text/similarity/JaccardDistanceTest.java
index 531fbdf..bf724a2 100644
--- a/src/test/java/org/apache/commons/text/similarity/JaccardDistanceTest.java
+++ b/src/test/java/org/apache/commons/text/similarity/JaccardDistanceTest.java
@@ -40,17 +40,17 @@
assertEquals(0.0, classBeingTested.apply("", ""));
assertEquals(1.0, classBeingTested.apply("left", ""));
assertEquals(1.0, classBeingTested.apply("", "right"));
- assertEquals(1.0 - (3.0 / 4), classBeingTested.apply("frog", "fog"));
+ assertEquals(1.0 - 3.0 / 4, classBeingTested.apply("frog", "fog"));
assertEquals(1.0, classBeingTested.apply("fly", "ant"));
- assertEquals(1.0 - (2.0 / 9), classBeingTested.apply("elephant", "hippo"));
- assertEquals(1.0 - (7.0 / 11), classBeingTested.apply("ABC Corporation", "ABC Corp"));
- assertEquals(1.0 - (13.0 / 17),
+ assertEquals(1.0 - 2.0 / 9, classBeingTested.apply("elephant", "hippo"));
+ assertEquals(1.0 - 7.0 / 11, classBeingTested.apply("ABC Corporation", "ABC Corp"));
+ assertEquals(1.0 - 13.0 / 17,
classBeingTested.apply("D N H Enterprises Inc", "D & H Enterprises, Inc."));
- assertEquals(1.0 - (16.0 / 18),
+ assertEquals(1.0 - 16.0 / 18,
classBeingTested.apply("My Gym Children's Fitness Center", "My Gym. Childrens Fitness"));
- assertEquals(1.0 - (9.0 / 10), classBeingTested.apply("PENNSYLVANIA", "PENNCISYLVNIA"));
- assertEquals(1.0 - (1.0 / 8), classBeingTested.apply("left", "right"));
- assertEquals(1.0 - (1.0 / 8), classBeingTested.apply("leettteft", "ritttght"));
+ assertEquals(1.0 - 9.0 / 10, classBeingTested.apply("PENNSYLVANIA", "PENNCISYLVNIA"));
+ assertEquals(1.0 - 1.0 / 8, classBeingTested.apply("left", "right"));
+ assertEquals(1.0 - 1.0 / 8, classBeingTested.apply("leettteft", "ritttght"));
assertEquals(0.0, classBeingTested.apply("the same string", "the same string"));
}
diff --git a/src/test/java/org/apache/commons/text/similarity/ParameterizedEditDistanceFromTest.java b/src/test/java/org/apache/commons/text/similarity/ParameterizedEditDistanceFromTest.java
index 618c1c5..edc496d 100644
--- a/src/test/java/org/apache/commons/text/similarity/ParameterizedEditDistanceFromTest.java
+++ b/src/test/java/org/apache/commons/text/similarity/ParameterizedEditDistanceFromTest.java
@@ -47,7 +47,7 @@
Arguments.of(new LevenshteinDistance(0), "go", "go", 0),
Arguments.of(
- (EditDistance<Boolean>) (left, right) -> left == right || (left != null && left.equals(right)),
+ (EditDistance<Boolean>) (left, right) -> left == right || left != null && left.equals(right),
"Bob's your uncle.",
"Every good boy does fine.",
false));
diff --git a/src/test/java/org/apache/commons/text/similarity/ParameterizedSimilarityScoreFromTest.java b/src/test/java/org/apache/commons/text/similarity/ParameterizedSimilarityScoreFromTest.java
index b511238..5feda36 100644
--- a/src/test/java/org/apache/commons/text/similarity/ParameterizedSimilarityScoreFromTest.java
+++ b/src/test/java/org/apache/commons/text/similarity/ParameterizedSimilarityScoreFromTest.java
@@ -39,7 +39,7 @@
Arguments.of(
(SimilarityScore<Boolean>) (left, right) -> left == right
- || (left != null && left.equals(right)),
+ || left != null && left.equals(right),
"Bob's your uncle.",
"Every good boy does fine.",
false