diff --git a/testng-core/src/main/java/org/testng/internal/Graph.java b/testng-core/src/main/java/org/testng/internal/Graph.java index b8e85dfff7..c88535e960 100644 --- a/testng-core/src/main/java/org/testng/internal/Graph.java +++ b/testng-core/src/main/java/org/testng/internal/Graph.java @@ -21,7 +21,6 @@ * @author Cedric Beust, Aug 19, 2004 */ public class Graph { - private static final boolean m_verbose = true; private final Map> m_nodes = Maps.newLinkedHashMap(); private List m_strictlySortedNodes = null; private final Comparator> comparator; @@ -59,7 +58,6 @@ public void addPredecessor(T tm, T predecessor) { throw new TestNGException("Non-existing node: " + tm); } else { node.addPredecessor(predecessor); - addNeighbor(tm, predecessor); // Remove these two nodes from the independent list initializeIndependentNodes(); m_independentNodes.remove(predecessor); @@ -68,10 +66,6 @@ public void addPredecessor(T tm, T predecessor) { } } - private void addNeighbor(T tm, T predecessor) { - findNode(tm).addNeighbor(findNode(predecessor)); - } - private Collection> getNodes() { return m_nodes.values(); } @@ -133,9 +127,7 @@ public void topologicalSort() { } log("=============== DONE SORTING"); - if (m_verbose) { - dumpSortedNodes(); - } + dumpSortedNodes(); } private void initializeIndependentNodes() { @@ -173,15 +165,11 @@ private void removeFromNodes(List> nodes, Node node) { } private static void log(String s) { - if (m_verbose) { - Logger.getLogger(Graph.class).debug("[Graph] " + s); - } + log(() -> s); } private static void log(Supplier s) { - if (m_verbose) { - Logger.getLogger(Graph.class).debug("[Graph] " + s.get()); - } + Logger.getLogger(Graph.class).debug("[Graph] " + s.get()); } private Node findNodeWithNoPredecessors(List> nodes) { @@ -253,12 +241,6 @@ public Node(T tm) { m_object = tm; } - private final Set> m_neighbors = new HashSet<>(); - - public void addNeighbor(Node neighbor) { - m_neighbors.add(neighbor); - } - @Override public Node clone() { Node result = new Node<>(m_object);