diff options
author | Jeffrey Huang <jeffreyhuang@google.com> | 2024-04-29 07:12:16 -0700 |
---|---|---|
committer | Jeffrey Huang <jeffreyhuang@google.com> | 2024-04-29 07:13:16 -0700 |
commit | b64764c62c89ef828a2b7413451fcacf2be3098c (patch) | |
tree | 7fe6bb07b46868ce35891a819e6bd318b778af4c /statsd/src/metrics/ValueMetricProducer.cpp | |
parent | 10e016418d6a0779995224c03b700ac0faf6b8b9 (diff) | |
download | StatsD-b64764c62c89ef828a2b7413451fcacf2be3098c.tar.gz |
Move use of bucket before std::move
Bug: 328659716
Test: atest statsd_test
Change-Id: I6bf2b9186719c32e737371f73c0e4fd50ed2ca61
Diffstat (limited to 'statsd/src/metrics/ValueMetricProducer.cpp')
-rw-r--r-- | statsd/src/metrics/ValueMetricProducer.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/statsd/src/metrics/ValueMetricProducer.cpp b/statsd/src/metrics/ValueMetricProducer.cpp index b1f0a331..ff65792e 100644 --- a/statsd/src/metrics/ValueMetricProducer.cpp +++ b/statsd/src/metrics/ValueMetricProducer.cpp @@ -851,9 +851,9 @@ void ValueMetricProducer<AggregatedValue, DimExtras>::closeCurrentBucket( auto& bucketList = mPastBuckets[metricDimensionKey]; const bool isFirstBucket = bucketList.empty(); - bucketList.push_back(std::move(bucket)); mTotalDataSize += computeValueBucketSizeLocked( eventTimeNs >= fullBucketEndTimeNs, metricDimensionKey, isFirstBucket, bucket); + bucketList.push_back(std::move(bucket)); } if (!bucketHasData) { skipCurrentBucket(eventTimeNs, BucketDropReason::NO_DATA); |