Skip to content

Rc and Arc are missing new_cyclic_in #132

@matthewpipie

Description

@matthewpipie

new_cyclic was added in Rust 1.60 and allows users to create cyclic data structures without unsafe code; however, there is no equivalent new_cyclic_in. If this is an oversight, I'm happy to make a PR.

Activity

Amanieu

Amanieu commented on Aug 29, 2024

@Amanieu
Member

Yes, this does seem to be an oversight, feel free to make a PR.

matthewpipie

matthewpipie commented on Aug 29, 2024

@matthewpipie
Author

Great, I made a PR here: rust-lang/rust#129674.

I have a few outstanding questions in that PR. If you wouldn't mind taking a look at that, it would be much appreciated. I'm very new to the standard library and don't know some of its conventions.

matthewpipie

matthewpipie commented on Sep 5, 2024

@matthewpipie
Author

Bump! Would love some feedback on if that PR is done appropriately.

zakarumych

zakarumych commented on Sep 6, 2024

@zakarumych

PR is still marked as "draft".

matthewpipie

matthewpipie commented on Sep 6, 2024

@matthewpipie
Author

Thank you, fixed!

added a commit that references this issue on Sep 17, 2024
5385303
added a commit that references this issue on Sep 17, 2024
added a commit that references this issue on Sep 17, 2024
f6fd305
added a commit that references this issue on Sep 17, 2024
matthewpipie

matthewpipie commented on Sep 17, 2024

@matthewpipie
Author

Merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @Amanieu@zakarumych@matthewpipie

        Issue actions

          Rc and Arc are missing new_cyclic_in · Issue #132 · rust-lang/wg-allocators