summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Vova Sharaienko <sharaienko@google.com> 2022-12-13 18:15:52 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2022-12-13 18:15:52 +0000
commit0a5afc0d69c82ee87b9a979c2a56dc1ada7e3947 (patch)
treee53a8fd6f813fce8130fb54d887d5f8dd34dd436
parentcd98f5dd102a5f5cb2ccbf1873f95c9d84df0787 (diff)
parent34b554b361d06f42f999e378b85e35ce88b539c0 (diff)
Merge "Removed extra string instance creation" am: 3cb50d840f am: 34b554b361
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2337144 Change-Id: Ib48ac37a18e3310e6de90f197fbaa32af2ade59e Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-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();