std: reduce visibility of some internal OsStr related types by pacak · Pull Request #161589 · rust-lang/rust

pacak · 2026-08-23T11:38:19Z

The std::sys::os_str::{Buf, Slice} types are only used within the std crate and not actually exported. Whole sys module is private. They don't need to be public. This might result in a better generated code, but more importantly it avoids some compile errors down the line.

This commit is extracted from #160971

The std::sys::os_str::{Buf, Slice} types are only used within the std crate and not actually exported. Whole `sys` module is private. They don't need to be public. This might result in a better generated code, but more importantly it avoids some compile errors down the line.

rustbot · 2026-08-23T11:38:24Z

r? @JohnTitor

rustbot has assigned @JohnTitor.
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: @ChrisDenton, libs
@ChrisDenton, libs expanded to 13 candidates
Random selection from ChrisDenton, JohnTitor, Mark-Simulacrum, clarfonthey, nia-e

clarfonthey · 2026-08-23T13:13:49Z

I thought we had unreachable_pub firing for all of the standard library? That would catch things like this.

pacak · 2026-08-23T13:24:37Z

I thought we had unreachable_pub firing for all of the standard library? That would catch things like this.

% jj diff --tool :git
diff --git a/library/std/src/lib.rs b/library/std/src/lib.rs
index 980ec4416f..670c1e8c0a 100644
--- a/library/std/src/lib.rs
+++ b/library/std/src/lib.rs
@@ -241,6 +241,7 @@
 // Lints:
 #![warn(deprecated_in_future)]
 #![warn(missing_docs)]
+#![warn(unreachable_pub)]
 #![warn(missing_debug_implementations)]
 #![allow(explicit_outlives_requirements)]
 #![allow(unused_lifetimes)]

error: `std` (lib) generated 868 warnings (run `cargo fix --lib -p std` to apply 865 suggestions)

pacak · 2026-08-23T13:26:56Z

I can make a separate pull request for all that stuff I guess. Want to keep this focused on #160971 related stuff.

clarfonthey · 2026-08-23T14:24:42Z

Fair enough. I'll just merge this then.

@bors r+ rollup

rust-bors · 2026-08-23T14:24:45Z

📌 Commit 00be974 has been approved by clarfonthey

It is now in the queue for this repository.

std: reduce visibility of some internal OsStr related types The std::sys::os_str::{Buf, Slice} types are only used within the std crate and not actually exported. Whole `sys` module is private. They don't need to be public. This might result in a better generated code, but more importantly it avoids some compile errors down the line. This commit is extracted from rust-lang#160971

…uwer Rollup of 14 pull requests Successful merges: - #160302 (target_features: sse (or at least avx2) is incompatible with soft-float ABI) - #161187 (add `Complex<T>` layout tests for straightforward targets) - #161524 (Put back `tests/rustdoc-gui/search-result-display.goml`) - #161586 (miri subtree update) - #155254 (Recover on attribute in use tree) - #158695 (Replace `CrateDump` with the Debug impl from `CStore`) - #160452 (-Ctarget-feature is not unsafe (any more)) - #161007 (tests/ui/union/union-nodrop.rs: fix typo "expressios") - #161166 (add crashtests [5/N], remove unused aux files ) - #161569 (Bump cfg_aliases to 0.2.2) - #161573 (re-bless `pretty-std` on windows) - #161588 (update eyre) - #161589 (std: reduce visibility of some internal OsStr related types) - #161598 ([Bootstrap] Pass exact CI `llvm-config` executable path)

std: reduce visibility of some internal OsStr related types The std::sys::os_str::{Buf, Slice} types are only used within the std crate and not actually exported. Whole `sys` module is private. They don't need to be public. This might result in a better generated code, but more importantly it avoids some compile errors down the line. This commit is extracted from rust-lang#160971

…uwer Rollup of 17 pull requests Successful merges: - #161187 (add `Complex<T>` layout tests for straightforward targets) - #161524 (Put back `tests/rustdoc-gui/search-result-display.goml`) - #161592 (core: refactor tests/pattern.rs tests) - #161602 (Fix flakyness issue for `tests/rustdoc-gui/headers-color.goml`) - #155254 (Recover on attribute in use tree) - #158695 (Replace `CrateDump` with the Debug impl from `CStore`) - #159456 (Fix the reporting of layout normalization failures in transmute checks) - #160452 (-Ctarget-feature is not unsafe (any more)) - #161007 (tests/ui/union/union-nodrop.rs: fix typo "expressios") - #161166 (add crashtests [5/N], remove unused aux files ) - #161569 (Bump cfg_aliases to 0.2.2) - #161573 (re-bless `pretty-std` on windows) - #161588 (update eyre) - #161589 (std: reduce visibility of some internal OsStr related types) - #161598 ([Bootstrap] Pass exact CI `llvm-config` executable path) - #161601 (Path: use optimized is_empty() method) - #161604 (coretests: Add a few tests for backward multibyte predicate)

std: reduce visibility of some internal OsStr related types The std::sys::os_str::{Buf, Slice} types are only used within the std crate and not actually exported. Whole `sys` module is private. They don't need to be public. This might result in a better generated code, but more importantly it avoids some compile errors down the line. This commit is extracted from rust-lang#160971

