Fix compilation on MSVC

* cast scalar value instead of function
* use `rb_pid_t` for the portability

1 files changed, 4 insertions, 9 deletions

diff --git a/spec/ruby/optional/capi/ext/thread_spec.c b/spec/ruby/optional/capi/ext/thread_spec.c
index 845c022fa1..11379bbf2f 100644
--- a/spec/ruby/optional/capi/ext/thread_spec.c
+++ b/spec/ruby/optional/capi/ext/thread_spec.c
@@ -73,20 +73,15 @@ static void* blocking_gvl_func_for_udf_io(void *data) {
   }
 }
 
-#if defined(__GNUC__) || defined(__clang__)
-#pragma GCC diagnostic push
-#pragma GCC diagnostic ignored "-Wcast-function-type"
-#endif
+static void* ubf_pid(void *_) {
+  return (void *) (SIGNED_VALUE) getpid();
+}
 
 static VALUE thread_spec_rb_thread_call_without_gvl_native_function(VALUE self) {
-  pid_t ret = (pid_t) (long) rb_thread_call_without_gvl((void *(*)(void *)) getpid, 0, RUBY_UBF_IO, 0);
+  rb_pid_t ret = (rb_pid_t) (SIGNED_VALUE) rb_thread_call_without_gvl(ubf_pid, 0, RUBY_UBF_IO, 0);
   return LONG2FIX(ret);
 }
 
-#if defined(__GNUC__) || defined(__clang__)
-#pragma GCC diagnostic pop
-#endif
-
 /* Returns true if the thread is interrupted. */
 static VALUE thread_spec_rb_thread_call_without_gvl_with_ubf_io(VALUE self) {
   int fds[2];