-
-
Notifications
You must be signed in to change notification settings - Fork 15k
Document and clean up ty::print some more. #59189
Copy link
Copy link
Open
Labels
A-prettyArea: Pretty printing (including `-Z unpretty`)Area: Pretty printing (including `-Z unpretty`)C-cleanupCategory: PRs that clean code up or issues documenting cleanup.Category: PRs that clean code up or issues documenting cleanup.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.
Description
Metadata
Metadata
Assignees
Labels
A-prettyArea: Pretty printing (including `-Z unpretty`)Area: Pretty printing (including `-Z unpretty`)C-cleanupCategory: PRs that clean code up or issues documenting cleanup.Category: PRs that clean code up or issues documenting cleanup.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.
Type
Fields
Give feedbackNo fields configured for issues without a type.
In order to get #58140 merged before it bitrots again, and needs an eleventh rebase, I've left some of the review comments (by @nikomatsakis) unresolved.
There's also a few things, such as const generics,
$(...)?in macros, etc. that weren't possible when the PR was originally authored, but can be taken into account now.