[Patches] [ python-Patches-1623563 ] Allow __class __ assignment for classes with __slots__
SourceForge.net
noreply at sourceforge.net
Thu Feb 8 19:41:25 CET 2007
Patches item #1623563, was opened at 2006-12-28 06:48
Message generated for change (Comment added) made by jimjjewett
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1623563&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: Core (C code)
Group: Python 2.6
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: TH (therve)
Assigned to: Nobody/Anonymous (nobody)
Summary: Allow __class __ assignment for classes with __slots__
Initial Comment:
I made a modification in typeobject.c to allow __class__ modification for classes with slots. It's basically a change in same_slots_added to count the offset of the slots and check if the names of the slots are the same (in the naive way).
I don't check if slots are in a different order, that may be an improve.
The patch is against trunk, with some tests. It's my first submission on Python, so every feedback will be welcome :).
----------------------------------------------------------------------
Comment By: Jim Jewett (jimjjewett)
Date: 2007-02-08 13:41
Message:
Logged In: YES
user_id=764593
Originator: NO
Review five other patches.
Post the review summaries (and tracker numbers) to the python-dev mailing
list.
In that same message, ask someone with commit privileges to do the 5:1
deal, pointing at this tracker number.
----------------------------------------------------------------------
Comment By: TH (therve)
Date: 2007-02-06 08:43
Message:
Logged In: YES
user_id=1038797
Originator: YES
Is there anything I can do to have a resolution on this ?
Thanks.
----------------------------------------------------------------------
Comment By: Jim Jewett (jimjjewett)
Date: 2006-12-30 19:26
Message:
Logged In: YES
user_id=764593
Originator: NO
Just going through the list doesn't seem so naive to me. If the slots are
in a different order, then you would need to move the data around -- which
borders on "maybe they ought to have written the translator themselves"
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1623563&group_id=5470
More information about the Patches
mailing list