Skip to content

doc: use ptr::addr in offset_from docs - #159122

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
kn1g78:fix/ptr-addr-docs
Jul 11, 2026
Merged

doc: use ptr::addr in offset_from docs#159122
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
kn1g78:fix/ptr-addr-docs

Conversation

@kn1g78

Copy link
Copy Markdown
Contributor

ptr::addr() has been stable since Rust 1.84. Update the three offset_from docs to use it for address-only arithmetic across allocations, and remove the stale FIXME comments.

@rustbotrustbot 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. labels Jul 11, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: libs
  • libs expanded to 12 candidates
  • Random selection from Darksonn, JohnTitor, Mark-Simulacrum, clarfonthey, jhpratt

@hanna-kruppe

Copy link
Copy Markdown
Contributor

Thanks!

r? hanna-kruppe @bors r+ rollup

@rust-bors

rust-borsBot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 05a2bf7 has been approved by hanna-kruppe

It is now in the queue for this repository.

@rust-borsrust-borsBot added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jul 11, 2026
@rust-borsrust-borsBot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 11, 2026
rust-borsBot pushed a commit that referenced this pull request Jul 11, 2026
…uwer
Rollup of 14 pull requests
Successful merges:
- #159126 (Fix PR number in bootstrap's change tracker)
- #155811 (Include AtomicU128/AtomicI128 in docs for any target)
- #156968 (diagnostics: suggest generic_const_args for const ops)
- #159012 (Shrink mir::Statement to 40 bytes)
- #156618 (rustdoc: test ignoring rustc lints in CLI)
- #158182 (std: use `OnceLock` for SGX argument storage)
- #159114 (Add regression test for $-prefixed fragment specifier in repetition)
- #158523 (Fix feature gate for `repr(simd)`)
- #158876 (Fix multiple logic bugs in `Arc::make_mut`)
- #158928 (std: fix Xous UDP send_to length mismatch and truncation)
- #159060 (Look for the cdb architecture that corresponds to the target triple)
- #159089 (riscv: update c-variadic test for LLVM changes)
- #159093 (pretty-print: use inline asm's actual macro name)
- #159122 (doc: use ptr::addr in offset_from docs)
Failed merges:
- #158732 (Apply MCP 1003 and move diagnostics.rs into its own module)
@rust-bors
rust-borsBot merged commit 7e86eec into rust-lang:mainJul 11, 2026
13 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Jul 11, 2026
rust-timer added a commit that referenced this pull request Jul 11, 2026
Rollup merge of #159122 - kn1g78:fix/ptr-addr-docs, r=hanna-kruppe
doc: use ptr::addr in offset_from docs
`ptr::addr()` has been stable since Rust 1.84. Update the three `offset_from` docs to use it for address-only arithmetic across allocations, and remove the stale FIXME comments.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-libsRelevant 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

@kn1g78@rustbot@hanna-kruppe@Mark-Simulacrum