Skip to content

Introduce enum rb_io_mode. #7894

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

Merged
merged 1 commit into from
Apr 16, 2025
Merged

Introduce enum rb_io_mode. #7894

merged 1 commit into from
Apr 16, 2025

Conversation

ioquatix
Copy link
Member

@ioquatix ioquatix commented Jun 2, 2023

It would be nice if auto-complete worked correctly, and we got a little more type-checking from the compiler.

  • Upstream changes to ruby/stringio first.

@ioquatix ioquatix marked this pull request as draft June 2, 2023 06:29
@nobu
Copy link
Member

nobu commented Jun 2, 2023

Why need to add enum?
If it is a typedef, it would work just by adding a fallback definition.

#ifndef HAVE_TYPE_RB_IO_MODE
typedef int rb_io_mode;
#endif

Or

have_type("rb_io_mode") or $defs << "-Drb_io_mode=int"

@ioquatix
Copy link
Member Author

ioquatix commented Jun 2, 2023

Ah yes, that's a great idea, thanks.

@ioquatix ioquatix force-pushed the enum-io-mode branch 2 times, most recently from 35414a5 to 4b48e24 Compare April 15, 2025 07:01
Copy link

launchable-app bot commented Apr 15, 2025

All Tests passed!

✖️no tests failed ✔️61933 tests passed(1 flake)

@ioquatix ioquatix marked this pull request as ready for review April 15, 2025 07:17
@ioquatix ioquatix requested review from nobu and Copilot April 15, 2025 08:55
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

@ioquatix
Copy link
Member Author

Upstream changes to ruby/stringio: ruby/stringio#129

kou pushed a commit to ruby/stringio that referenced this pull request Apr 15, 2025
matzbot pushed a commit that referenced this pull request Apr 15, 2025
@ioquatix ioquatix force-pushed the enum-io-mode branch 2 times, most recently from 68a7153 to a8903db Compare April 15, 2025 13:13
@ioquatix ioquatix enabled auto-merge (squash) April 15, 2025 13:15
@ioquatix ioquatix merged commit 8d21f66 into ruby:master Apr 16, 2025
75 checks passed
@ioquatix ioquatix deleted the enum-io-mode branch April 16, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants