Skip to content

Make char::is_default_ignorable unstably public - #157944

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
Jules-Bertholet:is-default-ignorable
Aug 9, 2026
Merged

Make char::is_default_ignorable unstably public#157944
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
Jules-Bertholet:is-default-ignorable

Conversation

@Jules-Bertholet

@Jules-Bertholet Jules-Bertholet commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Similar to #154849; this function is used in the implementation of char::escape_debug(), and there's no reason to force crates to duplicate a data table that std has to ship anyway.

I'll add a tracking issue if this is approved.

(I'll do the other data tables from #155527 in separate PRs.)

@rustbot label A-Unicode T-libs-api

Tracking issue: #160583

r? @Mark-Simulacrum

@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-Unicode Area: Unicode T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. labels Jun 15, 2026
@rust-log-analyzer

This comment has been minimized.

@Jules-Bertholet Jules-Bertholet changed the title Promote char::is_default_ignorable from perma-unstable to unstable Make char::is_default_ignorable public and unstable Jun 15, 2026
@Jules-Bertholet Jules-Bertholet changed the title Make char::is_default_ignorable public and unstable Make char::is_default_ignorable unstably public Jun 15, 2026
@Mark-Simulacrum

Copy link
Copy Markdown
Member

r=me here, I think this one is probably non-controversial so no need for an ACP. Worst case we move it back to private.

@Mark-Simulacrum Mark-Simulacrum added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 21, 2026
jhpratt added a commit to jhpratt/rust that referenced this pull request Jun 24, 2026
…ivate-use, r=Mark-Simulacrum

Make `char::is_private_use` and `char::is_assigned` unstably public

See also rust-lang#157944, rust-lang#154849. These functions are used in the implementation of `char::escape_debug()`, and there's no reason to force crates to duplicate data tables that std has to ship anyway. `is_assigned` is especially useful because crates relying on the stability guarantees of the other Unicode methods in `char` may wish to reject unassigned characters where those guarantees do not hold.

I'll add a tracking issue if this is approved.

The `is_assigned` vs `is_unassigned` bikeshed probably needs libs-API review before merging?

@rustbot label A-Unicode T-libs-api

r? @Mark-Simulacrum
jhpratt added a commit to jhpratt/rust that referenced this pull request Jun 24, 2026
…ivate-use, r=Mark-Simulacrum

Make `char::is_private_use` and `char::is_assigned` unstably public

See also rust-lang#157944, rust-lang#154849. These functions are used in the implementation of `char::escape_debug()`, and there's no reason to force crates to duplicate data tables that std has to ship anyway. `is_assigned` is especially useful because crates relying on the stability guarantees of the other Unicode methods in `char` may wish to reject unassigned characters where those guarantees do not hold.

I'll add a tracking issue if this is approved.

The `is_assigned` vs `is_unassigned` bikeshed probably needs libs-API review before merging?

@rustbot label A-Unicode T-libs-api

r? @Mark-Simulacrum
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jun 24, 2026
…ivate-use, r=Mark-Simulacrum

Make `char::is_private_use` and `char::is_assigned` unstably public

See also rust-lang#157944, rust-lang#154849. These functions are used in the implementation of `char::escape_debug()`, and there's no reason to force crates to duplicate data tables that std has to ship anyway. `is_assigned` is especially useful because crates relying on the stability guarantees of the other Unicode methods in `char` may wish to reject unassigned characters where those guarantees do not hold.

I'll add a tracking issue if this is approved.

The `is_assigned` vs `is_unassigned` bikeshed probably needs libs-API review before merging?

@rustbot label A-Unicode T-libs-api

r? @Mark-Simulacrum
rust-timer added a commit that referenced this pull request Jun 25, 2026
Rollup merge of #157946 - Jules-Bertholet:char-unassigned-private-use, r=Mark-Simulacrum

Make `char::is_private_use` and `char::is_assigned` unstably public

See also #157944, #154849. These functions are used in the implementation of `char::escape_debug()`, and there's no reason to force crates to duplicate data tables that std has to ship anyway. `is_assigned` is especially useful because crates relying on the stability guarantees of the other Unicode methods in `char` may wish to reject unassigned characters where those guarantees do not hold.

I'll add a tracking issue if this is approved.

The `is_assigned` vs `is_unassigned` bikeshed probably needs libs-API review before merging?

@rustbot label A-Unicode T-libs-api

r? @Mark-Simulacrum
pull Bot pushed a commit to xtqqczze/rust-lang-miri that referenced this pull request Jun 26, 2026
…, r=Mark-Simulacrum

Make `char::is_private_use` and `char::is_assigned` unstably public

See also rust-lang/rust#157944, rust-lang/rust#154849. These functions are used in the implementation of `char::escape_debug()`, and there's no reason to force crates to duplicate data tables that std has to ship anyway. `is_assigned` is especially useful because crates relying on the stability guarantees of the other Unicode methods in `char` may wish to reject unassigned characters where those guarantees do not hold.

I'll add a tracking issue if this is approved.

The `is_assigned` vs `is_unassigned` bikeshed probably needs libs-API review before merging?

@rustbot label A-Unicode T-libs-api

r? @Mark-Simulacrum
pull Bot pushed a commit to Kokoro2336/rust-analyzer that referenced this pull request Jun 29, 2026
…, r=Mark-Simulacrum

Make `char::is_private_use` and `char::is_assigned` unstably public

See also rust-lang/rust#157944, rust-lang/rust#154849. These functions are used in the implementation of `char::escape_debug()`, and there's no reason to force crates to duplicate data tables that std has to ship anyway. `is_assigned` is especially useful because crates relying on the stability guarantees of the other Unicode methods in `char` may wish to reject unassigned characters where those guarantees do not hold.

I'll add a tracking issue if this is approved.

The `is_assigned` vs `is_unassigned` bikeshed probably needs libs-API review before merging?

@rustbot label A-Unicode T-libs-api

r? @Mark-Simulacrum
@Jules-Bertholet

Copy link
Copy Markdown
Contributor Author

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 30, 2026

@Mark-Simulacrum Mark-Simulacrum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me with tracking issue added.

View changes since this review

Comment thread library/core/src/char/methods.rs Outdated
@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 3, 2026
@rustbot

rustbot commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@rustbot

rustbot commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

⚠️ Warning ⚠️

@Jules-Bertholet

Copy link
Copy Markdown
Contributor Author

@rustbot ready

@rustbot rustbot removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Aug 5, 2026
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
ghaaj pushed a commit to ghaaj/ungrammar that referenced this pull request Aug 7, 2026
…, r=Mark-Simulacrum

Make `char::is_private_use` and `char::is_assigned` unstably public

See also rust-lang/rust#157944, rust-lang/rust#154849. These functions are used in the implementation of `char::escape_debug()`, and there's no reason to force crates to duplicate data tables that std has to ship anyway. `is_assigned` is especially useful because crates relying on the stability guarantees of the other Unicode methods in `char` may wish to reject unassigned characters where those guarantees do not hold.

I'll add a tracking issue if this is approved.

The `is_assigned` vs `is_unassigned` bikeshed probably needs libs-API review before merging?

@rustbot label A-Unicode T-libs-api

r? @Mark-Simulacrum
@Mark-Simulacrum

Copy link
Copy Markdown
Member

@bors squash

@rust-bors

This comment has been minimized.

* Make `char::is_default_ignorable` unstably public
* Add tracking issue

Co-authored-by: Jules Bertholet <jules@bertholet.email>
@rust-bors

rust-bors Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

🔨 2 commits were squashed into 9846cb6.

@rust-bors
rust-bors Bot force-pushed the is-default-ignorable branch from 16f2d7a to 9846cb6 Compare August 8, 2026 15:08
@Mark-Simulacrum

Copy link
Copy Markdown
Member

@bors r+ rollup

@rust-bors

rust-bors Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 9846cb6 has been approved by Mark-Simulacrum

It is now in the queue for this repository.

@rust-bors 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 8, 2026
rust-bors Bot pushed a commit that referenced this pull request Aug 8, 2026
…uwer

Rollup of 9 pull requests

Successful merges:

 - #156935 (Introduce a `PinSafePointer` trait that generalizes `PinCoerceUnsized`)
 - #159834 (Apply str debugger visualizer to `*const str`, `*mut str` and `Box<str>`)
 - #160663 (Suggest add async for function sig with return expr in body)
 - #160732 (Optimize slice::contains for one-byte BytewiseEq types)
 - #157944 (Make `char::is_default_ignorable` unstably public)
 - #158835 (rustc_passes: lint unused `#[path]` attributes on inline modules)
 - #160754 (Rename more diagnostic files to `diagnostics.rs`)
 - #160755 (Merge `rustc_lint/lints.rs` into `diagnostics.rs`)
 - #160757 (Merge `rustc_attr_parsing/session_diagnostics.rs` into `diagnostics.rs`)
@rust-bors
rust-bors Bot merged commit c31da22 into rust-lang:main Aug 9, 2026
13 checks passed
@rustbot rustbot added this to the 1.99.0 milestone Aug 9, 2026
rust-timer added a commit that referenced this pull request Aug 9, 2026
Rollup merge of #157944 - Jules-Bertholet:is-default-ignorable, r=Mark-Simulacrum

Make `char::is_default_ignorable` unstably public

Similar to #154849; this function is used in the implementation of `char::escape_debug()`, and there's no reason to force crates to duplicate a data table that std has to ship anyway.

I'll add a tracking issue if this is approved.

(I'll do the other data tables from #155527 in separate PRs.)

@rustbot label A-Unicode T-libs-api

Tracking issue: #160583

r? @Mark-Simulacrum
@Jules-Bertholet
Jules-Bertholet deleted the is-default-ignorable branch August 9, 2026 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Unicode Area: Unicode 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. T-libs-api Relevant to the library API 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