Skip to content

Rollup of 6 pull requests - #61290

Closed
Centril wants to merge 32 commits into
rust-lang:masterfrom
Centril:rollup-az1mksn
Closed

Rollup of 6 pull requests#61290
Centril wants to merge 32 commits into
rust-lang:masterfrom
Centril:rollup-az1mksn

Conversation

@Centril

Copy link
Copy Markdown
Contributor

Successful merges:

Failed merges:

r? @ghost

matthewjasperand others added 30 commits May 18, 2019 13:16
Co-Authored-By: Mazdak Farrokhzad <twingoow@gmail.com>
…eddyb
Allow const parameters in array sizes to be unified
Fixesrust-lang#60632.
Fixesrust-lang#60744.
Fixesrust-lang#60923.
(The last commit should probably be viewed in isolation, as it just renames things from `type` to `kind`.)
r? @eddyb
…s, r=nikomatsakis
Add better tests for hidden lifetimes in impl trait
cc rust-lang#60670
tests: Centralize proc macros commonly used for testing
Many proc macros in `ui\proc-macro\auxiliary` were doing same things.
(I added a fair share of those myself.)
Now commonly used macros (empty, identity, etc) are collected in one place - `ui\proc-macro\auxiliary\test-macros.rs`.
…a-discard_buffer, r=nikomatsakis
BufReader: In Seek impl, remove extra discard_buffer call
As far as I can tell, this code does nothing. I'm not sure why it even is there.
Centril added 2 commits May 28, 2019 22:49
…atsakis
Mailmap fixes
I've made a few fixes and additions to the .mailmap file.
- I've added my information to it, as I noticed it was popping up twice at the [Thanks](https://thanks.rust-lang.org/rust/all-time/) website (I've made contributions to the Rust Book).
- I noticed that @carols10cents‘ information was popping up in multiple places at the [Thanks](https://thanks.rust-lang.org/rust/all-time/) website and when running `git shortlog -s -e > out.txt`, and fixed her information
- I alphabetized the list using Atom's [Sort Lines package](https://github.com/atom/sort-lines)
The output from before and after running `git shortlog -s -e > out.txt` for Carol:
<img width="708" alt="Carol Before" src="https://user-images.githubusercontent.com/19867440/58472843-2d8ca880-8115-11e9-9624-d31ed23ae9b5.png">
<img width="523" alt="Carol After" src="https://user-images.githubusercontent.com/19867440/58472844-2d8ca880-8115-11e9-95c6-80b17cc04790.png">
The output from before and after running `git shortlog -s -e > out.txt` for myself:
<img width="489" alt="Joseph After" src="https://user-images.githubusercontent.com/19867440/58472941-6e84bd00-8115-11e9-8266-1d7fcd53d0bc.png">
<img width="421" alt="Joseph Before" src="https://user-images.githubusercontent.com/19867440/58472943-6e84bd00-8115-11e9-9fd2-d7ace7e3d636.png">
mention that MaybeUninit is a bit like Option
@Centril

Copy link
Copy Markdown
ContributorAuthor

@bors r+ rollup=never p=6

@bors

bors commented May 28, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 76b94c5 has been approved by Centril

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label May 28, 2019
@Centril

Copy link
Copy Markdown
ContributorAuthor

@bors r-

@borsbors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels May 28, 2019
@CentrilCentril closed this May 28, 2019
@Centril
Centril deleted the rollup-az1mksn branch May 28, 2019 22:20
@CentrilCentril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollupA PR which is a rollupS-waiting-on-authorStatus: This is awaiting some action (such as code changes or more information) from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@Centril@bors@matthewjasper@petrochenkov@czipperz@JosephTLyons@RalfJung@varkor