Skip to content

Extend entry API to work on borrowed keys. #1769

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

Closed
cristicbz wants to merge 13 commits into from
Closed

Extend entry API to work on borrowed keys. #1769

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
74391a7
add entry-into-owned rfc
cristicbz Oct 12, 2016
bf03d12
fix example in summary
cristicbz Oct 12, 2016
6952692
fix a couple typos
cristicbz Oct 12, 2016
af7dc42
add note on insta-stable and IntoOwned usefulness
cristicbz Oct 12, 2016
9e35d5e
change to AsBorrowRef version
cristicbz Oct 13, 2016
10f8b13
fix some phrasing here and there
cristicbz Oct 13, 2016
c5d9df9
add bit about deref coercions
cristicbz Oct 14, 2016
1e329b6
update with knowledge from crater
cristicbz Oct 14, 2016
eb63ff8
fix rust highlighting in deref block
cristicbz Oct 14, 2016
8f41677
order alternatives correctly with code
cristicbz Oct 14, 2016
a4b35b7
introduce another option of getting rid of B
cristicbz Oct 14, 2016
12ba17a
fix a duplicated paragraph
cristicbz Oct 15, 2016
0d027ef
change to Query and add explicit impl alternative
cristicbz Feb 28, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix rust highlighting in deref block
  • Loading branch information
@cristicbz
cristicbz committed Oct 14, 2016
commit eb63ff84a0e0925802fe65d50b8be989ef855690
4 changes: 2 additions & 2 deletions text/0000-entry-into-owned.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -143,12 +143,12 @@ pub fn entry<'a, Q, B>(&'a self, k: Q) -> Entry<'a, K, V, Q>
}
```

### Deref coercions and backwards compatibility.
### Deref coercions and backwards compatibility

An unexpected backwards compatibility hazard comes from deref coercions.
Consider:

```
```rust
fn increment<'a>(map: &mut HashMap<&'a str, u32>, key: &'a String) {
*map.entry(key).or_insert(0) += 1;
}
Expand Down