Skip to content

vec::IntoIter<T>: UnwindSafe should not require T: RefUnwindSafe #144707

New issue
New issue
Closed
#145665
Closed
vec::IntoIter<T>: UnwindSafe should not require T: RefUnwindSafe#144707
#145665
Labels
C-bugCategory: This is a bug.T-libs-api[DEPRECATED; DO NOT USE]

Description

@SabrinaJewson
SabrinaJewson
opened on Jul 30, 2025

Currently, vec::IntoIter<T> is only UnwindSafe if T is RefUnwindSafe. It probably inherits this implementation from NonNull, but I don’t think it makes much sense. I really don’t think you can cause state corruption with a vec::IntoIter<RefCell<T>> in scope for example, since they’d all be dropped.

Meta

rustc --version --verbose:

rustc 1.88.0 (6b00bc388 2025-06-23)
binary: rustc
commit-hash: 6b00bc3880198600130e1cf62b8f8a93494488cc
commit-date: 2025-06-23
host: x86_64-unknown-linux-gnu
release: 1.88.0
LLVM version: 20.1.5

Activity

  1. ImageSabrinaJewson
    added
    C-bugCategory: This is a bug.
    on Jul 30, 2025
  2. Imagerustbot
    added
    needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging
    on Jul 30, 2025

  3. bjorn3 commented on Jul 31, 2025

    @bjorn3
    bjorn3
    on Jul 31, 2025
    Member

    Changing the bound to T: UnwindSafe would make sense to me, but I don't know if we can do so in a backwards compatible way.

  4. SabrinaJewson commented on Jul 31, 2025

    @SabrinaJewson
    SabrinaJewson
    on Jul 31, 2025
    ContributorAuthor

    It already requires T: UnwindSafe, so I don’t think it should be a problem.

  5. Imagelolbinarycat
    added
    T-libs-api[DEPRECATED; DO NOT USE]
    on Aug 4, 2025
  6. ImageZalathar
    added 2 commits that reference this issue on Oct 26, 2025

    Rollup merge of rust-lang#145665 - GrigorenkoPV:144707, r=tgross35

    bf3c754

    Rollup merge of rust-lang#145665 - GrigorenkoPV:144707, r=tgross35

    6fc0dea
  7. Imagebors
    added a commit that references this issue on Oct 26, 2025

    Auto merge of #145665 - GrigorenkoPV:144707, r=tgross35

    a8664a1
  8. Imagebors
    closed this as completedin a8664a1on Oct 26, 2025
  9. Imagegithub-actions
    added 2 commits that reference this issue on Oct 27, 2025

    Auto merge of #145665 - GrigorenkoPV:144707, r=tgross35

    427e7dd

    Auto merge of #145665 - GrigorenkoPV:144707, r=tgross35

    30b3369
  10. Imagejieyouxu
    removed
    needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging
    on Nov 1, 2025
  11. Imagemakai410
    added 4 commits that reference this issue on Nov 4, 2025

    Auto merge of #145665 - GrigorenkoPV:144707, r=tgross35

    3c117da

    Auto merge of rust-lang#145665 - GrigorenkoPV:144707, r=tgross35

    432c208

    Auto merge of rust-lang#145665 - GrigorenkoPV:144707, r=tgross35

    4e7509e

    Auto merge of #145665 - GrigorenkoPV:144707, r=tgross35

    b7cd24d
  12. Imagegithub-actions
    added a commit that references this issue on Nov 30, 2025

    Auto merge of rust-lang#145665 - GrigorenkoPV:144707, r=tgross35

    ce33db0
  13. ImageKobzol
    added a commit that references this issue on Dec 29, 2025

    Auto merge of #145665 - GrigorenkoPV:144707, r=tgross35

    1f79fdc
  14. Imagemakai410
    added a commit that references this issue on Mar 27, 2026

    Auto merge of #145665 - GrigorenkoPV:144707, r=tgross35

    41a86a2
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

    C-bugCategory: This is a bug.T-libs-api[DEPRECATED; DO NOT USE]

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions