commit | 5f709426836cc477dd491fa46663d38fd50e4055 | [log] [tgz] |
---|---|---|
author | Sho SHIMIZU <sshimizu@us.fujitsu.com> | Wed Sep 07 09:54:46 2016 -0700 |
committer | Gerrit Code Review <gerrit@onlab.us> | Fri Sep 09 17:35:30 2016 +0000 |
tree | 3b8a8ddec5f1e466f325f678ec67a74a8c324a93 | |
parent | 64ff13b415db441e8d7239922ef0dfbf612947f3 [diff] [blame] |
Add comments describing immutability/mutability Change-Id: I2fc947b3785c6edf1ad20eb432dafc1642781ca7
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 82c1c43..50d74fc 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
@@ -598,12 +598,13 @@ } private class FlowOperationsProcessor implements Runnable { - - private final List<Set<FlowRuleOperation>> stages; + // Immutable private final FlowRuleOperations fops; - private boolean hasFailed = false; + // Mutable + private final List<Set<FlowRuleOperation>> stages; private final Set<DeviceId> pendingDevices = new HashSet<>(); + private boolean hasFailed = false; FlowOperationsProcessor(FlowRuleOperations ops) { this.stages = Lists.newArrayList(ops.stages());