-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
Rename LangString::all_false() to default() #80015
Copy link
Copy link
Closed
Labels
A-markdown-parsingArea: Markdown parsing for doc-commentsArea: Markdown parsing for doc-commentsC-cleanupCategory: PRs that clean code up or issues documenting cleanup.Category: PRs that clean code up or issues documenting cleanup.T-rustdocRelevant to the rustdoc team, which will review and decide on the PR/issue.Relevant to the rustdoc team, which will review and decide on the PR/issue.
Metadata
Metadata
Assignees
Labels
A-markdown-parsingArea: Markdown parsing for doc-commentsArea: Markdown parsing for doc-commentsC-cleanupCategory: PRs that clean code up or issues documenting cleanup.Category: PRs that clean code up or issues documenting cleanup.T-rustdocRelevant to the rustdoc team, which will review and decide on the PR/issue.Relevant to the rustdoc team, which will review and decide on the PR/issue.
Type
Fields
Give feedbackNo fields configured for issues without a type.
LangString::all_falseis not actually all_false -rustwill returntrue. It would be nice to rename it todefault()at some point (and maybe addimpl Default for LangStringwhile we're at it).cc @poliorcetics
Originally posted by @jyn514 in #80013 (comment)