-
Notifications
You must be signed in to change notification settings - Fork 12.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename projection_def_id
to item_def_id
#134158
base: master
Are you sure you want to change the base?
Conversation
HIR ty lowering was modified cc @fmease |
Personally, I think something like "alias_def_id" is even better, since "item" could mean the trait item. But, given that this just aligns the name with elsewhere: |
@bors r- I can further unify the naming |
Actually I was mistaken, and the function is not defined on Anyways, I cannot be bothered to change this for now so 🤔 @bors r=jackh726 |
…kh726 Rename `projection_def_id` to `item_def_id` Renames `projection_def_id` to `item_def_id`, since `item_def_id` is what we call the analogous method for ~~`AliasTerm`/`AliasTy`~~ `PolyExistentialProjection`. I keep forgetting that this one is not called `item_def_id`.
Renames
projection_def_id
toitem_def_id
, sinceitem_def_id
is what we call the analogous method forAliasTerm
/AliasTy
PolyExistentialProjection
. I keep forgetting that this one is not calleditem_def_id
.