classification
Title: Backport intern() -> sys.intern()
Type: Stage:
Components: Interpreter Core Versions: Python 2.6
process
Status: closed Resolution: rejected
Dependencies: Superseder:
Assigned To: gvanrossum Nosy List: benjamin.peterson, brett.cannon, gvanrossum, rhettinger
Priority: low Keywords: 26backport, patch

Created on 2008-03-17 17:55 by brett.cannon, last changed 2008-03-17 23:45 by rhettinger. This issue is now closed.

Files
File name Uploaded Description Edit
backport_sys_intern.patch benjamin.peterson, 2008-03-17 20:03 Backports sys.intern
Messages (5)
msg63698 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2008-03-17 17:55
intern() needs to be added as sys.intern(). Then a 2to3 fixer needs to
be written.
msg63750 - (view) Author: Benjamin Peterson (benjamin.peterson) * (Python committer) Date: 2008-03-17 20:03
Here's a patch which moves intern to sysmodule.c, adds a Py3k warning,
updates the docs, and moves the tests.
msg63786 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2008-03-17 21:44
Guido, do you want this alias in Py2.6?

Seems like it should just be a 2-to-3 fixer issue only.

Also, I vaguely remembered that we weren't going to expose interning at
all.  There was a discussion on python-dev a couple years ago where I
believe that we concluded that this is primarily an internal
optimization and that almost no pure python programs benefitted from
calling intern() directly. IOW, I thought this was dead as a public API.
msg63806 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2008-03-17 23:23
I see no great advantage in having it backported, though I also don't
see no great harm.

Since we still use interning as an internal speed-up, I believe in
exposing the API, past discussions notwithstanding.
msg63807 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2008-03-17 23:45
Okay thanks.  Closing this one as something that isn't really needed.
History
Date User Action Args
2008-03-17 23:45:47rhettingersetstatus: open -> closed
resolution: rejected
messages: + msg63807
2008-03-17 23:23:39gvanrossumsetpriority: critical -> low
messages: + msg63806
2008-03-17 21:44:48rhettingersetassignee: gvanrossum
messages: + msg63786
nosy: + gvanrossum, rhettinger
2008-03-17 20:17:12brett.cannonsetpriority: release blocker -> critical
2008-03-17 20:03:58benjamin.petersonsetfiles: + backport_sys_intern.patch
keywords: + patch
messages: + msg63750
nosy: + benjamin.peterson
2008-03-17 17:55:48brett.cannoncreate