commit 9668bb7e705f4102e0558c522f1488a966a51826
author Fletcher Woodruff <fletcherw@chromium.org> Tue Nov 17 17:54:39 2020
committer Commit Bot <commit-bot@chromium.org> Fri Dec 11 23:16:39 2020
tree 2b6d2eacc8c755caa28c1c67eb3b6cf708bfa74c
parent 41683b370661dba1dee3aa0b30510c33231acad2

CRAS: update todo in cras_tests

Update a todo to reflect that our wav encode and decode crate does have
S24_LE support.

BUG=None
TEST=None

Change-Id: I7c54f1cb5869306d011a67720d4aeb43e0c0b83e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/adhd/+/2543299
Reviewed-by: Chih-Yang Hsia <paulhsia@chromium.org>
Tested-by: Chih-Yang Hsia <paulhsia@chromium.org>
Commit-Queue: Fletcher Woodruff <fletcherw@chromium.org>

cras/client/cras_tests/src/audio.rs

diff --git a/cras/client/cras_tests/src/audio.rs b/cras/client/cras_tests/src/audio.rs
index 261ad63..5ab2247 100644
--- a/cras/client/cras_tests/src/audio.rs
+++ b/cras/client/cras_tests/src/audio.rs
@@ -317,9 +317,11 @@
                         samples[sample_bytes * i + 3],
                     ]);
 
-                    // Upsample to 32 bit since CRAS doesn't support S24_3LE,
-                    // even though the wav encoder does.
-                    // TODO(fletcherw): add S24_LE support to hound.
+                    // Upsample to 32 bit since CRAS doesn't support S24_3LE.
+                    // Our wav encoder/decoder, hound, does have support for
+                    // S24_LE, but it hasn't released a new version since the
+                    // support was added. If getting that support is an issue,
+                    // push upstream to cut a new a release.
                     if self.format == SampleFormat::S24LE {
                         sample <<= 8;
                     }