Skip to content

Rollup of 7 pull requests #59964

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
Closed
wants to merge 22 commits into from
Closed

Rollup of 7 pull requests #59964

wants to merge 22 commits into from

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Apr 14, 2019

Successful merges:

Failed merges:

r? @ghost

Albin Stjerna and others added 22 commits April 10, 2019 22:44
update polonius-engine
c440c0a
@estebank
Remove duplicated redundant spans
fd4c6e9
@czipperz
Remove [mut] syntax in pin docs
94a5c3b
@czipperz
Make BufWriter use get_mut instead of manipulating inner in Write imp…
ac5e755 
…lementation

get_mut allows us to abstract over the implementation detail of inner being
optional.
@czipperz
Add comment that field projectin also works with mutable fields
e9c9d1c
@estebank
Add test and fix dedup
ca5a9ce
@czipperz
Expand note on mutable references
b754b8f
@Zoxc
Use a proc macro to declare preallocated symbols
37d325e
@Zoxc
Make check_name generic
d1fcdd9
@Zoxc
Ensure the symbols are pure strings
a55de31
@Zoxc
Move modules outside the proc macro
fe4a390
@Zoxc
Use colon for keyword defs
d66975c
@ljedrz @Zoxc
HirIdify hir::Def
3b99a48
@Zoxc
Update tests
b5d43fc
@krk
Add missing backtick to Symbol documentation.
8f3fd85
@Centril
Rollup merge of rust-lang#59655 - Zoxc:symbols, r=petrochenkov
31d5364 
Use a proc macro to declare preallocated symbols

r? @petrochenkov
@Centril
Rollup merge of rust-lang#59856 - albins:update-polonius, r=nikomatsakis
cee4567 
update polonius-engine

This updates polonius-engine to [version 0.7.0](https://github.com/rust-lang/polonius/blob/master/RELEASES.md#v070), which adds a hybrid algorithm that starts off with performing a cheaper, location-insensitive analysis before proceeding with the full analysis.

r? @nikomatsakis
@Centril
Rollup merge of rust-lang#59877 - Zoxc:hiridify_def_id, r=eddyb
6c5777e 
HirIdify hir::Def

cc @ljedrz
r? @oli-obk
@Centril
Rollup merge of rust-lang#59896 - estebank:dedup-spans, r=davidtwco
443925c 
Remove duplicated redundant spans

Fix rust-lang#59895.
@Centril
Rollup merge of rust-lang#59900 - czipperz:remove-bracket-mut-syntax-…
c25ae85 
…in-pin-docs, r=RalfJung

Remove [mut] syntax in pin docs

Resolves rust-lang#59832
@Centril
Rollup merge of rust-lang#59906 - czipperz:bufwriter-use-getmut, r=ke…
f6dec88 
…nnytm

Make BufWriter use get_mut instead of manipulating inner in Write implementation

`get_mut` allows us to abstract over the implementation detail of inner being optional.
@Centril
Rollup merge of rust-lang#59957 - krk:fix-doctypo-symbol, r=jonas-sch…
02b02cf 
…ievink

Add missing backtick to Symbol documentation.
@Centril
Copy link
Contributor Author

Centril commented Apr 14, 2019

@bors r+ p=7

@bors
Copy link
Collaborator

bors commented Apr 14, 2019

📌 Commit 02b02cf has been approved by Centril

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 14, 2019
@Centril Centril closed this Apr 14, 2019
@Centril Centril deleted the rollup-fupil6y branch April 14, 2019 15:27
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
rollup A PR which is a rollup S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Centril @bors @estebank @czipperz @Zoxc @ljedrz @krk