Committed by
Gerrit Code Review
Move field assignment to declaration and make the field final
The field assignment doesn't change over the life time Change-Id: I022dbe7ce8a1cbae8837fd97bff8610a4cada610
Showing
1 changed file
with
1 additions
and
2 deletions
| ... | @@ -578,13 +578,12 @@ public class FlowRuleManager | ... | @@ -578,13 +578,12 @@ public class FlowRuleManager |
| 578 | private final FlowRuleOperations fops; | 578 | private final FlowRuleOperations fops; |
| 579 | private final AtomicBoolean hasFailed = new AtomicBoolean(false); | 579 | private final AtomicBoolean hasFailed = new AtomicBoolean(false); |
| 580 | 580 | ||
| 581 | - private Set<DeviceId> pendingDevices; | 581 | + private final Set<DeviceId> pendingDevices = Sets.newConcurrentHashSet(); |
| 582 | 582 | ||
| 583 | public FlowOperationsProcessor(FlowRuleOperations ops) { | 583 | public FlowOperationsProcessor(FlowRuleOperations ops) { |
| 584 | this.stages = Lists.newArrayList(ops.stages()); | 584 | this.stages = Lists.newArrayList(ops.stages()); |
| 585 | this.context = ops.callback(); | 585 | this.context = ops.callback(); |
| 586 | this.fops = ops; | 586 | this.fops = ops; |
| 587 | - pendingDevices = Sets.newConcurrentHashSet(); | ||
| 588 | } | 587 | } |
| 589 | 588 | ||
| 590 | @Override | 589 | @Override | ... | ... |
-
Please register or login to post a comment