From: Robert Varga <robert.varga@pantheon.tech>
Date: Tue, 9 Jul 2019 20:27:30 +0000 (+0200)
Subject: Hide AbstractIdentifier.getValue()
X-Git-Tag: v4.0.0~53
X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=commitdiff_plain;h=29451e0ac6c9d57a23daa002fb08ff3131c8a5be;p=yangtools.git

Hide AbstractIdentifier.getValue()

AbstractIdentifier is a utility wrapper class, hence it should not
have a public surface users should be interested it. Anybody relying
on .getValue() who is not a subclass should expose a domain-specific
getter method.

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

diff --git a/common/util/src/main/java/org/opendaylight/yangtools/util/AbstractIdentifier.java b/common/util/src/main/java/org/opendaylight/yangtools/util/AbstractIdentifier.java
index 408afcb976..93c2da99b5 100644
--- a/common/util/src/main/java/org/opendaylight/yangtools/util/AbstractIdentifier.java
+++ b/common/util/src/main/java/org/opendaylight/yangtools/util/AbstractIdentifier.java
@@ -30,7 +30,7 @@ public abstract class AbstractIdentifier<T> implements Identifier {
         this.value = requireNonNull(value);
     }
 
-    public final @NonNull T getValue() {
+    protected final @NonNull T getValue() {
         return value;
     }