-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Conversation
Why need to add #ifndef HAVE_TYPE_RB_IO_MODE
typedef int rb_io_mode;
#endif Or have_type("rb_io_mode") or $defs << "-Drb_io_mode=int" |
Ah yes, that's a great idea, thanks. |
35414a5
to
4b48e24
Compare
✅ All Tests passed!✖️no tests failed ✔️61933 tests passed(1 flake) |
There was a problem hiding this 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.
Upstream changes to |
Companion PR to <ruby/ruby#7894>.
(ruby/stringio#129) Companion PR to <#7894>. ruby/stringio@2d3988e12f
68a7153
to
a8903db
Compare
It would be nice if auto-complete worked correctly, and we got a little more type-checking from the compiler.
ruby/stringio
first.