Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug fix 702858: deepcopying reflexive objects #38198

Closed
staschuk mannequin opened this issue Mar 22, 2003 · 7 comments
Closed

bug fix 702858: deepcopying reflexive objects #38198

staschuk mannequin opened this issue Mar 22, 2003 · 7 comments
Labels
stdlib Python modules in the Lib dir

Comments

@staschuk
Copy link
Mannequin

staschuk mannequin commented Mar 22, 2003

BPO 707900
Nosy @gvanrossum, @tim-one
Files
  • deepcopy-reflexive-object.patch
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
    closed_at = <Date 2005-01-09.17:09:17.000>
    created_at = <Date 2003-03-22.04:08:59.000>
    labels = ['library']
    title = 'bug fix 702858: deepcopying reflexive objects'
    updated_at = <Date 2005-01-09.17:09:17.000>
    user = 'https://bugs.python.org/staschuk'

    bugs.python.org fields:

    activity = <Date 2005-01-09.17:09:17.000>
    actor = 'jlgijsbers'
    assignee = 'nnorwitz'
    closed = True
    closed_date = None
    closer = None
    components = ['Library (Lib)']
    creation = <Date 2003-03-22.04:08:59.000>
    creator = 'staschuk'
    dependencies = []
    files = ['5112']
    hgrepos = []
    issue_num = 707900
    keywords = ['patch']
    message_count = 7.0
    messages = ['43106', '43107', '43108', '43109', '43110', '43111', '43112']
    nosy_count = 5.0
    nosy_names = ['gvanrossum', 'tim.peters', 'nnorwitz', 'jlgijsbers', 'staschuk']
    pr_nums = []
    priority = 'normal'
    resolution = 'out of date'
    stage = None
    status = 'closed'
    superseder = None
    type = None
    url = 'https://bugs.python.org/issue707900'
    versions = ['Python 2.2']

    @staschuk
    Copy link
    Mannequin Author

    staschuk mannequin commented Mar 22, 2003

    A fix for bug 702858, which concerns the inability of
    copy.deepcopy to correctly process reflexive
    new-style class instances, that is, instances referring
    to themselves.

    The fix is one line; the other 51 lines in the patch
    are altered and enhanced altered tests in
    test_copy.py for this kind of thing.

    @staschuk staschuk mannequin closed this as completed Mar 22, 2003
    @staschuk staschuk mannequin assigned nnorwitz Mar 22, 2003
    @staschuk staschuk mannequin added the stdlib Python modules in the Lib dir label Mar 22, 2003
    @staschuk staschuk mannequin closed this as completed Mar 22, 2003
    @staschuk staschuk mannequin assigned nnorwitz Mar 22, 2003
    @staschuk staschuk mannequin added the stdlib Python modules in the Lib dir label Mar 22, 2003
    @nnorwitz
    Copy link
    Mannequin

    nnorwitz mannequin commented May 25, 2003

    Logged In: YES
    user_id=33168

    This patch works on 2.3. I'm not sure if this is fixing a
    bug or a feature. The change seems reasonable, but I don't
    know enough about copy to know if there are any negative
    consequences. I can check in if someone else agrees. Tim?
    Guido?

    @tim-one
    Copy link
    Member

    tim-one commented May 25, 2003

    Logged In: YES
    user_id=31435

    Assigned to Guido because he knows more about this and
    I'm out of time for the next two days.

    @gvanrossum
    Copy link
    Member

    Logged In: YES
    user_id=6380

    Looks fine to me; classic instances already did the same
    thing. Thanks, Steven!

    @gvanrossum
    Copy link
    Member

    Logged In: YES
    user_id=6380

    Reopened - Neal, would you mind backporting this to 2.2?

    @nnorwitz
    Copy link
    Mannequin

    nnorwitz mannequin commented Jun 30, 2003

    Logged In: YES
    user_id=33168

    I'm working on this (slowly). It wasn't a simple backport.

    @jlgijsbers
    Copy link
    Mannequin

    jlgijsbers mannequin commented Jan 9, 2005

    Logged In: YES
    user_id=469548

    Closing this, as no 2.2 bugfix release if forthcoming anymore.

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 9, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    stdlib Python modules in the Lib dir
    Projects
    None yet
    Development

    No branches or pull requests

    2 participants