Skip to content
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

Document never type fallback in !'s docs #124419

Merged
merged 5 commits into from
May 2, 2024
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
41 changes: 41 additions & 0 deletions library/core/src/primitive_docs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -268,6 +268,47 @@ mod prim_bool {}
/// [`Debug`]: fmt::Debug
/// [`default()`]: Default::default
///
/// # Never type fallback
///
/// When the compiler sees a value of type `!` it implicitly inserts a coercion (if possible),
WaffleLapkin marked this conversation as resolved.
Show resolved Hide resolved
/// to allow type check to infer any type:
WaffleLapkin marked this conversation as resolved.
Show resolved Hide resolved
///
/// ```rust,ignore (illustrative-and-has-placeholders)
/// // this
/// let x: u8 = panic!();
///
/// // is (essentially) turned by the compiler into
/// let x: u8 = absurd(panic!());
///
/// // where absurd is a function with the following signature
/// // (it's sound, because `!` always marks unreachable code):
/// fn absurd<T>(_: !) -> T { ... }
// FIXME: use `core::convert::absurd` here instead, once it's merged
/// ```
///
/// While it's convenient to be able to use non-diverging code in one of the branches (like
/// `if a { b } else { return }`) this could lead to compilation errors:
WaffleLapkin marked this conversation as resolved.
Show resolved Hide resolved
///
/// ```compile_fail
/// // this
/// { panic!() };
///
/// // gets turned into this
/// { absurd(panic!()) }; // error: can't infer the type of `absurd`
/// ```
///
/// To prevent such errors, compiler remembers where it inserted `absurd` calls, and if it can't
/// infer their type, it sets the type to the fallback type. `{ absurd::<Fallback>(panic!()) };`.
/// This is what is known as "never type fallback".
WaffleLapkin marked this conversation as resolved.
Show resolved Hide resolved
///
/// Historically fallback was [`()`], causing confusing behavior where `!` spontaneously coerced
WaffleLapkin marked this conversation as resolved.
Show resolved Hide resolved
/// to `()`, even though `()` was never mentioned (because of the fallback). There are plans to
WaffleLapkin marked this conversation as resolved.
Show resolved Hide resolved
/// change it in 2024 edition (and possibly in all editions on a later date), see
WaffleLapkin marked this conversation as resolved.
Show resolved Hide resolved
/// [Tracking Issue for making `!` fall back to `!`][fallback-ti].
///
/// [`()`]: prim@unit
/// [fallback-ti]: https://github.com/rust-lang/rust/issues/123748
///
#[unstable(feature = "never_type", issue = "35121")]
mod prim_never {}

Expand Down