Skip to content

make Send/Sync impl of RawTable manual#21768

Merged
bors merged 1 commit into
rust-lang:masterfrom
Gankra:hash-send
Feb 1, 2015
Merged

make Send/Sync impl of RawTable manual#21768
bors merged 1 commit into
rust-lang:masterfrom
Gankra:hash-send

Conversation

@Gankra

@Gankra Gankra commented Jan 30, 2015

Copy link
Copy Markdown
Contributor

Fixes #21763

(untested)

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@Gankra Gankra force-pushed the hash-send branch 3 times, most recently from 6297550 to b4618c9 Compare January 30, 2015 04:25
@alexcrichton

Copy link
Copy Markdown
Member

r=me with a new compile-fail regression test

@Gankra

Gankra commented Jan 31, 2015

Copy link
Copy Markdown
Contributor Author

@alexcrichton Test added but I'm new to compile-fail so I'm not sure if this is exactly the desired format or what.

@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+ 343143d

Looks good to me, thanks!

@Manishearth

Copy link
Copy Markdown
Member

Needs rebase

@Gankra

Gankra commented Jan 31, 2015

Copy link
Copy Markdown
Contributor Author

@bors r=alexcrichton 9985991

@bors

bors commented Feb 1, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 9985991 with merge da2b4af...

@bors

bors commented Feb 1, 2015

Copy link
Copy Markdown
Collaborator

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

@Gankra

Gankra commented Feb 1, 2015

Copy link
Copy Markdown
Contributor Author

@bors retry

Seems like a spurious OOM?

@bors

bors commented Feb 1, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 9985991 with merge a2aee62...

bors added a commit that referenced this pull request Feb 1, 2015
@bors

bors commented Feb 1, 2015

Copy link
Copy Markdown
Collaborator

@bors bors merged commit 9985991 into rust-lang:master Feb 1, 2015
lnicola pushed a commit to lnicola/rust that referenced this pull request Mar 23, 2026
…-qualifier

fix: add ident_pat qualifier to fully fn param
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HashMap<K,V> is always Send/Sync

6 participants