classification
Title: audioop module needs an int -> Py_ssize_t upgrade
Type: performance Stage: resolved
Components: Extension Modules Versions: Python 3.2
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: mark.dickinson
Priority: normal Keywords: easy, patch

Created on 2010-05-10 18:38 by mark.dickinson, last changed 2010-05-11 13:35 by mark.dickinson. This issue is now closed.

Files
File name Uploaded Description Edit
audioop_Py_ssize_t.patch mark.dickinson, 2010-05-11 12:41
Messages (3)
msg105454 - (view) Author: Mark Dickinson (mark.dickinson) * (Python committer) Date: 2010-05-10 18:38
The audioop module still uses 'int' for most of its sizes;  it would be better if it used 'Py_ssize_t' instead, so that data aren't artificially limited to 2Gb on 64-bit systems.

Converting ought to be a fairly straightforward task for someone wanting to get their feet wet with CPython hacking;  marking as easy.
msg105506 - (view) Author: Mark Dickinson (mark.dickinson) * (Python committer) Date: 2010-05-11 12:41
Here's a patch.
msg105511 - (view) Author: Mark Dickinson (mark.dickinson) * (Python committer) Date: 2010-05-11 13:35
The audioop module was made PY_SSIZE_T_CLEAN in r81083.
History
Date User Action Args
2010-05-11 13:35:12mark.dickinsonsetstatus: open -> closed
messages: + msg105511

components: + Extension Modules
resolution: fixed
stage: resolved
2010-05-11 12:41:54mark.dickinsonsetfiles: + audioop_Py_ssize_t.patch
keywords: + patch
messages: + msg105506
2010-05-10 20:00:39pitroulinkissue8677 dependencies
2010-05-10 18:38:03mark.dickinsoncreate