Skip to content

delete some duplicated tests#153819

Open
cyrgani wants to merge 1 commit intorust-lang:mainfrom
cyrgani:dupe-tests
Open

delete some duplicated tests#153819
cyrgani wants to merge 1 commit intorust-lang:mainfrom
cyrgani:dupe-tests

Conversation

@cyrgani
Copy link
Contributor

@cyrgani cyrgani commented Mar 13, 2026

Delete some tests which have almost-exact duplicates (only difference is formatting or the concrete integer values used) in the tests/ui/moves directory.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 13, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 13, 2026

r? @adwinwhite

rustbot has assigned @adwinwhite.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 69 candidates
  • Random selection from 14 candidates

@bjorn3
Copy link
Member

bjorn3 commented Mar 13, 2026

I did some digging and it looks like there were originally tests for @T (managed boxes, aka GCed pointers) added in 78b0d33, which were then copied for ~T (unique boxes) in 18b01d5. Later both @T and ~T were changed to Box<T> in aa48654 and 090040b respectively.

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

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants