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.

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

Jump to bottom

std::any's documentation should use dyn #56062

Closed
0xrgb opened this issue Nov 19, 2018 · 1 comment · Fixed by #56063
Closed

std::any's documentation should use dyn #56062

0xrgb opened this issue Nov 19, 2018 · 1 comment · Fixed by #56063
Labels
C-bug Category: This is a bug.

Comments

@0xrgb
Copy link
Contributor

0xrgb commented Nov 19, 2018

https://doc.rust-lang.org/std/any/index.html
https://doc.rust-lang.org/std/any/trait.Any.html

Newcomers who learn rust with rust book 2018 edition can be confused, because official book recommends &dyn TraitObject and does not say about &TraitObject.
@Centril Centril added the C-bug Category: This is a bug. label Nov 19, 2018
@Centril
Copy link
Contributor

Centril commented Nov 19, 2018

@0xrgb I think a PR fixing this would be welcome. :)

@0xrgb 0xrgb mentioned this issue Nov 19, 2018
Merged
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this issue Nov 22, 2018
@GuillaumeGomez
Rollup merge of rust-lang#56063 - 0xrgb:patch-1, r=joshtriplett
1646fc9 
Update any.rs documentation using keyword dyn

This will fix rust-lang#56062.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-bug Category: This is a bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update any.rs documentation using keyword dyn 0xrgb/rust
2 participants
@Centril @0xrgb