-
Notifications
You must be signed in to change notification settings - Fork 13.6k
Closed
Labels
A-const-evalArea: Constant evaluation, covers all const contexts (static, const fn, ...)Area: Constant evaluation, covers all const contexts (static, const fn, ...)C-cleanupCategory: PRs that clean code up or issues documenting cleanup.Category: PRs that clean code up or issues documenting cleanup.E-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Description
This query is not doing anything anymore and only used in two locations.
Steps for resolving this issue:
- grep for
const_to_allocation
and remember where the twotcx.const_to_allocation
calls are - remove every mention of it
- grep for
ConstToAllocation
- remove every mentioon of it
- start building (this takes some time before it hits your use sites, let the computer work in parallel to you)
- go back to the former call sites and replace them by pattern matching on the argument to extract the necessary data from the
ByRef
and panic if it's notByRef
. - remove all the comments at the call sites talking about the query
- open a PR
- ping @RalfJung on the PR and watch his happiness meter go up
Metadata
Metadata
Assignees
Labels
A-const-evalArea: Constant evaluation, covers all const contexts (static, const fn, ...)Area: Constant evaluation, covers all const contexts (static, const fn, ...)C-cleanupCategory: PRs that clean code up or issues documenting cleanup.Category: PRs that clean code up or issues documenting cleanup.E-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.