earlephilhower / ESP8266Audio

Arduino library to play MOD, WAV, FLAC, MIDI, RTTTL, MP3, and AAC files on I2S DACs or with a software emulated delta-sigma DAC on the ESP8266 and ESP32
GNU General Public License v3.0
2.01k stars 432 forks source link

fix: Invalid data is assigned to the right channel when playing a mono MP3. #559

Closed lovyan03 closed 2 years ago

lovyan03 commented 2 years ago

Hello, Thanks for this great library and your work !

I have noticed that when I play a mono MP3, the right channel contains a buzzing sound.

https://user-images.githubusercontent.com/42724151/181280976-96016bac-9fe4-4213-b666-2deb13ee0616.MOV

On the other hand, the AAC decoder appears to assign the same data to both channels from mono data. So, in this pull request, I copied the data from the left channel to the right channel when the data is mono, so that the buzzer-like sound is not output.

I just couldn't decide if it would be best to make the change at this location in the source code. If you like this modification, I would be happy to hear from you, but if you have any suggestions on where I should fix it, please direct me.

Best regards.