Skip to content

collections: move hashmap's example to the struct. - #12864

Merged
bors merged 1 commit into
rust-lang:masterfrom
huonw:hash-docs
Mar 14, 2014
Merged

collections: move hashmap's example to the struct.#12864
bors merged 1 commit into
rust-lang:masterfrom
huonw:hash-docs

Conversation

@huonw

@huonw huonw commented Mar 13, 2014

Copy link
Copy Markdown
Contributor

collections: move hashmap's example to the struct.

Most people go straight to the struct, not looking at the module, so the
example was well hidden.

Most people go straight to the struct, not looking at the module, so the
example was well hidden.
@flaper87

Copy link
Copy Markdown
Contributor

LGTM, I've seen a couple of other places where this could be done too. I'll keep an eye on that and help organizing docstrings.

bors added a commit that referenced this pull request Mar 14, 2014
collections: move hashmap's example to the struct.

Most people go straight to the struct, not looking at the module, so the
example was well hidden.
@bors bors closed this Mar 14, 2014
@bors
bors merged commit d062669 into rust-lang:master Mar 14, 2014
@huonw
huonw deleted the hash-docs branch June 27, 2014 06:48
fasterthanlime pushed a commit to fasterthanlime/rust that referenced this pull request Jul 26, 2022
feat: Resolve derive helper attributes in IDE layer

![image](https://user-images.githubusercontent.com/3757771/180647125-98cae13b-a7b4-446b-8a64-fae014f982e2.png)

Also downmaps tokens inside deriver helpers to the derive expansion
![image](https://user-images.githubusercontent.com/3757771/180642110-699bdc73-e1f3-4692-94f2-21544f824ab9.png)

This does not handle the case where multiple derives make use of the same helper name though, unsure how to tackle that yet.

Partially addresses rust-lang/rust-analyzer#10935
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jun 13, 2024
ignore array from `deref_addrof` lint

Split from rust-lang/rust-clippy#12854

changelog: ignore array from `deref_addrof` lint

r? y21
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.

4 participants