From 94dee8f8cdeb8116ef8ccee975e49caf53e42dcd Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Mon, 19 Nov 2018 13:40:54 +0100 Subject: [PATCH] Make VpnUtil.getVpnListForVpnInterface() static This method is not touching any state, make it static and adjust the sole caller. Change-Id: I44ab35b0a7b3419663d516d817a549e72a62e0c8 Signed-off-by: Robert Varga --- .../opendaylight/netvirt/vpnmanager/VpnInterfaceManager.java | 4 ++-- .../java/org/opendaylight/netvirt/vpnmanager/VpnUtil.java | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnInterfaceManager.java b/vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnInterfaceManager.java index 62b2180aff..d681cf60ed 100755 --- a/vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnInterfaceManager.java +++ b/vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnInterfaceManager.java @@ -1571,10 +1571,10 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase oldVpnList = vpnUtil.getVpnListForVpnInterface(original); + List oldVpnList = VpnUtil.getVpnListForVpnInterface(original); List oldVpnListCopy = new ArrayList<>(); oldVpnListCopy.addAll(oldVpnList); - List newVpnList = vpnUtil.getVpnListForVpnInterface(update); + List newVpnList = VpnUtil.getVpnListForVpnInterface(update); List newVpnListCopy = new ArrayList<>(); newVpnListCopy.addAll(newVpnList); diff --git a/vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnUtil.java b/vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnUtil.java index 989a466a4a..f565525398 100644 --- a/vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnUtil.java +++ b/vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnUtil.java @@ -2300,7 +2300,7 @@ public final class VpnUtil { return VpnConstants.IP_MONITOR_JOB_PREFIX_KEY + "-" + vpnName + "-" + ip; } - public List getVpnListForVpnInterface(VpnInterface vpnInter) { + public static List getVpnListForVpnInterface(VpnInterface vpnInter) { return requireNonNullElse(vpnInter.getVpnInstanceNames(), Collections.emptyList()).stream() .map(VpnInstanceNames::getVpnName).collect(Collectors.toList()); } -- 2.36.6