homepage

Inadvertently strings accepted as arguments of audioop functions. This is a meaningless behavior and remnant of Python 2. We should drop string support.

What happens if you pass strings?
If it doesn't work we can just fix it to raise an exception and stop returning random results, if it works but bytes should be used instead, we should go through a deprecation process.

> What happens if you pass strings?
They are encoded with UTF-8. See 's#' and 's*' formats in PyArg_ParseTuple() ('y*' recommended for bytes).

Accepting Unicode strings is surprising and must fail. I don't think that a deprecation process is required here.

> Accepting Unicode strings is surprising and must fail. I don't think that a deprecation process is required here.
Should it be fixed in 3.4 only or in all 3.x?

The deprecation process can be avoided only if passing strings results in a meaningless result. If there are cases where users are passing strings and everything works fine -- even if they should be passing bytes instead -- we should deprecate strings first to warn the users that the "feature" is going to be removed.

In 2.x it was have a little sense (if you use some 8-bit encoding as default encoding, but the default was 7-bit ascii). But with utf-8 it doesn't have sense and should quickly lead to an unexpected result (or just fail in most cases).

> Should it be fixed in 3.4 only or in all 3.x?
Such change cannot be done in a minor release (like 3.3.x), so it can
only be done in Python 3.4.

Here is a patch. Now audioop functions no more accept str, but accept bytes-like objects instead.

audioop_buffer.patch: In general, I don't like goto in C. But to cleanup something when exiting a function, I like goto to implement something similar to Python try/finally. What do you think Serhiy?

Well. In updated patch I use gotos where it decreases the number of lines (total decreasing is almost 30 lines).

If there are no objections I will commit last patch tomorrow.

New changeset bab0cbf86835 by Serhiy Storchaka in branch 'default':
Issue #16685: Added support for any bytes-like objects in the audioop module.
http://hg.python.org/cpython/rev/bab0cbf86835 

The patch as committed causes the Windows 64-bit builds to fail to compile. http://buildbot.python.org/all/builders/AMD64%20Windows7%20SP1%203.x/builds/3187/steps/compile/logs/stdio 

New changeset 35cd00465624 by Serhiy Storchaka in branch 'default':
Fixed compile error on Windows caused by arithmetic with void * pointers
http://hg.python.org/cpython/rev/35cd00465624 

Fixed. Thank you Jason.

New changeset b96f4ee1b08b by Serhiy Storchaka in branch 'default':
Issue #16685: Added support for writing any bytes-like objects in the aifc,
http://hg.python.org/cpython/rev/b96f4ee1b08b 

New changeset c375697f062e by R David Murray in branch 'default':
whatsnew: auidioop.byteswap (#16685).
http://hg.python.org/cpython/rev/c375697f062e