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 rustc_middle::mir::Field #94025

Closed
pierwill opened this issue Feb 15, 2022 · 5 comments
Closed

Document rustc_middle::mir::Field #94025

pierwill opened this issue Feb 15, 2022 · 5 comments
Assignees
Labels
E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@pierwill
Copy link
Member

pierwill commented Feb 15, 2022

This type, defined here, lacks documentation. Documentation might want to answer questions like: What does this type represent? What are its uses?

@rustbot label +T-Compiler +E-Easy

@rustbot rustbot added E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 15, 2022
@MakitaToki
Copy link
Contributor

I recently read this section, can I try it?

@pierwill
Copy link
Member Author

Of course! You can self-assign an issue using @rustbot: https://rustc-dev-guide.rust-lang.org/rustbot.html#issue-claiming

@MakitaToki
Copy link
Contributor

Thanks. I'll do my best.@rustbot claim

@pierwill
Copy link
Member Author

pierwill commented Feb 17, 2022

@Mizobrook-kan: This comment might be helpful: #94006 (comment).

Field represents the position of a data field inside a datatype.

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Feb 19, 2022
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Feb 19, 2022
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Feb 19, 2022
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Feb 19, 2022
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Feb 19, 2022
@compiler-errors
Copy link
Member

Should we close this now that #94113 is landed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

4 participants