audio: avoid downmixing in a certain special-case

As indicated by the added test. In this case, fallback and downmix have
the same score, but fallback happens to give better results. So prefer
fallback over downmix.

(This is probably not a correct solution.)

(cherry picked from commit d3c7fd9d7c971086a3d6fde5f6f1bc4ef0b2e904)

1 files changed, 17 insertions, 0 deletions

diff --git a/test/chmap_sel.c b/test/chmap_sel.c
index c315d2f9a1..a1eae0b6b3 100644
--- a/test/chmap_sel.c
+++ b/test/chmap_sel.c
@@ -145,6 +145,22 @@ static void test_mp_chmap_sel_fallback_stereo_to_stereo(void **state) {
     assert_string_equal(mp_chmap_to_str(&c), "stereo");
 }
 
+static void test_mp_chmap_sel_fallback_no_downmix(void **state) {
+    struct mp_chmap a;
+    struct mp_chmap b;
+    struct mp_chmap c;
+    struct mp_chmap_sel s = {0};
+
+    mp_chmap_from_str(&a, bstr0("stereo"));
+    mp_chmap_from_str(&b, bstr0("7.1(rear)"));
+    mp_chmap_from_str(&c, bstr0("5.1(side)"));
+
+    mp_chmap_sel_add_map(&s, &a);
+    mp_chmap_sel_add_map(&s, &b);
+    assert_true(mp_chmap_sel_fallback(&s, &c));
+    assert_string_equal(mp_chmap_to_str(&c), "7.1(rear)");
+}
+
 int main(void) {
     const UnitTest tests[] = {
         unit_test(test_mp_chmap_sel_fallback_upmix),
@@ -157,6 +173,7 @@ int main(void) {
         unit_test(test_mp_chmap_sel_fallback_works_on_alsa_chmaps),
         unit_test(test_mp_chmap_sel_fallback_mono_to_stereo),
         unit_test(test_mp_chmap_sel_fallback_stereo_to_stereo),
+        unit_test(test_mp_chmap_sel_fallback_no_downmix),
     };
     return run_tests(tests);
 }