Committed by
Gerrit Code Review
Add comments describing immutability/mutability
Change-Id: I2fc947b3785c6edf1ad20eb432dafc1642781ca7
Showing
1 changed file
with
4 additions
and
3 deletions
| ... | @@ -598,12 +598,13 @@ public class FlowRuleManager | ... | @@ -598,12 +598,13 @@ public class FlowRuleManager |
| 598 | } | 598 | } |
| 599 | 599 | ||
| 600 | private class FlowOperationsProcessor implements Runnable { | 600 | private class FlowOperationsProcessor implements Runnable { |
| 601 | - | 601 | + // Immutable |
| 602 | - private final List<Set<FlowRuleOperation>> stages; | ||
| 603 | private final FlowRuleOperations fops; | 602 | private final FlowRuleOperations fops; |
| 604 | - private boolean hasFailed = false; | ||
| 605 | 603 | ||
| 604 | + // Mutable | ||
| 605 | + private final List<Set<FlowRuleOperation>> stages; | ||
| 606 | private final Set<DeviceId> pendingDevices = new HashSet<>(); | 606 | private final Set<DeviceId> pendingDevices = new HashSet<>(); |
| 607 | + private boolean hasFailed = false; | ||
| 607 | 608 | ||
| 608 | FlowOperationsProcessor(FlowRuleOperations ops) { | 609 | FlowOperationsProcessor(FlowRuleOperations ops) { |
| 609 | this.stages = Lists.newArrayList(ops.stages()); | 610 | this.stages = Lists.newArrayList(ops.stages()); | ... | ... |
-
Please register or login to post a comment