Skip to content

remove new_from_def_id and alias_ty_kind_from_def_id#155371

Draft
josetorrs wants to merge 1 commit intorust-lang:mainfrom
josetorrs:add-alias-term-kind-to-type-relative-path
Draft

remove new_from_def_id and alias_ty_kind_from_def_id#155371
josetorrs wants to merge 1 commit intorust-lang:mainfrom
josetorrs:add-alias-term-kind-to-type-relative-path

Conversation

@josetorrs
Copy link
Copy Markdown
Contributor

r? @WaffleLapkin

related issue: #154941

tackling the task:

  • remove calls to new_from_def_id and alias_ty_kind_from_def_id and replace them with calls to e.g. new_projection directly

mainly raising as an alternative to #155323

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-clippy Relevant to the Clippy team. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Apr 16, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 16, 2026

WaffleLapkin is not on the review rotation at the moment.
They may take a while to respond.

| ty::AliasTermKind::InherentConst
| ty::AliasTermKind::FreeConst
| ty::AliasTermKind::UnevaluatedConst => {
unreachable!()
Copy link
Copy Markdown
Contributor Author

@josetorrs josetorrs Apr 16, 2026

Choose a reason for hiding this comment

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

admittedly not sure if unreachable!() or something else is desired

View changes since the review

@rust-log-analyzer

This comment has been minimized.

@rust-bors

This comment has been minimized.

@josetorrs josetorrs force-pushed the add-alias-term-kind-to-type-relative-path branch from 5327d39 to deb0af0 Compare May 1, 2026 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-clippy Relevant to the Clippy team. 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