Skip to content

hir: add more HirId methods #58139

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Feb 5, 2019
Merged

Conversation

ljedrz
Copy link
Contributor

@ljedrz ljedrz commented Feb 4, 2019

Adds a few more methods operating on HirId instead of NodeId with the intention of replacing the old ones in the near future.

r? @Zoxc

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 4, 2019
@ljedrz ljedrz force-pushed the HirIdification_phase_2.5 branch 3 times, most recently from be75f4e to 26797a5 Compare February 4, 2019 10:47
@ljedrz ljedrz force-pushed the HirIdification_phase_2.5 branch from 26797a5 to 927614f Compare February 4, 2019 14:00
@Zoxc
Copy link
Contributor

Zoxc commented Feb 4, 2019

@bors r+

This was referenced Feb 4, 2019
@ljedrz
Copy link
Contributor Author

ljedrz commented Feb 4, 2019

@Zoxc somehing's off; has @bors noticed the approval?

@Zoxc
Copy link
Contributor

Zoxc commented Feb 4, 2019

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Feb 4, 2019

📌 Commit 927614f has been approved by Zoxc

@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 Feb 4, 2019
@ljedrz ljedrz mentioned this pull request Feb 4, 2019
kennytm added a commit to kennytm/rust that referenced this pull request Feb 5, 2019
…Zoxc

hir: add more HirId methods

Adds a few more methods operating on `HirId` instead of `NodeId` with the intention of replacing the old ones in the near future.

r? @Zoxc
bors added a commit that referenced this pull request Feb 5, 2019
Rollup of 23 pull requests

Successful merges:

 - #58001 (proc_macro: make `TokenStream::from_streams` pre-allocate its vector.)
 - #58096 (Transition linkchecker to 2018 edition)
 - #58097 (Transition remote test to Rust 2018)
 - #58106 (libfmt_macros => 2018)
 - #58107 (libgraphviz => 2018)
 - #58108 (Add NVPTX target to a build manifest)
 - #58109 (librustc_privacy => 2018)
 - #58112 (libpanic_abort => 2018)
 - #58113 (Transition build-manifest to 2018 edition)
 - #58114 (Transition tidy and unstable-book-gen to 2018 edition)
 - #58116 (Include the span of attributes of the lhs to the span of the assignment expression)
 - #58117 (Transition rustdoc-theme to 2018 edition)
 - #58128 (libunwind => 2018)
 - #58138 (Fix #58101)
 - #58139 (hir: add more HirId methods)
 - #58141 (Remove weasel word in docs for iter's take_while())
 - #58142 (Remove stray FIXME)
 - #58145 (Add #[must_use] to core::task::Poll)
 - #58162 (Add more debugging code to track down appveyor 259 exit code)
 - #58169 (Update contributor name in .mailmap)
 - #58172 (update split docs)
 - #58182 (SGX target: handle empty user buffers correctly)
 - #58186 (Add Rustlings to the doc index)

Failed merges:

r? @ghost
@bors bors merged commit 927614f into rust-lang:master Feb 5, 2019
@ljedrz ljedrz deleted the HirIdification_phase_2.5 branch February 5, 2019 19:14
# for free to join this conversation on GitHub. Already have an account? # 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.

4 participants