Skip to content

Reorganize tests/ui/issues [19/N]#158826

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
zedddie:gsoc-batch-19-meow
Jul 7, 2026
Merged

Reorganize tests/ui/issues [19/N]#158826
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
zedddie:gsoc-batch-19-meow

Conversation

@zedddie

@zedddie zedddie commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Part of GSoC'26 project

r? Kivooeo
@Teapot4195

@rustbot

rustbot commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator

This PR modifies tests/ui/issues/. If this PR is adding new tests to tests/ui/issues/,
please refrain from doing so, and instead add it to more descriptive subdirectories.

@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 Jul 5, 2026
Comment on lines +1 to +6
//! Regression test for <https://github.com/rust-lang/rust/issues/3220>.
//! Initially a pretty-printer bug, which omitted parentheses around `move`
//! keyword in lhs (`(move z).f()`), now this behaviour is untestable as
//! plain `move` outside a closure doesn't exist.
//!
//! Now appears to test that drop works correctly when value is moved by method.

@zedddie zedddie Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not really sure whether this test should stay or be deleted

View changes since the review

Comment on lines +23 to +37

{ fn f<X: ::std::marker()::Send>() {} }
//~^ ERROR parenthesized type parameters may only be used with a `Fn` trait

{ fn f() -> impl ::std::marker()::Send { } }
//~^ ERROR parenthesized type parameters may only be used with a `Fn` trait

}

#[derive(Clone)]
struct X;

impl ::std::marker()::Copy for X {}
//~^ ERROR parenthesized type parameters may only be used with a `Fn` trait

@zedddie zedddie Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merged tests/ui/typeck/parenthesized-type-parameters-error-32995.rs into this.

View changes since the review

@Teapot4195

Copy link
Copy Markdown
Contributor

Looks fine to me, although I'm not sure what to do with that 3220 test.

@Kivooeo

Kivooeo commented Jul 6, 2026

Copy link
Copy Markdown
Member

Let's keep that test since I didn't find any duplicates.

@bors r=Teapot4195 rollup

@rust-bors

rust-bors Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 082ff2f has been approved by Teapot4195

It is now in the queue for this repository.

⚠️ The following reviewer(s) could not be found: Teapot4195

🌲 The tree is currently closed for pull requests below priority 6. This pull request will be tested once the tree is reopened.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 6, 2026
rust-bors Bot pushed a commit that referenced this pull request Jul 7, 2026
Rollup of 13 pull requests

Successful merges:

 - #158478 (Use correct typing mode in mir building for the next solver)
 - #158796 (Distinguish null reference production from null pointer dereference in UB checks)
 - #158821 (add regression test for late-bound type method probe ICE)
 - #158245 (Update the rustc_perf submodule)
 - #158346 (Prevent rustdoc auto-trait ICEs with `-Znext-solver=globally`)
 - #158536 (Instatiate binder instead of skipping when suggesting function arg error)
 - #158553 (Avoid infinite recursion when trying to build valtrees for self-referential consts)
 - #158679 (Document blocking guarantees for `std::sync`)
 - #158826 (Reorganize `tests/ui/issues` [19/N])
 - #158860 (Delete `impl_def_id` field from `ImplHeader`)
 - #158867 (rewrite `Align::max_for_target` in a more obvious way)
 - #158878 (borrowck: Inline `free_region_constraint_info()` to simplify the code)
 - #158881 (Update mdbook to 0.5.4)
@rust-bors rust-bors Bot merged commit 41eeb9b into rust-lang:main Jul 7, 2026
13 checks passed
@rustbot rustbot added this to the 1.99.0 milestone Jul 7, 2026
rust-timer added a commit that referenced this pull request Jul 7, 2026
Rollup merge of #158826 - zedddie:gsoc-batch-19-meow, r=Teapot4195

Reorganize `tests/ui/issues` [19/N]

Part of [GSoC'26 project](https://rust-lang.zulipchat.com/#narrow/channel/421156-gsoc/topic/Project.3A.20Reorganizing.20tests.2Fui.2Fissues)

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

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

4 participants