Skip to content

rustc_public: Make fields that shouldn't be exposed visible only in rustc_public #152922

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.

Sign up for GitHub

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

Merged
rust-bors merged 1 commit into from
Feb 25, 2026
+3 −12
Merged

rustc_public: Make fields that shouldn't be exposed visible only in rustc_public #152922

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 3 additions & 12 deletions compiler/rustc_public/src/ty.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -862,17 +862,11 @@ pub struct Discr {
#[derive(Clone, Copy, Debug, PartialEq, Eq, Hash, Serialize)]
pub struct VariantDef {
/// The variant index.
///
/// ## Warning
/// Do not access this field directly!
pub idx: VariantIdx,
pub(crate) idx: VariantIdx,
/// The data type where this variant comes from.
/// For now, we use this to retrieve information about the variant itself so we don't need to
/// cache more information.
///
/// ## Warning
/// Do not access this field directly!
pub adt_def: AdtDef,
pub(crate) adt_def: AdtDef,
}

impl VariantDef {
Expand All @@ -894,10 +888,7 @@ impl VariantDef {
#[derive(Clone, Debug, Eq, PartialEq, Serialize)]
pub struct FieldDef {
/// The field definition.
///
/// ## Warning
/// Do not access this field directly! This is public for the compiler to have access to it.
pub def: DefId,
pub(crate) def: DefId,

/// The field name.
pub name: Symbol,
Expand Down
Loading