commit 8ba8b353ca1bc838428fa0637be11d131ca637c7
author Vova Sharaienko <sharaienko@google.com> Sat Feb 11 02:06:37 2023 +0000
committer Vova Sharaienko <sharaienko@google.com> Sat Feb 11 03:27:08 2023 +0000
tree c9166b4855ef13696ee5d36a60123d065e85eae5
parent d35f56b199866e17f5dc7f3341a9e3b0f0ebd1f7

[TeX] Improved readability for Histogram metric APIs

Bug: 268161449
Test: atest expresslog_test
Change-Id: Idd83ca6ec47288ca6ef7ecce333ceb3c88dd761a

libstats/expresslog/Histogram.cpp

diff --git a/libstats/expresslog/Histogram.cpp b/libstats/expresslog/Histogram.cpp
index c90282d..cb29a00 100644
--- a/libstats/expresslog/Histogram.cpp
+++ b/libstats/expresslog/Histogram.cpp
@@ -26,8 +26,8 @@
 namespace android {
 namespace expresslog {
 
-Histogram::UniformOptions* Histogram::UniformOptions::create(int binCount, float minValue,
-                                                             float exclusiveMaxValue) {
+std::shared_ptr<Histogram::UniformOptions> Histogram::UniformOptions::create(
+        int binCount, float minValue, float exclusiveMaxValue) {
     if (binCount < 1) {
         ALOGE("Bin count should be positive number");
         return nullptr;
@@ -38,7 +38,8 @@
         return nullptr;
     }
 
-    return new UniformOptions(binCount, minValue, exclusiveMaxValue);
+    return std::shared_ptr<UniformOptions>(
+            new UniformOptions(binCount, minValue, exclusiveMaxValue));
 }
 
 Histogram::UniformOptions::UniformOptions(int binCount, float minValue, float exclusiveMaxValue)