[issue32990] Supporting extensible format(PCM) for wave.open(read-mode)

Change by Zackery Spytz : -- keywords: +patch pull_requests: +8921 stage: needs patch -> patch review ___ Python tracker ___ ___ Pyt

[issue32990] Supporting extensible format(PCM) for wave.open(read-mode)

Zackery Spytz added the comment: Andrea, are you still working on this issue? If not, I will submit a PR. -- nosy: +ZackerySpytz ___ Python tracker ___ ___

[issue32990] Supporting extensible format(PCM) for wave.open(read-mode)

Ned Deily added the comment: Thanks for working on this. Please follow the process in our Developers Guide and submit a PR against the master branch (for 3.8). After a core developer reviews it and if it is accepted, we can then decide about backports to other release branches. Also, if yo

[issue32990] Supporting extensible format(PCM) for wave.open(read-mode)

Andrea Celletti added the comment: I am currently working on a patch for this. When done can I try pushing this in 3.7 rather than 3.8? It is not much of an enhancement but rather fixing the code that raises an error for a perfectly valid PCM wave file (bugfix). -- versions: -Python

[issue32990] Supporting extensible format(PCM) for wave.open(read-mode)

Change by Serhiy Storchaka : -- nosy: +serhiy.storchaka stage: -> needs patch versions: +Python 3.8 ___ Python tracker ___ ___ Pytho

[issue32990] Supporting extensible format(PCM) for wave.open(read-mode)

New submission from Andrea Celletti : The wave.Wave_read class currently supports 8, 16, 24, and 32 bit PCM files. Wave files are only supported if the wFormatTag in the format chunk matches the flag WAVE_FORMAT_PCM, which is correct but incomplete for 24 bit files. According to the specificati