Skip to content

Rollup of PRs in the queue; Sunday - #23593

Merged
bors merged 16 commits into
rust-lang:masterfrom
Manishearth:rollup
Mar 23, 2015
Merged

Rollup of PRs in the queue; Sunday#23593
bors merged 16 commits into
rust-lang:masterfrom
Manishearth:rollup

Conversation

@Manishearth

Copy link
Copy Markdown
Member

(yay, no Saturday)

Ms2gerand others added 8 commits March 20, 2015 13:29
This allows github to render it with formatting.
This commit removes the `IndexMut` impls on `HashMap` and `BTreeMap`, in
order to future-proof the API against the eventual inclusion of an
`IndexSet` trait.
Ideally, we would eventually be able to support:
```rust
map[owned_key] = val;
map[borrowed_key].mutating_method(arguments);
&mut map[borrowed_key];
```
but to keep the design space as unconstrained as possible, we do not
currently want to support `IndexMut`, in case some other strategy will
eventually be needed.
Code currently using mutating index notation can use `get_mut` instead.
[breaking-change]
Closesrust-lang#23448
Rustbook already does this.
…eatures.
Namely:
* `box_syntax`
* `box_patterns`
* `simd_ffi`
* `macro_reexport`
cc rust-lang#22820
Boolean values and small aggregates have a different type in
args/allocas than in SSA values but the intrinsics for volatile and
atomic ops were missing the necessary casts to handle that.
Fixesrust-lang#23550
@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors: r+ p=20

@bors

bors commented Mar 21, 2015

Copy link
Copy Markdown
Collaborator

📌 Commit e7d77cf has been approved by Manishearth

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors: force

@bors

bors commented Mar 21, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit e7d77cf with merge 1d1f2c0...

@bors

bors commented Mar 21, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-mac-64-nopt-t

@dotdash

Copy link
Copy Markdown
Contributor

@bors retry

@bors

bors commented Mar 21, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit e7d77cf with merge 1c15687...

@bors

bors commented Mar 21, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-mac-64-nopt-t

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors: retry force

@bors

bors commented Mar 22, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit e7d77cf with merge 28a5342...

@bors

bors commented Mar 22, 2015

Copy link
Copy Markdown
Collaborator

⛄ The build was interrupted to prioritize another pull request.

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors: retry force

@bors

bors commented Mar 22, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit e7d77cf with merge bd0881a...

bors added a commit that referenced this pull request Mar 22, 2015
@bors

bors commented Mar 22, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-mac-32-opt

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors: retry

@bors

bors commented Mar 22, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit e7d77cf with merge 34ead18...

@bors

bors commented Mar 22, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-mac-64-nopt-t

…nkro
This commit removes the `IndexMut` impls on `HashMap` and `BTreeMap`, in
order to future-proof the API against the eventual inclusion of an
`IndexSet` trait.
Ideally, we would eventually be able to support:
```rust
map[owned_key] = val;
map[borrowed_key].mutating_method(arguments);
&mut map[borrowed_key];
```
but to keep the design space as unconstrained as possible, we do not
currently want to support `IndexMut`, in case some other strategy will
eventually be needed.
Code currently using mutating index notation can use `get_mut` instead.
[breaking-change]
Closesrust-lang#23448
r? @gankro
 Boolean values and small aggregates have a different type in args/allocas than
in SSA values but the intrinsics for volatile and atomic ops were
missing the necessary casts to handle that.
Fixesrust-lang#23550
…s1, r=alexcrichton
...ures.
Namely:
* `box_syntax`
* `box_patterns`
* `simd_ffi`
* `macro_reexport`
cc rust-lang#22820
bors added a commit that referenced this pull request Mar 23, 2015
@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit b4e9106 with merge 809a554...

@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-mac-64-nopt-t

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors: retry force

@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-win-64-nopt-t

@barosl

Copy link
Copy Markdown
Contributor

Wow, this rollup is very, very unfortunate!

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors: retry

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@barosl Most of this is just because buildbot died over the weekend.

@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-win-64-nopt-t

@Manishearth

Copy link
Copy Markdown
MemberAuthor

@bors: retry force

@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-win-32-nopt-t

@alexcrichton

Copy link
Copy Markdown
Member

@bors: retry

@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

1 similar comment
@bors

bors commented Mar 23, 2015

Copy link
Copy Markdown
Collaborator

@bors
bors merged commit b4e9106 into rust-lang:masterMar 23, 2015
@CentrilCentril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollupA PR which is a rollup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@Manishearth@bors@dotdash@alexcrichton@barosl@Centril@Ms2ger@steveklabnik@aturon@fhahn