commit 2ded2ddb85dbd3fc63aebb3f026fb14f116dd60e
author Yuta HIGUCHI <y-higuchi@onlab.us> Wed Oct 09 18:06:41 2013 -0700
committer Yuta HIGUCHI <y-higuchi@onlab.us> Tue Oct 15 10:15:33 2013 -0700
tree 051aaa8000d2da79655b24fcb25aebed22513efd
parent 5208f1a48ac2d00722a4a0cbcc0c2a3cd3a19bcd

Fix ugly hack to a slightly better hack

src/main/java/net/onrc/onos/ofcontroller/flowmanager/FlowManager.java

diff --git a/src/main/java/net/onrc/onos/ofcontroller/flowmanager/FlowManager.java b/src/main/java/net/onrc/onos/ofcontroller/flowmanager/FlowManager.java
index ad98227..8e4d81a 100644
--- a/src/main/java/net/onrc/onos/ofcontroller/flowmanager/FlowManager.java
+++ b/src/main/java/net/onrc/onos/ofcontroller/flowmanager/FlowManager.java
@@ -332,11 +332,7 @@
 		    if ((flowUserState != null)
 			&& flowUserState.equals("FE_USER_DELETE")) {
 			Iterable<IFlowEntry> flowEntries = flowPathObj.getFlowEntries();
-			boolean empty = true;	// TODO: an ugly hack
-			for (IFlowEntry flowEntryObj : flowEntries) {
-			    empty = false;
-			    break;
-			}
+			final boolean empty = !flowEntries.iterator().hasNext();
 			if (empty)
 			    deleteFlows.add(flowPathObj);
 		    }