Update BindingReflections.getQName()

Use orElseThrow() instead of checkState() and explicit check/get.

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

diff --git a/binding/mdsal-binding-spec-util/src/main/java/org/opendaylight/mdsal/binding/spec/reflect/BindingReflections.java b/binding/mdsal-binding-spec-util/src/main/java/org/opendaylight/mdsal/binding/spec/reflect/BindingReflections.java
index 92d4d784ebd140cb24c7e9509210910b4a004a8a..332719b4e3424b188a214d70802527916c985e93 100644 (file)
--- a/binding/mdsal-binding-spec-util/src/main/java/org/opendaylight/mdsal/binding/spec/reflect/BindingReflections.java
+++ b/binding/mdsal-binding-spec-util/src/main/java/org/opendaylight/mdsal/binding/spec/reflect/BindingReflections.java
@@ -194,9 +194,8 @@ public final class BindingReflections {
     }
 
     public static @NonNull QName getQName(final Class<? extends BaseIdentity> bindingClass) {
-        final Optional<QName> qname = CLASS_TO_QNAME.getUnchecked(requireNonNull(bindingClass));
-        checkState(qname.isPresent(), "Failed to resolve QName of %s", bindingClass);
-        return qname.get();
+        return CLASS_TO_QNAME.getUnchecked(requireNonNull(bindingClass))
+            .orElseThrow(() -> new IllegalStateException("Failed to resolve QName of " + bindingClass));
     }
 
     /**