Skip to content

Fix rb_interned_str: create strings with BINARY (akak ASCII_8BIT) encoding#15888

Merged
byroot merged 1 commit intoruby:masterfrom
byroot:rb-interned-str-binary
Jan 16, 2026
Merged

Fix rb_interned_str: create strings with BINARY (akak ASCII_8BIT) encoding#15888
byroot merged 1 commit intoruby:masterfrom
byroot:rb-interned-str-binary

Conversation

@byroot
Copy link
Copy Markdown
Member

@byroot byroot commented Jan 16, 2026

[Bug #21842]

The documentation always stated as much, and it's consistent with the rb_str_* family of functions.

@byroot byroot enabled auto-merge (rebase) January 16, 2026 20:45
@jhawthorn
Copy link
Copy Markdown
Member

Did you intend to update all the internal references to ASCII? We're also using this for encoding names via rb_interned_str_cstr

@byroot byroot disabled auto-merge January 16, 2026 21:04
@byroot
Copy link
Copy Markdown
Member Author

byroot commented Jan 16, 2026

I did yes, but I only checked for rb_interned_str, thanks for the heads up, I'll update these too.

@byroot byroot force-pushed the rb-interned-str-binary branch from cbd31d7 to ff60414 Compare January 16, 2026 21:12
…oding

[Bug #21842]

The documentation always stated as much, and it's consistent with the
rb_str_* family of functions.
@byroot byroot force-pushed the rb-interned-str-binary branch from ff60414 to a54b1e6 Compare January 16, 2026 21:14
@byroot byroot enabled auto-merge (rebase) January 16, 2026 21:20
@byroot byroot merged commit 1f3c52d into ruby:master Jan 16, 2026
89 checks passed
@byroot byroot deleted the rb-interned-str-binary branch January 16, 2026 22:06
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.

3 participants