summaryrefslogtreecommitdiff
path: root/services/stats/StatsHal.cpp
diff options
context:
space:
mode:
author Vova Sharaienko <sharaienko@google.com> 2022-12-09 00:07:23 +0000
committer Vova Sharaienko <sharaienko@google.com> 2022-12-13 00:00:05 +0000
commit34efca02e67491f7ba3c698f259715a8fb31a78e (patch)
tree9d33764d4acf6ba4c48b8b63c69f9e328598e54b /services/stats/StatsHal.cpp
parent73dcda7eada36b429c4788f05d340ce7f8dfcc4f (diff)
Removed extra string instance creation
According to the auto-generated code the reverseDomainName is a android::hardware::hidl_string which follow RAII approach Similar to std::string Bug: 212755214 Test: aidl_stats_client Change-Id: I2fe90721a56abcbf20586e17550304be608de21b
Diffstat (limited to 'services/stats/StatsHal.cpp')
-rw-r--r--services/stats/StatsHal.cpp6
1 files changed, 3 insertions, 3 deletions
diff --git a/services/stats/StatsHal.cpp b/services/stats/StatsHal.cpp
index ae0a9843f6..d27d989c81 100644
--- a/services/stats/StatsHal.cpp
+++ b/services/stats/StatsHal.cpp
@@ -112,13 +112,13 @@ hardware::Return<void> StatsHal::reportSpeechDspStat(
}
hardware::Return<void> StatsHal::reportVendorAtom(const VendorAtom& vendorAtom) {
- std::string reverseDomainName = (std::string) vendorAtom.reverseDomainName;
if (vendorAtom.atomId < 100000 || vendorAtom.atomId >= 200000) {
ALOGE("Atom ID %ld is not a valid vendor atom ID", (long) vendorAtom.atomId);
return hardware::Void();
}
- if (reverseDomainName.length() > 50) {
- ALOGE("Vendor atom reverse domain name %s is too long.", reverseDomainName.c_str());
+ if (vendorAtom.reverseDomainName.size() > 50) {
+ ALOGE("Vendor atom reverse domain name %s is too long.",
+ vendorAtom.reverseDomainName.c_str());
return hardware::Void();
}
AStatsEvent* event = AStatsEvent_obtain();