Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 34b554b3 authored by Vova Sharaienko's avatar Vova Sharaienko Committed by Automerger Merge Worker
Browse files

Merge "Removed extra string instance creation" am: 3cb50d84 

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2337144



Change-Id: I863b8d8805de61094746e0491334e8ca75efeae9
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 8d2b4331 3cb50d84

services/stats/StatsHal.cpp

+3 −3
Original line number Diff line number Diff line
@@ -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();