Skip to content

Conversation

@JOE1994
Copy link
Contributor

@JOE1994 JOE1994 commented Apr 17, 2020

  1. Fixed doc comment of struct 'rustc_middle::mir::Location'
    Currently, the general explanation of the struct appears at the field
    explanation section. I moved and changed the doc comments slightly, so
    that the general explanation would appear in the proper location in
    docs.
  2. Added doc comment explaining 'rustc_mir::util::pretty::write_mir_fn'
    Unlike other counterparts, brief explanation for this function was missing,
    so I added one.

Thank you for reviewing this PR :)

1. Fixed doc comment of struct 'rustc_middle::mir::Location'
Currently, the general explanation of the struct appears at the field
explanation section. I moved and changed the doc comments slightly, so
that the general explanation would appear in the proper location in
docs.
2. Added doc comment explaining 'rustc_mir::util::pretty::write_mir_fn'
Unlike other counterparts, brief explanation for this function was missing,
so I added one.

Thank you for reviewing this PR :)
@rust-highfive
Copy link
Contributor

r? @ecstatic-morse

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 17, 2020
@jonas-schievink
Copy link
Contributor

Thanks!

@bors r+ rollup
r? @jonas-schievink

@bors
Copy link
Collaborator

bors commented Apr 17, 2020

📌 Commit 5847c58 has been approved by jonas-schievink

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 17, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 17, 2020
Rollup of 5 pull requests

Successful merges:

 - rust-lang#69642 (Use query to determine whether function needs const checking)
 - rust-lang#71239 (Rename `asm` test directory in favor of `llvm_asm`)
 - rust-lang#71246 (Implement `Clone` for `liballoc::collections::linked_list::Cursor`.)
 - rust-lang#71247 (Remove unnecessary variable intialization)
 - rust-lang#71254 (Minor fix and addition to doc comments)

Failed merges:

r? @ghost
@bors bors merged commit 4132642 into rust-lang:master Apr 17, 2020
@JOE1994 JOE1994 deleted the add_comment branch April 18, 2020 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants