commit cc21b6f2e6c56e2129b710e66f4f47217666335b
author Mikhail Naganov <mnaganov@google.com> Tue Oct 17 19:39:27 2023 -0700
committer Mikhail Naganov <mnaganov@google.com> Tue Oct 17 19:53:55 2023 -0700
tree 9c35e56cc869e6e75f3fb6ced9a66f8835b68a74
parent 394e2527b5618cfb12bb55e29640c84651467eb1

audio: Implement proper parsing of masks in XsdcConversion

Since the parsed XML uses compliant separator for lists
in attribute values, there is no need to use legacy
'maskFromString' function.

Bug: 305281224
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Ib257fd010b923ac651931e5976f9b1363d693f1b

audio/aidl/default/XsdcConversion.cpp

diff --git a/audio/aidl/default/XsdcConversion.cpp b/audio/aidl/default/XsdcConversion.cpp
index 2feb0c3..9e30347 100644
--- a/audio/aidl/default/XsdcConversion.cpp
+++ b/audio/aidl/default/XsdcConversion.cpp
@@ -141,11 +141,12 @@
 }
 
 ConversionResult<int> convertGainModeToAidl(const std::vector<ap_xsd::AudioGainMode>& gainModeVec) {
-    static const char gainModeSeparator = ' ';
     int gainModeMask = 0;
     for (const ap_xsd::AudioGainMode& gainMode : gainModeVec) {
-        gainModeMask |= static_cast<int>(::android::GainModeConverter::maskFromString(
-                ap_xsd::toString(gainMode), &gainModeSeparator));
+        audio_gain_mode_t legacyGainMode;
+        if (::android::GainModeConverter::fromString(ap_xsd::toString(gainMode), legacyGainMode)) {
+            gainModeMask |= static_cast<int>(legacyGainMode);
+        }
     }
     return gainModeMask;
 }
@@ -204,22 +205,25 @@
 ConversionResult<AudioIoFlags> convertIoFlagsToAidl(
         const std::vector<ap_xsd::AudioInOutFlag>& flags, const ap_xsd::Role role,
         bool flagsForMixPort) {
-    static const char flagSeparator = ' ';
     int flagMask = 0;
     if ((role == ap_xsd::Role::sink && flagsForMixPort) ||
         (role == ap_xsd::Role::source && !flagsForMixPort)) {
         for (const ap_xsd::AudioInOutFlag& flag : flags) {
-            flagMask |= static_cast<int>(::android::InputFlagConverter::maskFromString(
-                    ap_xsd::toString(flag), &flagSeparator));
+            audio_input_flags_t legacyFlag;
+            if (::android::InputFlagConverter::fromString(ap_xsd::toString(flag), legacyFlag)) {
+                flagMask |= static_cast<int>(legacyFlag);
+            }
         }
         return AudioIoFlags::make<AudioIoFlags::Tag::input>(flagMask);
     } else {
         for (const ap_xsd::AudioInOutFlag& flag : flags) {
-            flagMask |= static_cast<int>(::android::OutputFlagConverter::maskFromString(
-                    ap_xsd::toString(flag), &flagSeparator));
+            audio_output_flags_t legacyFlag;
+            if (::android::OutputFlagConverter::fromString(ap_xsd::toString(flag), legacyFlag)) {
+                flagMask |= static_cast<int>(legacyFlag);
+            }
         }
+        return AudioIoFlags::make<AudioIoFlags::Tag::output>(flagMask);
     }
-    return AudioIoFlags::make<AudioIoFlags::Tag::output>(flagMask);
 }
 
 ConversionResult<AudioPort> convertDevicePortToAidl(