➜
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.
Created on 2008-03-17 17:55 by brett.cannon, last changed 2022-04-11 14:56 by admin. 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) *  |
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) * |
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) * |
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) * |
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) * |
Date: 2008-03-17 23:45 | |
Okay thanks. Closing this one as something that isn't really needed. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:56:31 | admin | set | github: 46592 |
| 2008-03-17 23:45:47 | rhettinger | set | status: open -> closed resolution: rejected messages: + msg63807 |
| 2008-03-17 23:23:39 | gvanrossum | set | priority: critical -> low messages: + msg63806 |
| 2008-03-17 21:44:48 | rhettinger | set | assignee: gvanrossum messages: + msg63786 nosy: + gvanrossum, rhettinger |
| 2008-03-17 20:17:12 | brett.cannon | set | priority: release blocker -> critical |
| 2008-03-17 20:03:58 | benjamin.peterson | set | files:
+ backport_sys_intern.patch keywords: + patch messages: + msg63750 nosy: + benjamin.peterson |
| 2008-03-17 17:55:48 | brett.cannon | create | |