Skip to content
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.

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

Document behavior of <dyn Any as Any>::type_id() #118028

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

Jules-Bertholet
Copy link
Contributor

@Jules-Bertholet Jules-Bertholet commented Nov 18, 2023

See also #57893

@rustbot label A-docs T-libs

@rustbot
Copy link
Collaborator

rustbot commented Nov 18, 2023

r? @thomcc

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

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools labels Nov 18, 2023
@thomcc
Copy link
Member

thomcc commented Nov 24, 2023

Hmm, I think this has the possibility of confusing more people than it helps, since it's completely possible to end up with it returning the type id of dyn Any unintentionally.

I suppose this at least lets them know it's an issue with their code rather than the expected behavior, though.

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Nov 24, 2023

📌 Commit db62921 has been approved by thomcc

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 24, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 24, 2023
…iaskrgr

Rollup of 6 pull requests

Successful merges:

 - rust-lang#116807 (Improve rewind documentation)
 - rust-lang#117656 (Update windows-bindgen and define `INVALID_HANDLE_VALUE` ourselves)
 - rust-lang#117940 (chore: remove unnecessary drop)
 - rust-lang#118028 (Document behavior of `<dyn Any as Any>::type_id()`)
 - rust-lang#118060 (Use an absolute path to the NUL device)
 - rust-lang#118224 (Sort unstable items last in rustdoc, instead of first)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit c3216e2 into rust-lang:master Nov 24, 2023
11 checks passed
@rustbot rustbot added this to the 1.76.0 milestone Nov 24, 2023
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Nov 24, 2023
Rollup merge of rust-lang#118028 - Jules-Bertholet:dyn-any-doc, r=thomcc

Document behavior of `<dyn Any as Any>::type_id()`

See also rust-lang#57893

`@rustbot` label A-docs T-libs
@Jules-Bertholet Jules-Bertholet deleted the dyn-any-doc branch December 4, 2023 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants