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

Commit 13c9c12e authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "audiopolicy: capengine: fix missing comma when chaining compound rules"... 

Merge "audiopolicy: capengine: fix missing comma when chaining compound rules" into main am: a660a45f am: 8d3ae49b

Original change: https://android-review.googlesource.com/c/platform/frameworks/av/+/3450491



Change-Id: I973a2c48fc04ad3f0201daaf3a52f9e5e42f1f07
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents de3f48e1 8d3ae49b

services/audiopolicy/engineconfigurable/config/src/CapEngineConfig.cpp

+6 −5
Original line number Diff line number Diff line
@@ -316,13 +316,14 @@ ConversionResult<std::string> aidl2legacy_AudioHalCapRule_CapRule( 
    }

    rule += "{";

    if (!aidlRule.nestedRules.empty()) {

        for (const auto& nestedRule: aidlRule.nestedRules) {

            rule += VALUE_OR_FATAL(aidl2legacy_AudioHalCapRule_CapRule(nestedRule));

        }

        if (!aidlRule.criterionRules.empty()) {

        for (auto ruleIter = aidlRule.nestedRules.begin(); ruleIter != aidlRule.nestedRules.end();

                ++ruleIter) {

            rule += VALUE_OR_FATAL(aidl2legacy_AudioHalCapRule_CapRule(*ruleIter));

            if (ruleIter != (aidlRule.nestedRules.end()  - 1) || !aidlRule.criterionRules.empty()) {

                rule += ",";

            }

        }

    }

    bool isFirstCriterionRule = true;

    for (const auto& criterionRule: aidlRule.criterionRules) {

        if (!isFirstCriterionRule) {