Skip to content

Conversation

@agrare
Copy link
Contributor

@agrare agrare commented Jan 23, 2025

rb_block_call_func_t is of type VALUE (*)(VALUE, VALUE, int, const VALUE *, VALUE)

../../../../../ext/surro-gate/selector_ext.c: In function ‘SurroGate_Selector_each_ready’:
../../../../../ext/surro-gate/selector_ext.c:165:64: error: passing argument 5 of ‘rb_block_call’ from incompatible pointer type [-Wincompatible-pointer-types]
  165 |   return rb_block_call(scoreboard, rb_intern("each"), 0, NULL, scoreboard_iterate, self);
      |                                                                ^~~~~~~~~~~~~~~~~~
      |                                                                |
      |                                                                VALUE (*)(VALUE,  VALUE,  int,  VALUE *) {aka long unsigned int (*)(long unsigned int,  long unsigned int,  int,  long unsigned int *)}
In file included from /usr/include/ruby-3.3.0/ruby/ruby.h:41:
/usr/include/ruby-3.3.0/ruby/internal/iterator.h:324:90: note: expected ‘rb_block_call_func_t’ {aka ‘long unsigned int (*)(long unsigned int,  long unsigned int,  int,  const long unsigned int *, long unsigned int)’} but argument is of type ‘VALUE (*)(VALUE,  VALUE,  int,  VALUE *)’ {aka ‘long unsigned int (*)(long unsigned int,  long unsigned int,  int,  long unsigned int *)’}
  324 | VALUE rb_block_call(VALUE obj, ID mid, int argc, const VALUE *argv, rb_block_call_func_t proc, VALUE data2);
      |                                                                     ~~~~~~~~~~~~~~~~~~~~~^~~~
gmake: *** [Makefile:248: selector_ext.o] Error 1
rake aborted!
Command failed with status (2): [/usr/bin/gmake]

@agrare agrare force-pushed the fix_rb_block_call_incompatible_pointer_type branch from e06d50a to b088329 Compare January 24, 2025 14:43
@agrare agrare closed this Jan 24, 2025
@agrare agrare reopened this Jan 24, 2025
@agrare agrare force-pushed the fix_rb_block_call_incompatible_pointer_type branch from c13b650 to fc5d11d Compare January 24, 2025 21:25
@kbrock
Copy link
Contributor

kbrock commented Feb 21, 2025

Is this change particular to a ruby version?

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.

2 participants