commit 5f709426836cc477dd491fa46663d38fd50e4055
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

Add comments describing immutability/mutability

Change-Id: I2fc947b3785c6edf1ad20eb432dafc1642781ca7

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 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());