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

Commit c5a55dea authored by Eric Laurent's avatar Eric Laurent
Browse files

audio policy: remove unnecessary error log 

Remove unnecessary error log when the optional engine
config file is not found.

Bug: 181269159
Test: make, flash and reboot.
Change-Id: If30792c59fa0e392861317c40aa00464ff6fae00
parent 341f4d5c

services/audiopolicy/engine/common/src/EngineBase.cpp

+1 −1
Original line number Diff line number Diff line
@@ -151,7 +151,7 @@ engineConfig::ParsingResult EngineBase::loadAudioPolicyEngineConfig() 


    auto result = engineConfig::parse();

    if (result.parsedConfig == nullptr) {

        ALOGW("%s: No configuration found, using default matching phone experience.", __FUNCTION__);

        ALOGD("%s: No configuration found, using default matching phone experience.", __FUNCTION__);

        engineConfig::Config config = gDefaultEngineConfig;

        android::status_t ret = engineConfig::parseLegacyVolumes(config.volumeGroups);

        result = {std::make_unique<engineConfig::Config>(config),

services/audiopolicy/engine/config/src/EngineConfig.cpp

+5 −1
Original line number Diff line number Diff line
@@ -643,7 +643,11 @@ ParsingResult parse(const char* path) { 
    xmlDocPtr doc;

    doc = xmlParseFile(path);

    if (doc == NULL) {

        ALOGE("%s: Could not parse document %s", __FUNCTION__, path);

        // It is OK not to find an engine config file at the default location

        // as the caller will default to hardcoded default config

        if (strncmp(path, DEFAULT_PATH, strlen(DEFAULT_PATH))) {

            ALOGW("%s: Could not parse document %s", __FUNCTION__, path);

        }

        return {nullptr, 0};

    }

    xmlNodePtr cur = xmlDocGetRootElement(doc);