From 26184491fe9cf70ea8920aff47c8ed2fdf78562c Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Wed, 18 Mar 2020 14:38:43 +0100 Subject: [PATCH] Remove StackedReversePathArguments.haveSize 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 (cherry picked from commit 5198ae72c04d103b4e6e98442df2587b52c202a8) --- .../yang/data/api/StackedReversePathArguments.java | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/yang/yang-data-api/src/main/java/org/opendaylight/yangtools/yang/data/api/StackedReversePathArguments.java b/yang/yang-data-api/src/main/java/org/opendaylight/yangtools/yang/data/api/StackedReversePathArguments.java index 46ba12e0d0..9250515342 100644 --- a/yang/yang-data-api/src/main/java/org/opendaylight/yangtools/yang/data/api/StackedReversePathArguments.java +++ b/yang/yang-data-api/src/main/java/org/opendaylight/yangtools/yang/data/api/StackedReversePathArguments.java @@ -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; -- 2.36.6