Add comments describing immutability/mutability
Change-Id: I2fc947b3785c6edf1ad20eb432dafc1642781ca7
Showing
1 changed file
with
4 additions
and
3 deletions
... | @@ -577,12 +577,13 @@ public class FlowRuleManager | ... | @@ -577,12 +577,13 @@ public class FlowRuleManager |
577 | } | 577 | } |
578 | 578 | ||
579 | private class FlowOperationsProcessor implements Runnable { | 579 | private class FlowOperationsProcessor implements Runnable { |
580 | - | 580 | + // Immutable |
581 | - private final List<Set<FlowRuleOperation>> stages; | ||
582 | private final FlowRuleOperations fops; | 581 | private final FlowRuleOperations fops; |
583 | - private boolean hasFailed = false; | ||
584 | 582 | ||
583 | + // Mutable | ||
584 | + private final List<Set<FlowRuleOperation>> stages; | ||
585 | private final Set<DeviceId> pendingDevices = new HashSet<>(); | 585 | private final Set<DeviceId> pendingDevices = new HashSet<>(); |
586 | + private boolean hasFailed = false; | ||
586 | 587 | ||
587 | FlowOperationsProcessor(FlowRuleOperations ops) { | 588 | FlowOperationsProcessor(FlowRuleOperations ops) { |
588 | this.stages = Lists.newArrayList(ops.stages()); | 589 | this.stages = Lists.newArrayList(ops.stages()); | ... | ... |
-
Please register or login to post a comment