Remove StackedReversePathArguments.haveSize 02/89102/2
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 18 Mar 2020 13:38:43 +0000 (14:38 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Fri, 17 Apr 2020 16:21:47 +0000 (18:21 +0200)
We do not need to have a separate field here, as the size cannot
legally be here 0 -- as stacking guarantees we have at least one
parent.

Change-Id: Id319d4ef05efc1c3691b7b97b6b188cc09701547
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 5198ae72c04d103b4e6e98442df2587b52c202a8)

yang/yang-data-api/src/main/java/org/opendaylight/yangtools/yang/data/api/StackedReversePathArguments.java

index 46ba12e0d0de15af9f93aca41c52111be70baea9..9250515342c986f80992f3115891e0fe30eac273 100644 (file)
@@ -19,8 +19,8 @@ import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.PathArgum
 
 final class StackedReversePathArguments extends PathArgumentList {
     private final StackedYangInstanceIdentifier identifier;
-    private int size;
-    private volatile boolean haveSize;
+
+    private volatile int size;
 
     StackedReversePathArguments(final StackedYangInstanceIdentifier identifier) {
         this.identifier = requireNonNull(identifier);
@@ -42,10 +42,9 @@ final class StackedReversePathArguments extends PathArgumentList {
     @Override
     public int size() {
         int ret = size;
-        if (!haveSize) {
+        if (ret == 0) {
             ret = calculateSize(identifier.getParent());
             size = ret;
-            haveSize = true;
         }
 
         return ret;