From 8fe4bf97c127e5472e2408c42916da85fc252de8 Mon Sep 17 00:00:00 2001
From: Robert Varga <robert.varga@pantheon.tech>
Date: Tue, 6 Nov 2018 09:36:29 +0100
Subject: [PATCH] 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>
---
 .../restconf/common/util/MultivaluedHashMap.java            | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

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 158b846b28..d272d06d94 100644
--- 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
-- 
2.36.6