diff options
author | 2022-12-09 00:07:23 +0000 | |
---|---|---|
committer | 2022-12-13 00:00:05 +0000 | |
commit | 34efca02e67491f7ba3c698f259715a8fb31a78e (patch) | |
tree | 9d33764d4acf6ba4c48b8b63c69f9e328598e54b | |
parent | 73dcda7eada36b429c4788f05d340ce7f8dfcc4f (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
-rw-r--r-- | services/stats/StatsHal.cpp | 6 |
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(); |