opendev/gerrit
Code Issues Proposed changes

Merge "Fix ModifyCollectionInEnhancedForLoop warning flagged by error prone"

Browse Source
This commit is contained in:
David Pursehouse
2020-05-21 14:30:18 +00:00
committed by Gerrit Code Review
parent 32370c8b13 7619a2508e
commit f480286bca
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
java/com/google/gerrit/metrics/dropwizard/BucketedCallback.java
View File

@@ -21,8 +21,8 @@ import com.google.common.collect.ImmutableList;
import com.google.common.collect.Maps; import com.google.common.collect.Maps;
import com.google.gerrit.metrics.Description; import com.google.gerrit.metrics.Description;
import com.google.gerrit.metrics.Field; import com.google.gerrit.metrics.Field;
import java.util.Iterator;
import java.util.Map; import java.util.Map;
import java.util.Set;
import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ConcurrentHashMap;
/** Abstract callback metric broken down into buckets. */ /** Abstract callback metric broken down into buckets. */
@@ -67,10 +67,11 @@ abstract class BucketedCallback<V> implements BucketedMetric {
} }
void doPrune() { void doPrune() {
Set<Map.Entry<Object, BucketedCallback<V>.ValueGauge>> entries = cells.entrySet(); Iterator<Map.Entry<Object, ValueGauge>> it = cells.entrySet().iterator();
for (Map.Entry<Object, ValueGauge> e : entries) { while (it.hasNext()) {
Map.Entry<Object, ValueGauge> e = it.next();
if (!e.getValue().set) { if (!e.getValue().set) {
entries.remove(e); it.remove();
registry.remove(submetric(e.getKey())); registry.remove(submetric(e.getKey()));
} }
} }