Skip to content

rename local parameter, refactor ty::Free#155063

Open
josetorrs wants to merge 1 commit intorust-lang:mainfrom
josetorrs:refactor-alias-free
Open

rename local parameter, refactor ty::Free#155063
josetorrs wants to merge 1 commit intorust-lang:mainfrom
josetorrs:refactor-alias-free

Conversation

@josetorrs
Copy link
Copy Markdown
Contributor

related issue: #154941

tackling these two tasks:

  • replace all the @ ty::AliasTy { matches with just using args: _args instead
  • replace def_id: did with def_id where possible

though this PR only refactors for the following type (due to just trying to familiarize myself with this):

pub enum AliasTyKind<I: Interner> {
    Free { def_id: I::DefId },
    // omitted
}

r? @WaffleLapkin

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

rustbot commented Apr 9, 2026

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

@josetorrs josetorrs marked this pull request as ready for review April 9, 2026 20:23
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 9, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 9, 2026

HIR ty lowering was modified

cc @fmease

@rustbot rustbot removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Apr 9, 2026
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.

3 participants