[Patches] [ python-Patches-1601678 ] sys.intern()

[SourceForge.net]

Patches item #1601678, was opened at 2006-11-23 11:10
Message generated for change (Comment added) made by gbrandl
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1601678&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Library (Lib)
Group: Python 3000
>Status: Closed
Resolution: Accepted
Priority: 5
Private: No
Submitted By: Georg Brandl (gbrandl)
Assigned to: Guido van Rossum (gvanrossum)
Summary: sys.intern()

Initial Comment:
This patch moves id() and intern() into the sys module, as listed in PEP 3100.

----------------------------------------------------------------------

>Comment By: Georg Brandl (gbrandl)
Date: 2006-12-19 20:54

Message:
Logged In: YES 
user_id=849994
Originator: YES

Committed in rev. 53080. Updated the PEP in rev. 53082.
Submitted new patch for the 2to3 refactoring tool in #1619049.

----------------------------------------------------------------------

Comment By: Guido van Rossum (gvanrossum)
Date: 2006-12-19 19:26

Message:
Logged In: YES 
user_id=6380
Originator: NO

Looks good, check it in (and update PEP 3100 please).

Were there no other places where intern() is actually *used* in the
stdlib? Amazing!

Can I convince you to also contribute a fixer for the sandbox/2to3
refactoring tool?

----------------------------------------------------------------------

Comment By: Georg Brandl (gbrandl)
Date: 2006-12-19 18:55

Message:
Logged In: YES 
user_id=849994
Originator: YES

Updated patch only moves intern(), as per python-3000 discussion.
File Added: sys-intern.diff

----------------------------------------------------------------------

Comment By: Guido van Rossum (gvanrossum)
Date: 2006-11-23 16:43

Message:
Logged In: YES 
user_id=6380
Originator: NO

Aargh. I need to think more about whether I even want this. Doesn't PEP
3100 have a ? near that item? Everything with a ? needs to be reconsidered
before implemented. Add a ? if it's not already there for this issue.
Sorry.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1601678&group_id=5470