Repository: commons-collections Updated Branches: refs/heads/master 92c5e6f1e -> 8b66a577f
No need to nest in an else clause. Project: http://git-wip-us.apache.org/repos/asf/commons-collections/repo Commit: http://git-wip-us.apache.org/repos/asf/commons-collections/commit/8b66a577 Tree: http://git-wip-us.apache.org/repos/asf/commons-collections/tree/8b66a577 Diff: http://git-wip-us.apache.org/repos/asf/commons-collections/diff/8b66a577 Branch: refs/heads/master Commit: 8b66a577f44a16e259ad5ba11695d20e7c3a9cde Parents: 92c5e6f Author: Gary Gregory <[email protected]> Authored: Thu Dec 28 13:17:33 2017 -0700 Committer: Gary Gregory <[email protected]> Committed: Thu Dec 28 13:17:33 2017 -0700 ---------------------------------------------------------------------- .../apache/commons/collections4/ArrayStack.java | 9 +-- .../commons/collections4/CollectionUtils.java | 60 ++++++++++---------- .../commons/collections4/EnumerationUtils.java | 3 +- .../commons/collections4/FluentIterable.java | 3 +- .../commons/collections4/IterableUtils.java | 15 ++--- .../commons/collections4/bag/CollectionBag.java | 10 ++-- .../collections4/bag/CollectionSortedBag.java | 10 ++-- .../collections4/bidimap/TreeBidiMap.java | 3 +- .../collections4/functors/EqualPredicate.java | 3 +- .../collections4/functors/IfTransformer.java | 3 +- .../collections4/iterators/ZippingIterator.java | 5 +- .../multimap/AbstractMultiValuedMap.java | 11 ++-- .../collections4/CollectionUtilsTest.java | 3 +- .../bidimap/DualTreeBidiMap2Test.java | 3 +- .../bidimap/DualTreeBidiMapTest.java | 3 +- .../iterators/NodeListIteratorTest.java | 11 ++-- .../collections4/list/AbstractListTest.java | 3 +- .../commons/collections4/map/LRUMapTest.java | 3 +- .../map/ReferenceIdentityMapTest.java | 9 ++- .../collections4/map/ReferenceMapTest.java | 9 ++- .../map/TransformedSortedMapTest.java | 3 +- 21 files changed, 75 insertions(+), 107 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/ArrayStack.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/ArrayStack.java b/src/main/java/org/apache/commons/collections4/ArrayStack.java index e346ed0..5a6e1f4 100644 --- a/src/main/java/org/apache/commons/collections4/ArrayStack.java +++ b/src/main/java/org/apache/commons/collections4/ArrayStack.java @@ -87,9 +87,8 @@ public class ArrayStack<E> extends ArrayList<E> { final int n = size(); if (n <= 0) { throw new EmptyStackException(); - } else { - return get(n - 1); } + return get(n - 1); } /** @@ -105,9 +104,8 @@ public class ArrayStack<E> extends ArrayList<E> { final int m = (size() - n) - 1; if (m < 0) { throw new EmptyStackException(); - } else { - return get(m); } + return get(m); } /** @@ -120,9 +118,8 @@ public class ArrayStack<E> extends ArrayList<E> { final int n = size(); if (n <= 0) { throw new EmptyStackException(); - } else { - return remove(n - 1); } + return remove(n - 1); } /** http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/CollectionUtils.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/CollectionUtils.java b/src/main/java/org/apache/commons/collections4/CollectionUtils.java index 5b974cb..7e6bc58 100644 --- a/src/main/java/org/apache/commons/collections4/CollectionUtils.java +++ b/src/main/java/org/apache/commons/collections4/CollectionUtils.java @@ -368,30 +368,29 @@ public class CollectionUtils { public static boolean containsAll(final Collection<?> coll1, final Collection<?> coll2) { if (coll2.isEmpty()) { return true; - } else { - final Iterator<?> it = coll1.iterator(); - final Set<Object> elementsAlreadySeen = new HashSet<>(); - for (final Object nextElement : coll2) { - if (elementsAlreadySeen.contains(nextElement)) { - continue; - } + } + final Iterator<?> it = coll1.iterator(); + final Set<Object> elementsAlreadySeen = new HashSet<>(); + for (final Object nextElement : coll2) { + if (elementsAlreadySeen.contains(nextElement)) { + continue; + } - boolean foundCurrentElement = false; - while (it.hasNext()) { - final Object p = it.next(); - elementsAlreadySeen.add(p); - if (nextElement == null ? p == null : nextElement.equals(p)) { - foundCurrentElement = true; - break; - } + boolean foundCurrentElement = false; + while (it.hasNext()) { + final Object p = it.next(); + elementsAlreadySeen.add(p); + if (nextElement == null ? p == null : nextElement.equals(p)) { + foundCurrentElement = true; + break; } + } - if (!foundCurrentElement) { - return false; - } + if (!foundCurrentElement) { + return false; } - return true; } + return true; } /** @@ -1584,21 +1583,20 @@ public class CollectionUtils { final Iterator<O> iterator = new CollatingIterator<>(c, a.iterator(), b.iterator()); if (includeDuplicates) { return IteratorUtils.toList(iterator, totalSize); - } else { - final ArrayList<O> mergedList = new ArrayList<>(totalSize); + } + final ArrayList<O> mergedList = new ArrayList<>(totalSize); - O lastItem = null; - while (iterator.hasNext()) { - final O item = iterator.next(); - if (lastItem == null || !lastItem.equals(item)) { - mergedList.add(item); - } - lastItem = item; + O lastItem = null; + while (iterator.hasNext()) { + final O item = iterator.next(); + if (lastItem == null || !lastItem.equals(item)) { + mergedList.add(item); } - - mergedList.trimToSize(); - return mergedList; + lastItem = item; } + + mergedList.trimToSize(); + return mergedList; } //----------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/EnumerationUtils.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/EnumerationUtils.java b/src/main/java/org/apache/commons/collections4/EnumerationUtils.java index 7e726d0..9c5fabe 100644 --- a/src/main/java/org/apache/commons/collections4/EnumerationUtils.java +++ b/src/main/java/org/apache/commons/collections4/EnumerationUtils.java @@ -57,9 +57,8 @@ public class EnumerationUtils { i--; if (i == -1) { return e.nextElement(); - } else { - e.nextElement(); } + e.nextElement(); } throw new IndexOutOfBoundsException("Entry does not exist: " + i); } http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/FluentIterable.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/FluentIterable.java b/src/main/java/org/apache/commons/collections4/FluentIterable.java index 46acdbf..7893e7a 100644 --- a/src/main/java/org/apache/commons/collections4/FluentIterable.java +++ b/src/main/java/org/apache/commons/collections4/FluentIterable.java @@ -124,9 +124,8 @@ public class FluentIterable<E> implements Iterable<E> { IterableUtils.checkNotNull(iterable); if (iterable instanceof FluentIterable<?>) { return (FluentIterable<T>) iterable; - } else { - return new FluentIterable<>(iterable); } + return new FluentIterable<>(iterable); } // Constructor http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/IterableUtils.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/IterableUtils.java b/src/main/java/org/apache/commons/collections4/IterableUtils.java index ad24eeb..1783641 100644 --- a/src/main/java/org/apache/commons/collections4/IterableUtils.java +++ b/src/main/java/org/apache/commons/collections4/IterableUtils.java @@ -175,9 +175,8 @@ public class IterableUtils { protected Iterator<? extends E> nextIterator(int count) { if (count > iterables.length) { return null; - } else { - return iterables[count - 1].iterator(); } + return iterables[count - 1].iterator(); } }; } @@ -328,9 +327,8 @@ public class IterableUtils { protected Iterator<? extends E> nextIterator(int count) { if (IterableUtils.isEmpty(iterable)) { return null; - } else { - return iterable.iterator(); } + return iterable.iterator(); } }; } @@ -690,9 +688,8 @@ public class IterableUtils { public static boolean isEmpty(final Iterable<?> iterable) { if (iterable instanceof Collection<?>) { return ((Collection<?>) iterable).isEmpty(); - } else { - return IteratorUtils.isEmpty(emptyIteratorIfNull(iterable)); } + return IteratorUtils.isEmpty(emptyIteratorIfNull(iterable)); } /** @@ -708,9 +705,8 @@ public class IterableUtils { public static <E> boolean contains(final Iterable<E> iterable, final Object object) { if (iterable instanceof Collection<?>) { return ((Collection<E>) iterable).contains(object); - } else { - return IteratorUtils.contains(emptyIteratorIfNull(iterable), object); } + return IteratorUtils.contains(emptyIteratorIfNull(iterable), object); } /** @@ -788,9 +784,8 @@ public class IterableUtils { public static int size(final Iterable<?> iterable) { if (iterable instanceof Collection<?>) { return ((Collection<?>) iterable).size(); - } else { - return IteratorUtils.size(emptyIteratorIfNull(iterable)); } + return IteratorUtils.size(emptyIteratorIfNull(iterable)); } /** http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/bag/CollectionBag.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/bag/CollectionBag.java b/src/main/java/org/apache/commons/collections4/bag/CollectionBag.java index 5ea08f2..7fbe346 100644 --- a/src/main/java/org/apache/commons/collections4/bag/CollectionBag.java +++ b/src/main/java/org/apache/commons/collections4/bag/CollectionBag.java @@ -177,10 +177,9 @@ public final class CollectionBag<E> extends AbstractBagDecorator<E> { result = result || changed; } return result; - } else { - // let the decorated bag handle the case of null argument - return decorated().removeAll(null); } + // let the decorated bag handle the case of null argument + return decorated().removeAll(null); } /** @@ -212,10 +211,9 @@ public final class CollectionBag<E> extends AbstractBagDecorator<E> { } } return modified; - } else { - // let the decorated bag handle the case of null argument - return decorated().retainAll(null); } + // let the decorated bag handle the case of null argument + return decorated().retainAll(null); } //----------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/bag/CollectionSortedBag.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/bag/CollectionSortedBag.java b/src/main/java/org/apache/commons/collections4/bag/CollectionSortedBag.java index 9459834..88a4c76 100644 --- a/src/main/java/org/apache/commons/collections4/bag/CollectionSortedBag.java +++ b/src/main/java/org/apache/commons/collections4/bag/CollectionSortedBag.java @@ -130,10 +130,9 @@ public final class CollectionSortedBag<E> extends AbstractSortedBagDecorator<E> result = result || changed; } return result; - } else { - // let the decorated bag handle the case of null argument - return decorated().removeAll(null); } + // let the decorated bag handle the case of null argument + return decorated().removeAll(null); } @Override @@ -148,10 +147,9 @@ public final class CollectionSortedBag<E> extends AbstractSortedBagDecorator<E> } } return modified; - } else { - // let the decorated bag handle the case of null argument - return decorated().retainAll(null); } + // let the decorated bag handle the case of null argument + return decorated().retainAll(null); } //----------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/bidimap/TreeBidiMap.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/bidimap/TreeBidiMap.java b/src/main/java/org/apache/commons/collections4/bidimap/TreeBidiMap.java index 1f48b9c..5622cc1 100644 --- a/src/main/java/org/apache/commons/collections4/bidimap/TreeBidiMap.java +++ b/src/main/java/org/apache/commons/collections4/bidimap/TreeBidiMap.java @@ -590,9 +590,8 @@ public class TreeBidiMap<K extends Comparable<K>, V extends Comparable<V>> if (cmp == 0) { rval = node; break; - } else { - node = cmp < 0 ? node.getLeft(dataElement) : node.getRight(dataElement); } + node = cmp < 0 ? node.getLeft(dataElement) : node.getRight(dataElement); } return rval; http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/functors/EqualPredicate.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/functors/EqualPredicate.java b/src/main/java/org/apache/commons/collections4/functors/EqualPredicate.java index 04b5530..90ed037 100644 --- a/src/main/java/org/apache/commons/collections4/functors/EqualPredicate.java +++ b/src/main/java/org/apache/commons/collections4/functors/EqualPredicate.java @@ -104,9 +104,8 @@ public final class EqualPredicate<T> implements Predicate<T>, Serializable { public boolean evaluate(final T object) { if (equator != null) { return equator.equate(iValue, object); - } else { - return iValue.equals(object); } + return iValue.equals(object); } /** http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/functors/IfTransformer.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/functors/IfTransformer.java b/src/main/java/org/apache/commons/collections4/functors/IfTransformer.java index 081513c..75dc7b9 100644 --- a/src/main/java/org/apache/commons/collections4/functors/IfTransformer.java +++ b/src/main/java/org/apache/commons/collections4/functors/IfTransformer.java @@ -120,9 +120,8 @@ public class IfTransformer<I, O> implements Transformer<I, O>, Serializable { public O transform(final I input) { if(iPredicate.evaluate(input)){ return iTrueTransformer.transform(input); - } else { - return iFalseTransformer.transform(input); } + return iFalseTransformer.transform(input); } /** http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/iterators/ZippingIterator.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/iterators/ZippingIterator.java b/src/main/java/org/apache/commons/collections4/iterators/ZippingIterator.java index 6bdf20a..e82ee9c 100644 --- a/src/main/java/org/apache/commons/collections4/iterators/ZippingIterator.java +++ b/src/main/java/org/apache/commons/collections4/iterators/ZippingIterator.java @@ -116,10 +116,9 @@ public class ZippingIterator<E> implements Iterator<E> { if (childIterator.hasNext()) { nextIterator = childIterator; return true; - } else { - // iterator is exhausted, remove it - iterators.remove(); } + // iterator is exhausted, remove it + iterators.remove(); } return false; } http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java b/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java index 620667b..4772468 100644 --- a/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java +++ b/src/main/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMap.java @@ -259,12 +259,10 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K, if (coll.add(value)) { map.put(key, coll); return true; - } else { - return false; } - } else { - return coll.add(value); + return false; } + return coll.add(value); } /** @@ -355,10 +353,9 @@ public abstract class AbstractMultiValuedMap<K, V> implements MultiValuedMap<K, if (values instanceof Collection<?>) { Collection<? extends V> valueCollection = (Collection<? extends V>) values; return !valueCollection.isEmpty() && get(key).addAll(valueCollection); - } else { - Iterator<? extends V> it = values.iterator(); - return it.hasNext() && CollectionUtils.addAll(get(key), it); } + Iterator<? extends V> it = values.iterator(); + return it.hasNext() && CollectionUtils.addAll(get(key), it); } @Override http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java b/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java index dcfa872..d61aa42 100644 --- a/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java +++ b/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java @@ -568,9 +568,8 @@ public class CollectionUtilsTest extends MockTestCase { public boolean equate(final Integer o1, final Integer o2) { if (o1.intValue() % 2 == 0 ^ o2.intValue() % 2 == 0) { return false; - } else { - return true; } + return true; } @Override http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap2Test.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap2Test.java b/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap2Test.java index d4d8461..2c3978d 100644 --- a/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap2Test.java +++ b/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMap2Test.java @@ -171,9 +171,8 @@ public class DualTreeBidiMap2Test<K extends Comparable<K>, V extends Comparable< preTail + "bulkTestMapValues.testCollectionRemoveAll", preTail + "bulkTestMapValues.testCollectionRetainAll" }; - } else { - return new String[] { recursiveTest }; } + return new String[] { recursiveTest }; } http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMapTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMapTest.java b/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMapTest.java index ed56dda..fbd34bf 100644 --- a/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMapTest.java +++ b/src/test/java/org/apache/commons/collections4/bidimap/DualTreeBidiMapTest.java @@ -65,9 +65,8 @@ public class DualTreeBidiMapTest<K extends Comparable<K>, V extends Comparable<V preTail + "bulkTestMapValues.testCollectionRemoveAll", preTail + "bulkTestMapValues.testCollectionRetainAll" }; - } else { - return new String[] { recursiveTest }; } + return new String[] { recursiveTest }; } // public void testCreate() throws Exception { http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/test/java/org/apache/commons/collections4/iterators/NodeListIteratorTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/collections4/iterators/NodeListIteratorTest.java b/src/test/java/org/apache/commons/collections4/iterators/NodeListIteratorTest.java index 8a5da1f..dfd62ed 100644 --- a/src/test/java/org/apache/commons/collections4/iterators/NodeListIteratorTest.java +++ b/src/test/java/org/apache/commons/collections4/iterators/NodeListIteratorTest.java @@ -80,13 +80,12 @@ public class NodeListIteratorTest extends AbstractIteratorTest<Node> { if (createIteratorWithStandardConstr) { return new NodeListIterator(emptyNodeList); - } else { - final Node parentNode = createMock(Node.class); - expect(parentNode.getChildNodes()).andStubReturn(emptyNodeList); - replay(parentNode); - - return new NodeListIterator(parentNode); } + final Node parentNode = createMock(Node.class); + expect(parentNode.getChildNodes()).andStubReturn(emptyNodeList); + replay(parentNode); + + return new NodeListIterator(parentNode); } @Override http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java b/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java index bb6741f..ccd53cd 100644 --- a/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java +++ b/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java @@ -1337,9 +1337,8 @@ public abstract class AbstractListTest<E> extends AbstractCollectionTest<E> { if (t instanceof ConcurrentModificationException) { // expected return; - } else { - fail(m.getName() + " raised unexpected " + e); } + fail(m.getName() + " raised unexpected " + e); } } http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/test/java/org/apache/commons/collections4/map/LRUMapTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/collections4/map/LRUMapTest.java b/src/test/java/org/apache/commons/collections4/map/LRUMapTest.java index 49b365a..46368c8 100644 --- a/src/test/java/org/apache/commons/collections4/map/LRUMapTest.java +++ b/src/test/java/org/apache/commons/collections4/map/LRUMapTest.java @@ -447,9 +447,8 @@ public class LRUMapTest<K, V> extends AbstractOrderedMapTest<K, V> { protected boolean removeLRU(final LinkEntry<K, V> entry) { if ("a".equals(entry.getValue())) { return false; - } else { - return true; } + return true; } } http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/test/java/org/apache/commons/collections4/map/ReferenceIdentityMapTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/collections4/map/ReferenceIdentityMapTest.java b/src/test/java/org/apache/commons/collections4/map/ReferenceIdentityMapTest.java index 04210b7..1ccc8ec 100644 --- a/src/test/java/org/apache/commons/collections4/map/ReferenceIdentityMapTest.java +++ b/src/test/java/org/apache/commons/collections4/map/ReferenceIdentityMapTest.java @@ -310,12 +310,11 @@ public class ReferenceIdentityMapTest<K, V> extends AbstractIterableMapTest<K, V valueReference.get() == null) { break; - } else { - // create garbage: - @SuppressWarnings("unused") - final byte[] b = new byte[bytz]; - bytz = bytz * 2; } + // create garbage: + @SuppressWarnings("unused") + final byte[] b = new byte[bytz]; + bytz = bytz * 2; } } http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/test/java/org/apache/commons/collections4/map/ReferenceMapTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/collections4/map/ReferenceMapTest.java b/src/test/java/org/apache/commons/collections4/map/ReferenceMapTest.java index 901cb0f..4ce341f 100644 --- a/src/test/java/org/apache/commons/collections4/map/ReferenceMapTest.java +++ b/src/test/java/org/apache/commons/collections4/map/ReferenceMapTest.java @@ -241,12 +241,11 @@ public class ReferenceMapTest<K, V> extends AbstractIterableMapTest<K, V> { if (keyReference.get() == null && valueReference.get() == null) { break; - } else { - // create garbage: - @SuppressWarnings("unused") - final byte[] b = new byte[bytz]; - bytz = bytz * 2; } + // create garbage: + @SuppressWarnings("unused") + final byte[] b = new byte[bytz]; + bytz = bytz * 2; } } http://git-wip-us.apache.org/repos/asf/commons-collections/blob/8b66a577/src/test/java/org/apache/commons/collections4/map/TransformedSortedMapTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/collections4/map/TransformedSortedMapTest.java b/src/test/java/org/apache/commons/collections4/map/TransformedSortedMapTest.java index bf7ba53..7be48c9 100644 --- a/src/test/java/org/apache/commons/collections4/map/TransformedSortedMapTest.java +++ b/src/test/java/org/apache/commons/collections4/map/TransformedSortedMapTest.java @@ -62,9 +62,8 @@ public class TransformedSortedMapTest<K, V> extends AbstractSortedMapTest<K, V> preTailMap + "bulkTestMapValues.testCollectionRemoveAll", preTailMap + "bulkTestMapValues.testCollectionRetainAll" }; - } else { - return null; } + return null; } //-----------------------------------------------------------------------
