Add Checkstyle UnnecessaryParentheses

Author: garydgregory

src/conf/checkstyle.xml

@@ -91,6 +91,7 @@ limitations under the License.
     <module name="SuperFinalize" />
     <module name="TypecastParenPad" />    
     <module name="UpperEll" />
+    <module name="UnnecessaryParentheses" />
     <module name="UnusedImports"/>
     <module name="WhitespaceAfter"/>
     <module name="WhitespaceAround"/>

src/test/java/org/apache/commons/collections4/MapUtilsTest.java

@@ -416,7 +416,7 @@ public void testGetByteValue() {
         assertEquals(val, MapUtils.getByteValue(in, "key", val), 0);
         assertEquals(val, MapUtils.getByteValue(in, "key"), 0);
         assertEquals(val, MapUtils.getByteValue(in, "noKey", val), 0);
-        assertEquals(val, MapUtils.getByteValue(in, "noKey", key -> ((byte) 100)), 0);
+        assertEquals(val, MapUtils.getByteValue(in, "noKey", key -> (byte) 100), 0);
         assertEquals(0, MapUtils.getByteValue(in, "noKey"), 0);
         assertEquals(val, MapUtils.getByte(in, "key", val), 0);
         assertEquals(val, MapUtils.getByte(in, "noKey", val), 0);

src/test/java/org/apache/commons/collections4/bloomfilter/LayeredBloomFilterTest.java

@@ -235,7 +235,7 @@ public void testCleanup() {
         final LayerManager<NumberedBloomFilter> layerManager = LayerManager.<NumberedBloomFilter>builder()
                 .setSupplier(() -> new NumberedBloomFilter(getTestShape(), 3, sequence[0]++))
                 .setExtendCheck(ExtendCheck.neverAdvance())
-                .setCleanup(ll -> ll.removeIf(f -> (f.value-- == 0))).get();
+                .setCleanup(ll -> ll.removeIf(f -> f.value-- == 0)).get();
         final LayeredBloomFilter<NumberedBloomFilter> underTest = new LayeredBloomFilter<>(getTestShape(), layerManager);
         assertEquals(1, underTest.getDepth());
         underTest.merge(TestingHashers.randomHasher());