commit b1f4e0f2f93e8f49c4aea98baeb8706e5fbfe2cb
author Jon Hall <jhall@ciena.com> Wed Feb 22 13:36:16 2017 -0800
committer Yuta HIGUCHI <y-higuchi@onlab.us> Wed Feb 22 23:05:51 2017 +0000
tree 37030edccd399bca156e14c82869acf38f95e3a9
parent 5881be845db0ee735305d7e652a81a9ee7dbee67

Replace non-looping while with an if

Change-Id: I1e05b4b9c311f5ded063663f2113e0b8a1bb7e85

protocols/ovsdb/ctl/src/main/java/org/onosproject/ovsdb/controller/impl/OvsdbControllerImpl.java

diff --git a/protocols/ovsdb/ctl/src/main/java/org/onosproject/ovsdb/controller/impl/OvsdbControllerImpl.java b/protocols/ovsdb/ctl/src/main/java/org/onosproject/ovsdb/controller/impl/OvsdbControllerImpl.java
index 8258a74..1b9e7d4 100644
--- a/protocols/ovsdb/ctl/src/main/java/org/onosproject/ovsdb/controller/impl/OvsdbControllerImpl.java
+++ b/protocols/ovsdb/ctl/src/main/java/org/onosproject/ovsdb/controller/impl/OvsdbControllerImpl.java
@@ -342,7 +342,7 @@
         OvsdbSet ofPortSet = (OvsdbSet) intf.getOpenFlowPortColumn().data();
         @SuppressWarnings("unchecked")
         Set<Integer> ofPorts = ofPortSet.set();
-        while (ofPorts == null || ofPorts.isEmpty()) {
+        if (ofPorts == null || ofPorts.isEmpty()) {
             log.debug("The ofport is null in {}", intf.getName());
             return -1;
         }