Skip to content

Commit

Permalink
rustdoc: inherited vis. for struct variant fields
Browse files Browse the repository at this point in the history
Teach rustdoc that struct variant fields have inherited visibility.

Fix #19048
  • Loading branch information
tomjakubowski committed Nov 26, 2014
1 parent 1a44875 commit c4da418
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/librustdoc/passes.rs
Original file line number Diff line number Diff line change
Expand Up @@ -202,6 +202,10 @@ impl<'a> fold::DocFolder for Stripper<'a> {
// implementations of traits are always public.
clean::ImplItem(ref imp) if imp.trait_.is_some() => true,

// Struct variant fields have inherited visibility
clean::VariantItem(clean::Variant {
kind: clean::StructVariant(..)
}) => true,
_ => false,
};

Expand Down

9 comments on commit c4da418

@bors
Copy link
Contributor

@bors bors commented on c4da418 Nov 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from sfackler
at tomjakubowski@c4da418

@bors
Copy link
Contributor

@bors bors commented on c4da418 Nov 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging tomjakubowski/rust/rustdoc-struct-variant-vis = c4da418 into auto

@bors
Copy link
Contributor

@bors bors commented on c4da418 Nov 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tomjakubowski/rust/rustdoc-struct-variant-vis = c4da418 merged ok, testing candidate = b40b199

@bors
Copy link
Contributor

@bors bors commented on c4da418 Nov 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on c4da418 Nov 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from sfackler
at tomjakubowski@c4da418

@bors
Copy link
Contributor

@bors bors commented on c4da418 Nov 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging tomjakubowski/rust/rustdoc-struct-variant-vis = c4da418 into auto

@bors
Copy link
Contributor

@bors bors commented on c4da418 Nov 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tomjakubowski/rust/rustdoc-struct-variant-vis = c4da418 merged ok, testing candidate = 3dd5443

@bors
Copy link
Contributor

@bors bors commented on c4da418 Nov 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on c4da418 Nov 27, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 3dd5443

Please sign in to comment.