Simplify MultivaluedHashMap.getFirst()

Rather than using an if/else block, use a trigraph, fixing a minor
eclipse warning about an unnecessary 'else'.

Change-Id: I8cc0d6d5bb4ac51fb2672e5aa047030919a47202
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/restconf/restconf-common/src/main/java/org/opendaylight/restconf/common/util/MultivaluedHashMap.java b/restconf/restconf-common/src/main/java/org/opendaylight/restconf/common/util/MultivaluedHashMap.java
index 158b846b28b5b3934534aa27991794d3832f6494..d272d06d9404dd5e6946f8cf140835ec3fbc89a2 100644 (file)
--- a/restconf/restconf-common/src/main/java/org/opendaylight/restconf/common/util/MultivaluedHashMap.java
+++ b/restconf/restconf-common/src/main/java/org/opendaylight/restconf/common/util/MultivaluedHashMap.java
@@ -78,11 +78,7 @@ public class MultivaluedHashMap<K, V> implements MultivaluedMap<K, V> {
     @Override
     public V getFirst(K key) {
         List<V> values = store.get(key);
-        if (values != null && values.size() > 0) {
-            return values.get(0);
-        } else {
-            return null;
-        }
+        return values == null || values.isEmpty() ? null : values.get(0);
     }
 
     @Override