diff options
| author | 2022-12-12 14:09:25 +0000 | |
|---|---|---|
| committer | 2022-12-12 14:09:25 +0000 | |
| commit | ee36521096f5f13f40f20033e984b97a1e704c37 (patch) | |
| tree | bf10f0488c10a1e5ca9a759d3ee84212126b42a3 | |
| parent | 80ecd60b6b703cd743c32404078e38d35b6add39 (diff) | |
| parent | 5444011ece014f85e19a17e1cda8f3810fdb70dd (diff) | |
Merge "StatsAIDL: eliminated extra reverseDomainName string copy creation" am: 30ec4430c3 am: dfbf9224cc am: 5444011ece
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2336529
Change-Id: I0e703d564017a27915ddd60e7eefd6c23419542e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | services/stats/StatsAidl.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/services/stats/StatsAidl.cpp b/services/stats/StatsAidl.cpp index 9e13849485..410a5af421 100644 --- a/services/stats/StatsAidl.cpp +++ b/services/stats/StatsAidl.cpp @@ -30,14 +30,14 @@ namespace stats { StatsHal::StatsHal() {} ndk::ScopedAStatus 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 ndk::ScopedAStatus::fromServiceSpecificErrorWithMessage( -1, "Not a valid vendor atom ID"); } - if (reverseDomainName.length() > 50) { - ALOGE("Vendor atom reverse domain name %s is too long.", reverseDomainName.c_str()); + if (vendorAtom.reverseDomainName.length() > 50) { + ALOGE("Vendor atom reverse domain name %s is too long.", + vendorAtom.reverseDomainName.c_str()); return ndk::ScopedAStatus::fromServiceSpecificErrorWithMessage( -1, "Vendor atom reverse domain name is too long"); } |