commit 2ad5f410e498203f15012aead69cdfbf0b4839b0
author Sho SHIMIZU <sshimizu@us.fujitsu.com> Wed Aug 31 15:23:42 2016 -0700
committer Thomas Vachuska <tom@onlab.us> Thu Sep 01 18:50:09 2016 +0000
tree 088debef3f6ce3a131547271b698f514c6b1c09e
parent 8efc89605bfa75e0b237171fcf9c3ca48dd8eb12

Refactor: Get updating pendingDevices out of the loop

Change-Id: I1b1c5ac141a68a6d6308238ae0934249c7a3af96

core/net/src/main/java/org/onosproject/net/flow/impl/FlowRuleManager.java

diff --git a/core/net/src/main/java/org/onosproject/net/flow/impl/FlowRuleManager.java b/core/net/src/main/java/org/onosproject/net/flow/impl/FlowRuleManager.java
index 04d8385..ae0b153 100644
--- a/core/net/src/main/java/org/onosproject/net/flow/impl/FlowRuleManager.java
+++ b/core/net/src/main/java/org/onosproject/net/flow/impl/FlowRuleManager.java
@@ -629,9 +629,9 @@
             for (FlowRuleOperation flowRuleOperation : ops) {
                 FlowRuleBatchEntry fbe =
                         new FlowRuleBatchEntry(mapOperationType(flowRuleOperation.type()), flowRuleOperation.rule());
-                pendingDevices.add(flowRuleOperation.rule().deviceId());
                 perDeviceBatches.put(flowRuleOperation.rule().deviceId(), fbe);
             }
+            pendingDevices.addAll(perDeviceBatches.keySet());
 
             for (DeviceId deviceId : perDeviceBatches.keySet()) {
                 long id = idGenerator.getNewId();