This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: PEP 3137: Remove the buffer API from PyUnicode
Type: Stage:
Components: Versions: Python 3.0
process
Status: closed Resolution: accepted
Dependencies: Superseder:
Assigned To: gvanrossum Nosy List: alexandre.vassalotti, gvanrossum
Priority: normal Keywords: patch

Created on 2007-10-11 03:00 by alexandre.vassalotti, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
unicode_rm_buf_api.patch alexandre.vassalotti, 2007-10-11 03:00
unicode_rm_buf_api_02.patch alexandre.vassalotti, 2007-10-13 23:46
Messages (5)
msg56336 - (view) Author: Alexandre Vassalotti (alexandre.vassalotti) * (Python committer) Date: 2007-10-11 03:00
This patch removes the buffer API from PyUnicode, as specified by PEP
3137. All the unit tests passes. However, I believe there is a few
function calls to PyArg API that could become problematic if they are
given a PyUnicode object:

  % egrep -R --include='*.c' 'PyArg.*Parse.*"[^;:]*[cwt].*"' py3k/

I haven't checked these function calls yet. So, it would probably be a
good idea to wait I do so before committing this patch.
msg56392 - (view) Author: Alexandre Vassalotti (alexandre.vassalotti) * (Python committer) Date: 2007-10-13 23:46
There was a problem with one of the call of PyArg_ParseTuple in the OS/2
version of listdir() in the posix module. I also clarified the error
message of the 't' format unit.
msg56395 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2007-10-14 00:49
Can you svn update and resolve conflicts? Travis made some changes to
unicodeobject.c that conflic with your patch.
msg56400 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2007-10-14 01:19
You can check this in. You do have checkin privs right?
msg56403 - (view) Author: Alexandre Vassalotti (alexandre.vassalotti) * (Python committer) Date: 2007-10-14 02:06
Committed in r58455.
History
Date User Action Args
2022-04-11 14:56:27adminsetgithub: 45601
2007-10-14 02:09:05gvanrossumsetstatus: open -> closed
2007-10-14 02:06:21alexandre.vassalottisetmessages: + msg56403
2007-10-14 01:19:42gvanrossumsetresolution: accepted
messages: + msg56400
2007-10-14 00:49:08gvanrossumsetmessages: + msg56395
2007-10-13 23:46:02alexandre.vassalottisetfiles: + unicode_rm_buf_api_02.patch
messages: + msg56392
2007-10-11 16:51:51gvanrossumsetassignee: gvanrossum
nosy: + gvanrossum
2007-10-11 05:29:12loewissetkeywords: + patch
2007-10-11 03:00:26alexandre.vassalotticreate