Rollup of 24 pull requests Successful merges: - #161586 (miri subtree update) - #161187 (add `Complex<T>` layout tests for straightforward targets) - #161524 (Put back `tests/rustdoc-gui/search-result-display.goml`) - #161592 (core: refactor tests/pattern.rs tests) - #161602 (Fix flakyness issue for `tests/rustdoc-gui/headers-color.goml`) - #152433 (Use `symlink_dir` to create junctions on Windows instead of trying to use symbolic links in `copy_link_internal`) - #155254 (Recover on attribute in use tree) - #158695 (Replace `CrateDump` with the Debug impl from `CStore`) - #159456 (Fix the reporting of layout normalization failures in transmute checks) - #159940 (Eliminate some buggy `unreachable!()`s in `expand_[option_]env()`) - #160452 (-Ctarget-feature is not unsafe (any more)) - #161007 (tests/ui/union/union-nodrop.rs: fix typo "expressios") - #161166 (add crashtests [5/N], remove unused aux files ) - #161294 (add crashtests [6/N]) - #161552 (also trigger overflow FCW when going from overflow -> error) - #161569 (Bump cfg_aliases to 0.2.2) - #161573 (re-bless `pretty-std` on windows) - #161580 (Add codegen test for redundant bit mask after a range check) - #161588 (update eyre) - #161589 (std: reduce visibility of some internal OsStr related types) - #161598 ([Bootstrap] Pass exact CI `llvm-config` executable path) - #161601 (Path: use optimized is_empty() method) - #161604 (coretests: Add a few tests for backward multibyte predicate) - #161615 (interpret: make validate_c_variadic_compatible_ty public)

Rollup of 22 pull requests Successful merges: - #161586 (miri subtree update) - #161187 (add `Complex<T>` layout tests for straightforward targets) - #161524 (Put back `tests/rustdoc-gui/search-result-display.goml`) - #161592 (core: refactor tests/pattern.rs tests) - #161602 (Fix flakyness issue for `tests/rustdoc-gui/headers-color.goml`) - #152433 (Use `symlink_dir` to create junctions on Windows instead of trying to use symbolic links in `copy_link_internal`) - #155254 (Recover on attribute in use tree) - #158695 (Replace `CrateDump` with the Debug impl from `CStore`) - #159456 (Fix the reporting of layout normalization failures in transmute checks) - #159940 (Eliminate some buggy `unreachable!()`s in `expand_[option_]env()`) - #160452 (-Ctarget-feature is not unsafe (any more)) - #161007 (tests/ui/union/union-nodrop.rs: fix typo "expressios") - #161552 (also trigger overflow FCW when going from overflow -> error) - #161569 (Bump cfg_aliases to 0.2.2) - #161573 (re-bless `pretty-std` on windows) - #161580 (Add codegen test for redundant bit mask after a range check) - #161588 (update eyre) - #161589 (std: reduce visibility of some internal OsStr related types) - #161598 ([Bootstrap] Pass exact CI `llvm-config` executable path) - #161601 (Path: use optimized is_empty() method) - #161604 (coretests: Add a few tests for backward multibyte predicate) - #161615 (interpret: make validate_c_variadic_compatible_ty public)

Rollup merge of #161589 - pacak:private-osstr, r=clarfonthey std: reduce visibility of some internal OsStr related types The std::sys::os_str::{Buf, Slice} types are only used within the std crate and not actually exported. Whole `sys` module is private. They don't need to be public. This might result in a better generated code, but more importantly it avoids some compile errors down the line. This commit is extracted from #160971

rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 23, 2026

rustbot assigned JohnTitor Aug 23, 2026

rustbot added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label Aug 23, 2026

rust-bors Bot 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 Aug 23, 2026

clarfonthey assigned clarfonthey and unassigned JohnTitor Aug 23, 2026

JonathanBrouwer mentioned this pull request Aug 23, 2026

Rollup of 14 pull requests #161605

Closed

This was referenced Aug 23, 2026

Update Pattern machinery to use more generic version of string like things #161608

Open

And finally we can implement Pattern machinery for OsStr #161610

Open

JonathanBrouwer mentioned this pull request Aug 23, 2026

Rollup of 17 pull requests #161618

Closed

Zalathar mentioned this pull request Aug 24, 2026

Rollup of 24 pull requests #161636

Closed

Zalathar mentioned this pull request Aug 24, 2026

Rollup of 22 pull requests #161638

Merged

rust-bors Bot merged commit a0dd6f5 into rust-lang:main Aug 24, 2026
13 checks passed

rustbot added this to the 1.100.0 milestone Aug 24, 2026

Navigation Menu

Uh oh!

Uh oh!

std: reduce visibility of some internal OsStr related types - #161589

std: reduce visibility of some internal OsStr related types#161589
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
pacak:private-osstr

pacak commented Aug 23, 2026

Uh oh!

rustbot commented Aug 23, 2026

Uh oh!

clarfonthey commented Aug 23, 2026

Uh oh!

pacak commented Aug 23, 2026

Uh oh!

pacak commented Aug 23, 2026

Uh oh!

clarfonthey commented Aug 23, 2026

Uh oh!

rust-bors Bot commented Aug 23, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Uh oh!

Conversation

pacak commented Aug 23, 2026

Uh oh!

rustbot commented Aug 23, 2026

Uh oh!

clarfonthey commented Aug 23, 2026

Uh oh!

pacak commented Aug 23, 2026

Uh oh!

pacak commented Aug 23, 2026

Uh oh!

clarfonthey commented Aug 23, 2026

Uh oh!

rust-bors Bot commented Aug 23, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants