-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
rustdoc-json: Default value of assoc type in blanket impls missing #98658
Copy link
Copy link
Closed
Labels
A-rustdoc-jsonArea: Rustdoc JSON backendArea: Rustdoc JSON backendC-bugCategory: This is a bug.Category: This is a bug.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.requires-nightlyThis issue requires a nightly compiler in some way. When possible, use a F-* label instead.This issue requires a nightly compiler in some way. When possible, use a F-* label instead.
Metadata
Metadata
Assignees
Labels
A-rustdoc-jsonArea: Rustdoc JSON backendArea: Rustdoc JSON backendC-bugCategory: This is a bug.Category: This is a bug.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.requires-nightlyThis issue requires a nightly compiler in some way. When possible, use a F-* label instead.This issue requires a nightly compiler in some way. When possible, use a F-* label instead.
Type
Fields
Give feedbackNo fields configured for issues without a type.
Step-by-step
Run
cargo +nightly-2022-06-29 rustdoc -- -Z unstable-options --output-format jsonwith this insrc/lib.rs:and inspect
target/doc/repro.json.Expected result
There is an item for the
Errorof the blanket implimpl<T, U> const TryFrom<U> for T where U: ~const Into<T>and it's default value isInfallible.Actual
There is an item but the default value is missing. It looks like this:
When using
nightly-2022-06-28instead ofnightly-2022-06-29the information about the default value is present. It looks like this:This change, which in practice is a regression, is caused by #98577. But we can't revert it of course because that will result in ICEs which is even worse. So we need some kind of follow-up fix.
@rustbot labels +A-rustdoc-json +T-rustdoc +C-bug +requires-nightly