Uh oh!
There was an error while loading. Please reload this page.
Remove unnecessary lamda on emitter map. - #77673
Conversation
rust-highfive
commented
Oct 7, 2020
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @lcnr (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
Uh oh!
There was an error while loading. Please reload this page.
Thanks, do you mind implementing the suggestion by @jonas-schievink? Afterwards this can be merged |
Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
lcnr
commented
Oct 8, 2020
@bors r+ rollup |
bors
commented
Oct 8, 2020
📌 Commit dd60ab3 has been approved by |
Remove unnecessary lamda on emitter map.
…as-schievink Rollup of 8 pull requests Successful merges: - rust-lang#76750 (Don't discourage implementing `core::fmt::Write`) - rust-lang#77449 (BTreeMap: comment why drain_filter's size_hint is somewhat pessimistic) - rust-lang#77660 ((docs): make mutex error comment consistent with codebase) - rust-lang#77663 (Add compile fail test for issue 27675) - rust-lang#77673 (Remove unnecessary lamda on emitter map.) - rust-lang#77701 (Make `max_log_info` easily greppable (for figuring out why debug logging is disabled)) - rust-lang#77702 (Remove not needed lambda.) - rust-lang#77710 (Update submodule llvm to get LVI bugfix) Failed merges: r? `@ghost`
No description provided.