commit 19bf4210e6a7b7721a3e75ae4218ed2e5109f702
author tom <tom@onlab.us> Fri Aug 29 13:08:29 2014 -0700
committer tom <tom@onlab.us> Fri Aug 29 13:08:29 2014 -0700
tree 88d105a0db13155d6297fd863384cf26d8b33b75
parent 984d782e972fd1378d8eb239d8657dd5573b9a3b

Fixed some sonar-reported issues.

utils/misc/src/main/java/org/onlab/graph/BreadthFirstSearch.java

diff --git a/utils/misc/src/main/java/org/onlab/graph/BreadthFirstSearch.java b/utils/misc/src/main/java/org/onlab/graph/BreadthFirstSearch.java
index 2346ff6..c02afea 100644
--- a/utils/misc/src/main/java/org/onlab/graph/BreadthFirstSearch.java
+++ b/utils/misc/src/main/java/org/onlab/graph/BreadthFirstSearch.java
@@ -21,8 +21,8 @@
         result.costs.put(src, 0.0);
         frontier.add(src);
 
-        search:
-        while (!frontier.isEmpty()) {
+        boolean reachedEnd = false;
+        while (!reachedEnd && !frontier.isEmpty()) {
             // Prepare the next frontier.
             Set<V> next = new HashSet<>();
 
@@ -40,10 +40,15 @@
                                             true);
                         // If we have reached our intended destination, bail.
                         if (nextVertex.equals(dst)) {
-                            break search;
+                            reachedEnd = true;
+                            break;
                         }
                         next.add(nextVertex);
                     }
+
+                    if (reachedEnd) {
+                        break;
+                    }
                 }
             }