From fd24f4003fbc7f444114e814eb537503c6673f12 Mon Sep 17 00:00:00 2001 From: Ryan Izard <ryan.izard@bigswitch.com> Date: Mon, 18 Jul 2016 17:14:56 -0400 Subject: [PATCH] Restrict access to TopologyInstance constructor and compute() functions. Other public functions should also be changed to private or protected. --- .../net/floodlightcontroller/topology/TopologyInstance.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/net/floodlightcontroller/topology/TopologyInstance.java b/src/main/java/net/floodlightcontroller/topology/TopologyInstance.java index 07a0404b5..e7bf66ae9 100644 --- a/src/main/java/net/floodlightcontroller/topology/TopologyInstance.java +++ b/src/main/java/net/floodlightcontroller/topology/TopologyInstance.java @@ -79,7 +79,7 @@ public class TopologyInstance { private Map<DatapathId, Set<NodePortTuple>> portsBroadcastPerArchipelago; /* broadcast ports in each archipelago ID */ private Map<PathId, List<Path>> pathcache; /* contains computed paths ordered best to worst */ - public TopologyInstance(Map<DatapathId, Set<OFPort>> portsWithLinks, + protected TopologyInstance(Map<DatapathId, Set<OFPort>> portsWithLinks, Set<NodePortTuple> portsBlocked, Map<NodePortTuple, Set<Link>> linksNonBcastNonTunnel, Set<NodePortTuple> portsWithMoreThanTwoLinks, @@ -135,7 +135,7 @@ public class TopologyInstance { this.archipelagoFromCluster = new HashMap<Cluster, Archipelago>(); } - public void compute() { + protected void compute() { /* * Step 1: Compute clusters ignoring ports with > 2 links and * blocked links. -- GitLab