LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-20 01:37:39 +03:00
Code Issues Packages Projects Releases Wiki Activity

AudioServer: Skip mixing when volume is zero

Browse Source 
When volume is zero it is not necessary to go through the mixing loop.
The zero-filled buffer can be written directly to the device, instead,
similar to the muted case. Tested by using the piano app and the main
volume control.
This commit is contained in:
Alex Chronopoulos 2022-11-04 00:01:03 +01:00 committed by Sam Atkins
parent 5822bc9cb5
commit e86cab00b6
Notes: sideshowbarker 2024-07-17 04:43:24 +09:00 
Author: https://github.com/achronop 🔰
Commit: https://github.com/SerenityOS/serenity/commit/e86cab00b6
Pull-request: https://github.com/SerenityOS/serenity/pull/15963
Reviewed-by: https://github.com/AtkinsSJ ✅
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
Userland/Services/AudioServer/Mixer.cpp
View File

@ -96,19 +96,15 @@ void Mixer::mix()
}
}
if (m_muted) {
// Even though it's not realistic, the user expects no sound at 0%.
if (m_muted || m_main_volume < 0.01) {
m_device->write(m_zero_filled_buffer.data(), static_cast<int>(m_zero_filled_buffer.size()));
} else {
Array<u8, HARDWARE_BUFFER_SIZE_BYTES> buffer;
OutputMemoryStream stream { buffer };
for (auto& mixed_sample : mixed_buffer) {
// Even though it's not realistic, the user expects no sound at 0%.
if (m_main_volume < 0.01)
mixed_sample = Audio::Sample { 0 };
else
mixed_sample.log_multiply(static_cast<float>(m_main_volume));
mixed_sample.log_multiply(static_cast<float>(m_main_volume));
mixed_sample.clip();
LittleEndian<i16> out_sample;