Skip to content

Conversation

@reddevilmidzy
Copy link
Member

Note

Intermediate commits are intended to help review, but will be squashed add comment commit prior to merge.

part of #133895

r? Kivooeo

@rustbot
Copy link
Collaborator

rustbot commented Dec 26, 2025

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 Dec 26, 2025
@@ -1,8 +1,9 @@
//@ run-pass
// Regression test for #20676. Error was that we didn't support
// UFCS-style calls to a method in `Trait` where `Self` was bound to a
// trait object of type `Trait`. See also `ufcs-trait-object.rs`.
Copy link
Member Author

@reddevilmidzy reddevilmidzy Dec 26, 2025

Choose a reason for hiding this comment

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

ufcs-trait-object.rs had been renamed.
Considering the possibility of future changes, I changed it to point to the file of the current commit, but would it be better to set it to the commit when this file was created?

Copy link
Member

Choose a reason for hiding this comment

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

Honestly, I don't have a strong opinion about this and am absolutely not familiar with UFCS

Copy link
Member Author

Choose a reason for hiding this comment

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

Okay, I'll leave it as is.

Copy link
Member

Choose a reason for hiding this comment

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

Not sure if this has anything to share with consts and const-eval, let's move it, whether to cast or if we have anything related to chars

Copy link
Member Author

Choose a reason for hiding this comment

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

In issue #41998, I saw that the ICE log was coming from librustc_const_eval/eval.rs, so I placed it in the consts/const-eval directory.
Moved to cast\cast-to-char-compare.rs

delete tests/ui/issues/issue-22426.rs duplicated of tests/ui/match/guards.rs
Copy link
Member Author

@reddevilmidzy reddevilmidzy left a comment

Choose a reason for hiding this comment

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

thank you for the review.

View changes since this review

Copy link
Member Author

Choose a reason for hiding this comment

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

In issue #41998, I saw that the ICE log was coming from librustc_const_eval/eval.rs, so I placed it in the consts/const-eval directory.
Moved to cast\cast-to-char-compare.rs

Comment on lines 588 to 591
## `tests/ui/fn_traits/`

Tests for `#![feature(fn_traits)]`. See [`fn_traits` | The Unstable book](https://doc.rust-lang.org/nightly/unstable-book/library-features/fn-traits.html).

Copy link
Member Author

@reddevilmidzy reddevilmidzy Jan 5, 2026

Choose a reason for hiding this comment

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

and I dropped this commit because it overlaps with changes in #150474

cleaned up cast-enum-const.rs

cleaned up ufcs-trait-object-format.rs

add comment to cast-to-box-arr.rs

add comment to shadow-primitives.rs

add comment to associated-type-const-nomalization.rs

add comment to fn-trait-explicit-call.rs

add comment to call-unit-struct-impl-fn-once.rs

add comment to cast-to-char-compare.rs

add comment to self-in-method-body-resolves.rs

add comment to derive-debug-newtype-unsized-slice.rs

add comment to tuple-ref-order-distinct-impls.rs

add comment to derive-debug-generic-with-lifetime.rs

add comment to recursive-trait-bound-on-type-param.rs

cleaned up const-static-ref-to-closure.rs

add comment to const-iter-no-conflict-for-loop.rs
@Kivooeo
Copy link
Member

Kivooeo commented Jan 5, 2026

thanks for working on it!

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jan 5, 2026

📌 Commit 93f8ad9 has been approved by Kivooeo

It is now in the queue for this repository.

@bors bors 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 Jan 5, 2026
bors added a commit that referenced this pull request Jan 5, 2026
Rollup of 3 pull requests

Successful merges:

 - #149790 (Remove `Span` from segments of `AttrPath`)
 - #150384 (Tidying up tests/ui/issues 16 tests [7/N])
 - #150697 (relate.rs: tiny cleanup: eliminate temp vars 2)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 5bcdc7c into rust-lang:main Jan 5, 2026
11 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Jan 5, 2026
rust-timer added a commit that referenced this pull request Jan 5, 2026
Rollup merge of #150384 - reddevilmidzy:t14, r=Kivooeo

Tidying up tests/ui/issues 16 tests [7/N]

> [!NOTE]
> Intermediate commits are intended to help review, but will be squashed add comment commit prior to merge.

part of #133895

r? Kivooeo
@reddevilmidzy reddevilmidzy deleted the t14 branch January 6, 2026 13:37
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