Skip to content

Conversation

@nobu
Copy link
Member

@nobu nobu commented May 4, 2024

No description provided.

VALUE rb_fiddle_ptr_new_wrap(void *ptr, long size, rb_fiddle_freefunc_t func, VALUE wrap0, VALUE wrap1);

enum {
FIDDLE_DEFAULT_TYPED_DATA_FALGS = (
Copy link
Contributor

Choose a reason for hiding this comment

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

s/FALGS/FLAGS

Copy link
Member

Choose a reason for hiding this comment

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

Good catch.
I've fixed it.

@kou kou merged commit 91d0ea9 into ruby:master Oct 11, 2024
47 checks passed
matzbot pushed a commit to ruby/ruby that referenced this pull request Oct 16, 2024
kou added a commit that referenced this pull request Nov 9, 2024
GitHub: fix GH-159

It's used by FFI test. So Symbol may be used by other use cases.

GH-139 introduced the "Function name is String" limitation. This
commit removed the limitation.

Reported by Mamoru TASAKA. Thanks!!!
matzbot pushed a commit to ruby/ruby that referenced this pull request Nov 11, 2024
GitHub: fix ruby/fiddle#159

It's used by FFI test. So Symbol may be used by other use cases.

ruby/fiddle#139 introduced the "Function name is String" limitation. This
commit removed the limitation.

Reported by Mamoru TASAKA. Thanks!!!

ruby/fiddle@cea30fe5f9
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