From ad3b6f4df6aad9bbb09b98cc00ca0b7b3534d0a0 Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Fri, 4 Mar 2016 15:16:52 +0100 Subject: [PATCH] Do not leak internal state The state transition enum should not be visible from outside, hide it. Change-Id: I2b3fa8af7829509b5a968eb952a53c829e189aaa Signed-off-by: Robert Varga --- .../openflowplugin/impl/device/TransactionChainManager.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/TransactionChainManager.java b/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/TransactionChainManager.java index 36685c5182..657aebaf07 100644 --- a/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/TransactionChainManager.java +++ b/openflowplugin-impl/src/main/java/org/opendaylight/openflowplugin/impl/device/TransactionChainManager.java @@ -70,10 +70,6 @@ class TransactionChainManager implements TransactionChainListener, AutoCloseable private BindingTransactionChain txChainFactory; private boolean submitIsEnabled; - public TransactionChainManagerStatus getTransactionChainManagerStatus() { - return transactionChainManagerStatus; - } - @GuardedBy("txLock") private TransactionChainManagerStatus transactionChainManagerStatus; private final KeyedInstanceIdentifier nodeII; @@ -337,7 +333,7 @@ class TransactionChainManager implements TransactionChainListener, AutoCloseable Preconditions.checkState(txChainFactory == null); } - public enum TransactionChainManagerStatus { + private enum TransactionChainManagerStatus { /** txChainManager is sleeping - is not active (SLAVE or default init value) */ WORKING, /** txChainManager is working - is active (MASTER) */ -- 2.36.6