You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by mb...@apache.org on 2017/10/07 12:13:05 UTC
svn commit: r1811414 -
/ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/test/GenericTestCaseBase.java
Author: mbrohl
Date: Sat Oct 7 12:13:05 2017
New Revision: 1811414
URL: http://svn.apache.org/viewvc?rev=1811414&view=rev
Log:
Improved: Fixing defects reported by FindBugs, package
org.apache.ofbiz.base.test.
(OFBIZ-9574)
Thanks Dennis Balkir for reporting and providing the patch.
Modified:
ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/test/GenericTestCaseBase.java
Modified: ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/test/GenericTestCaseBase.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/test/GenericTestCaseBase.java?rev=1811414&r1=1811413&r2=1811414&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/test/GenericTestCaseBase.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/test/GenericTestCaseBase.java Sat Oct 7 12:13:05 2017
@@ -44,7 +44,7 @@ public abstract class GenericTestCaseBas
}
public static void useAllMemory() throws Exception {
- List<long[]> dummy = new LinkedList<long[]>();
+ List<long[]> dummy = new LinkedList<>();
try {
do {
dummy.add(new long[1048576]);
@@ -154,7 +154,7 @@ public abstract class GenericTestCaseBas
if (got.equals(wanted)) return;
msg = msg == null ? "" : msg + ' ';
assertNotNull(msg + "expected a value", got);
- List<T> list = new ArrayList<T>(wanted);
+ List<T> list = new ArrayList<>(wanted);
Iterator<?> rightIt = ((Collection<?>) got).iterator();
OUTER:
while (rightIt.hasNext()) {
@@ -188,7 +188,7 @@ OUTER:
if (got.equals(wanted)) return;
msg = msg == null ? "" : msg + ' ';
assertNotNull(msg + "expected a value", got);
- Set<T> wantedSet = new HashSet<T>(wanted);
+ Set<T> wantedSet = new HashSet<>(wanted);
Iterator<?> rightIt = ((Set<?>) got).iterator();
while (rightIt.hasNext()) {
Object right = rightIt.next();
@@ -299,8 +299,8 @@ OUTER:
if (!(got instanceof Map<?, ?>)) fail(msg + "expected a map");
Map<?, ?> gotMap = (Map<?, ?>) got;
if (!got.equals(wanted)) {
- Set<T> leftKeys = new LinkedHashSet<T>(wanted.keySet());
- Set<Object> rightKeys = new HashSet<Object>(gotMap.keySet());
+ Set<T> leftKeys = new LinkedHashSet<>(wanted.keySet());
+ Set<Object> rightKeys = new HashSet<>(gotMap.keySet());
for (T key: leftKeys) {
assertTrue(msg + "got key(" + key + ")", rightKeys.remove(key));
assertEquals(msg + "key(" + key + ") value", wanted.get(key), gotMap.get(key));
@@ -345,23 +345,23 @@ OUTER:
}
public static <T> List<T> list(T value) {
- List<T> list = new ArrayList<T>(1);
+ List<T> list = new ArrayList<>(1);
list.add(value);
return list;
}
public static <T> List<T> list(T... list) {
- return new ArrayList<T>(Arrays.asList(list));
+ return new ArrayList<>(Arrays.asList(list));
}
public static <T> Set<T> set(T value) {
- Set<T> set = new HashSet<T>(1);
+ Set<T> set = new HashSet<>(1);
set.add(value);
return set;
}
public static <T> Set<T> set(T... list) {
- return new HashSet<T>(Arrays.asList(list));
+ return new HashSet<>(Arrays.asList(list));
}
public static <T> Set<T> set(Iterable<T> iterable) {
@@ -369,7 +369,7 @@ OUTER:
}
public static <T> Set<T> set(Iterator<T> it) {
- Set<T> set = new HashSet<T>();
+ Set<T> set = new HashSet<>();
while (it.hasNext()) {
T item = it.next();
set.add(item);
@@ -380,7 +380,7 @@ OUTER:
@SuppressWarnings("unchecked")
public static <K, V> Map<K, V> map(Object... list) {
assertEquals("list has even number of elements", 0, list.length % 2);
- Map<K, V> map = new LinkedHashMap<K, V>();
+ Map<K, V> map = new LinkedHashMap<>();
for (int i = 0; i < list.length; i += 2) {
map.put((K) list[i], (V) list[i + 1]);
}