classification
Title: Deprecate openfp() in aifc, sunau and wave
Type: enhancement Stage: patch review
Components: Library (Lib) Versions: Python 3.7
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: brian.curtin Nosy List: brian.curtin, serhiy.storchaka
Priority: normal Keywords: patch

Created on 2017-11-08 21:56 by brian.curtin, last changed 2017-11-10 16:38 by brian.curtin.

Pull Requests
URL Status Linked Edit
PR 4344 merged brian.curtin, 2017-11-08 22:05
Messages (7)
msg305912 - (view) Author: Brian Curtin (brian.curtin) * (Python committer) Date: 2017-11-08 21:56
Since 1993, aifc.openfp has simply pointed to aifc.open as a matter of backwards compatibility. See https://github.com/python/cpython/commit/7bc817d5ba917528e8bd07ec461c635291e7b06a for the exact change.

aifc.openfp is both undocumented and untested, and in looking at doc coverage I decided it's better to deprecate it since it's silently been there for 25 years.
msg305917 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-11-08 22:41
Don't forget about wave and sunau.

You can use a common test for all three modules.
msg305924 - (view) Author: Brian Curtin (brian.curtin) * (Python committer) Date: 2017-11-09 00:18
i was going to do them as separate bugs and PRs per module, but if one is fine then i can do that.

Updating the title as well since nothing is beings removed (was originally going to suggest skipping to removal but won’t  do that).
msg305989 - (view) Author: Brian Curtin (brian.curtin) * (Python committer) Date: 2017-11-09 22:04
Serhiy, where should a common test that covers all three of these go? I'm not seeing an obvious place for it.
msg305995 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-11-09 23:03
audiotests.py. Create class AudioMiscTests and inherit classes AifcMiscTest, SunauMiscTest, WaveMiscTest from it instead of AudioTests. There is more work because there was no base for misc tests, but this will make adding other common misc test easier.
msg305999 - (view) Author: Brian Curtin (brian.curtin) * (Python committer) Date: 2017-11-09 23:46
I think https://github.com/python/cpython/pull/4344 covers what you're looking for.
msg306038 - (view) Author: Brian Curtin (brian.curtin) * (Python committer) Date: 2017-11-10 16:38
New changeset 9f914a01affc55abe799afc521ce71612bb495a5 by Brian Curtin in branch 'master':
bpo-31985: Deprecate openfp in aifc, sunau, and wave (#4344)
https://github.com/python/cpython/commit/9f914a01affc55abe799afc521ce71612bb495a5
History
Date User Action Args
2017-11-10 16:38:28brian.curtinsetmessages: + msg306038
2017-11-09 23:46:08brian.curtinsetmessages: + msg305999
2017-11-09 23:03:32serhiy.storchakasetmessages: + msg305995
2017-11-09 22:04:05brian.curtinsetmessages: + msg305989
2017-11-09 22:01:13serhiy.storchakasettitle: Deprecate aifc.openfp -> Deprecate openfp() in aifc, sunau and wave
2017-11-09 00:18:21brian.curtinsetmessages: + msg305924
title: Deprecate/remove aifc.openfp -> Deprecate aifc.openfp
2017-11-08 22:41:24serhiy.storchakasetnosy: + serhiy.storchaka
messages: + msg305917
2017-11-08 22:05:04brian.curtinsetkeywords: + patch
stage: needs patch -> patch review
pull_requests: + pull_request4302
2017-11-08 21:56:36brian.curtincreate