polyphony 0.94 → 0.96

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 02f56fa13c8397d86a0ceae20dd253461809397e8ad9eb5827a04650f3dc5020
-  data.tar.gz: f956d6e226e4a2b4239dc367abe9b4c1de89173df1b9968748f0da730b88410d
+  metadata.gz: 2db26acbbe9457dc3b3fcdd1e34d35e5347a2e67a5c126b9ddea672957ca3318
+  data.tar.gz: ac02e27bd659dafbfe0b68db9b22fc3ca8acb744ff82049dbb2bbea999f7e87e
 SHA512:
-  metadata.gz: 262b6272e6f2cdc86c09eca16b60e31713139ef37b7203c77e858d0c991a5754ee842311caac9ee76e09ce71a1f473e18f102f0ce460e903f31acb0317b6a0b5
-  data.tar.gz: d628e38d39ee4bbbeb1f5f9e7b5ba9d2e406911be1cb9af809a310f9cdb0315beb417e41462b68ed58e23e2c94435a5702192fd5946d5c51a9d4ae392ee50872
+  metadata.gz: e99fb459ea41504df54826634d219f8ef1feea3324fe6c56ab18fc46867728d57f36f4c15a35787e0ba42fe588b10d3e849badfcbee0a720c77bb1d8619e8fc3
+  data.tar.gz: d52ee81b1f1f5a5294cecf09837ccce8cc68f0233d73a0d868326f0a144897542f5ad3e05cc398eb4bf963742fe1fac01a096abf0d77e3201d8c50ca8880e65a

data/.github/workflows/test.yml

@@ -1,4 +1,4 @@
-name: Tests
+name: Tests (libev)
 
 on: [push, pull_request]
 
@@ -8,7 +8,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, macos-latest]
-        ruby: ['2.7', '3.0', '3.1', 'head']
+        ruby: ['3.0', '3.1', '3.2']
 
     name: >-
       ${{matrix.os}}, ${{matrix.ruby}}
@@ -20,7 +20,7 @@ jobs:
 
     steps:
     - name: Setup machine
-      uses: actions/checkout@v1
+      uses: actions/checkout@v3
     - name: Setup Ruby
       uses: ruby/setup-ruby@v1
       with:

data/.github/workflows/test_io_uring.yml

@@ -1,4 +1,4 @@
-name: Tests with io_uring
+name: Tests (io_uring)
 
 on: [push, pull_request]
 
@@ -8,7 +8,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest]
-        ruby: ['2.7', '3.0', '3.1', 'head']
+        ruby: ['3.0', '3.1', '3.2']
 
     name: >-
       ${{matrix.os}}, ${{matrix.ruby}}
@@ -16,7 +16,7 @@ jobs:
     runs-on: ${{matrix.os}}
     steps:
     - name: Checkout repository and submodules
-      uses: actions/checkout@v2
+      uses: actions/checkout@v3
       with:
         submodules: recursive
     - name: Setup Ruby
@@ -33,4 +33,4 @@ jobs:
     - name: Compile C-extension
       run: bundle exec rake compile
     - name: Run tests
-      run:  bundle exec ruby test/run.rb --verbose --name test_sleep
+      run:  bundle exec ruby test/run.rb

data/.gitignore

@@ -42,9 +42,9 @@ build-iPhoneSimulator/
 
 # for a library or gem, you might want to ignore these files since the code is
 # intended to run in multiple environments; otherwise, check them in:
-# Gemfile.lock
-# .ruby-version
-# .ruby-gemset
+Gemfile.lock
+.ruby-version
+.ruby-gemset
 
 # unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
 .rvmrc

data/CHANGELOG.md

@@ -1,3 +1,22 @@
+## 0.96 2023-02-11
+ - Rework Redis client adapter
+- Fix working with Pry (#95, thanks @floriandejonckheere)
+- Improve compatibility with Enumerator and other APIs.
+
+## 0.95 2023-01-16
+
+- Remove support for Ruby 2.7
+- Add support for explicit Enumerator (external enumeration) (#93)
+- Fix `Socket#readpartial`
+- Fix `receive_all_pending` docs (#92)
+- Improve linux kernel version detection
+- Update liburing
+- Always reset SQE user data in io_uring backend
+
+## 0.94 2022-10-03
+
+- Fix linux kernel version detection (#89)
+
 ## 0.93 2022-04-05
 
 - Add support for IO::Buffer in Ruby 3.1 (#80)

data/docs/api-reference/fiber.md

@@ -212,7 +212,7 @@ spin { Fiber.current.parent << 'hello from child' }
 message = receive #=> 'hello from child'
 ```
 
-### #receive_pending → [*object]
+### #receive_all_pending → [*object]
 
 Returns all messages currently in the mailbox, emptying the mailbox. This method
 does not block if no the mailbox is already empty. This method may be used to
@@ -225,7 +225,7 @@ worker = spin do
     handle_job(job)
   end
 rescue Polyphony::Terminate => e
-  receive_pending.each { |job| handle_job(job) }
+  receive_all_pending.each { |job| handle_job(job) }
 end
 ```
 

data/docs/api-reference/object.md

@@ -63,9 +63,9 @@ result #=> 'bar'
 
 Shortcut for `Fiber.current.receive`
 
-### #receive_pending → [*object]
+### #receive_all_pending → [*object]
 
-Shortcut for `Fiber.current.receive_pending`
+Shortcut for `Fiber.current.receive_all_pending`
 
 ### #sleep(duration = nil) → fiber
 
@@ -96,4 +96,4 @@ returns. Otherwise, the loop is infinite (unless an exception is raised).
 ```ruby
 # twice a second
 throttled_loop(2) { puts 'hello world' }
-```
+```

data/docs/main-concepts/exception-handling.md

@@ -229,7 +229,7 @@ def do_work
   end
 rescue Polyphony::Terminate
   # We still need to handle any pending request
-  receive_pending.each { handle_req(req) }
+  receive_all_pending.each { handle_req(req) }
 end
 
 # on the main fiber
@@ -288,4 +288,4 @@ loop.
 To ensure proper thread termination, including the termination of all the
 thread's fibers, Polyphony patches the `Thread#kill` and `Thread#raise` methods
 to schedule the thread's main fiber with the corresponding exceptions, thus
-ensuring an orderly termination or exception handling.
+ensuring an orderly termination or exception handling.

data/examples/adapters/redis_blpop.rb

@@ -1,8 +1,9 @@
 # frozen_string_literal: true
 
-require 'bundler/setup'
-require 'polyphony/adapters/redis'
-# require 'redis'
+require 'redis'
+require_relative '../../lib/polyphony/adapters/redis'
+require_relative '../../lib/polyphony'
+
 
 redis = Redis.new(host: ENV['REDISHOST'] || 'localhost')
 

data/examples/adapters/redis_channels.rb

@@ -1,7 +1,9 @@
 # frozen_string_literal: true
 
-require 'bundler/setup'
-require 'polyphony/adapters/redis'
+require 'redis'
+require_relative '../../lib/polyphony/adapters/redis'
+require_relative '../../lib/polyphony'
+require_relative '../../lib/polyphony/core/channel'
 
 class RedisChannel < Polyphony::Channel
   def self.publish_connection
@@ -17,20 +19,24 @@ class RedisChannel < Polyphony::Channel
   def self.start_monitor
     @channels = {}
     @monitor = spin do
+      p start_monitor: 1
       subscribe_connection.subscribe(CHANNEL_MASTER_TOPIC) do |on|
+        p start_monitor: 2
         on.message do |topic, message|
+          p start_monitor: 3, topic: topic, message: message
           message = Marshal.load(message)
-          topic == if CHANNEL_MASTER_TOPIC
-                     handle_master_message(message)
-                   else
-                     handle_channel_message(topic, message)
-                   end
+          if CHANNEL_MASTER_TOPIC
+            handle_master_message(message)
+          else
+            handle_channel_message(topic, message)
+          end
         end
       end
     end
   end
 
   def self.stop_monitor
+    p stop_monitor: 1
     @monitor&.interrupt
   end
 
@@ -111,10 +117,13 @@ end
 
 spin do
   move_on_after(3) do
+    p :move_on_after
     throttled_loop(1) do
+      # p :throttled_loop
       channel << Time.now
     end
   end
+  p :done
   channel.close
   RedisChannel.stop_monitor
 end

data/examples/core/await.rb

@@ -2,7 +2,7 @@
 
 require 'bundler/setup'
 require 'polyphony'
-require 'polyphony/extensions/debug'
+require 'polyphony/core/debug'
 
 Exception.__disable_sanitized_backtrace__ = true
 

data/examples/io/readline.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'bundler/setup'
+require 'polyphony'
+require 'polyphony/adapters/readline'
+
+$counter = 0
+timer = spin do
+  throttled_loop(5) do
+    $counter += 1
+  end
+end
+
+at_exit { timer.stop }
+
+puts 'try typing $counter to see the counter incremented in the background'
+loop do
+  puts eval Readline::readline('> ', true)
+end

data/ext/polyphony/backend_common.c

@@ -60,6 +60,23 @@ inline void conditional_nonblocking_poll(VALUE backend, struct Backend_base *bas
     Backend_poll(backend, Qnil);
 }
 
+// Verifies that the given fiber has the @thread ivar set. If not, sets it to
+// the current thread. This is essential for the correct functioning of fibers
+// not created by Polyphony, such as those used by the Enumerator class, and
+// probably others as well. With a nil thread ivar, a fiber can not be
+// scheduled. This macro is called in backend_base_switch_fiber() and in
+// backend_snooze().
+//
+// In order to refrain from calling rb_thread_current() multiple times, we
+// allow it to be passed to this macro, and if not we call it on the spot.
+#define CHECK_FIBER_THREAD_REF(fiber, current_thread) { \
+  VALUE thread = rb_ivar_get(fiber, ID_ivar_thread); \
+  if (thread == Qnil)  { \
+    thread = (current_thread != Qnil) ? current_thread : rb_thread_current(); \
+    rb_ivar_set(fiber, ID_ivar_thread, thread); \
+  } \
+}
+
 VALUE backend_base_switch_fiber(VALUE backend, struct Backend_base *base) {
   VALUE current_fiber = rb_fiber_current();
   runqueue_entry next;
@@ -71,6 +88,8 @@ VALUE backend_base_switch_fiber(VALUE backend, struct Backend_base *base) {
   if (SHOULD_TRACE(base))
     TRACE(base, 3, SYM_block, current_fiber, CALLER());
 
+  CHECK_FIBER_THREAD_REF(current_fiber, Qnil);
+
   while (1) {
     next = runqueue_shift(&base->runqueue);
     if (next.fiber != Qnil) {
@@ -261,6 +280,9 @@ inline VALUE backend_snooze(struct Backend_base *backend) {
   VALUE ret;
   VALUE fiber = rb_fiber_current();
   VALUE thread = rb_thread_current();
+  
+  CHECK_FIBER_THREAD_REF(fiber, thread);
+  
   Fiber_make_runnable(fiber, Qnil);
   ret = Thread_switch_fiber(thread);
 
@@ -280,7 +302,7 @@ inline void rectify_io_file_pos(rb_io_t *fptr) {
   }
 }
 
-inline double current_time() {
+inline double current_time(void) {
   struct timespec ts;
   double t;
   uint64_t ns;
@@ -292,7 +314,7 @@ inline double current_time() {
   return t / 1e9;
 }
 
-inline uint64_t current_time_ns() {
+inline uint64_t current_time_ns(void) {
   struct timespec ts;
   uint64_t ns;
 
@@ -439,7 +461,7 @@ VALUE Backend_verify_blocking_mode(VALUE self, VALUE io, VALUE blocking) {
   return self;
 }
 
-void backend_setup_stats_symbols() {
+void backend_setup_stats_symbols(void) {
   SYM_runqueue_size       = ID2SYM(rb_intern("runqueue_size"));
   SYM_runqueue_length     = ID2SYM(rb_intern("runqueue_length"));
   SYM_runqueue_max_length = ID2SYM(rb_intern("runqueue_max_length"));

data/ext/polyphony/backend_io_uring.c

@@ -286,6 +286,7 @@ VALUE Backend_wakeup(VALUE self) {
     // NOP which would cause the io_uring_enter syscall to return
     struct io_uring_sqe *sqe = io_uring_backend_get_sqe(backend);
     io_uring_prep_nop(sqe);
+    io_uring_sqe_set_data(sqe, NULL);
     io_uring_backend_immediate_submit(backend);
 
     return Qtrue;
@@ -304,10 +305,7 @@ int io_uring_backend_defer_submit_and_await(
   VALUE switchpoint_result = Qnil;
 
   backend->base.op_count++;
-  if (sqe) {
-    io_uring_sqe_set_data(sqe, ctx);
-    io_uring_sqe_set_flags(sqe, IOSQE_ASYNC);
-  }
+  if (sqe) io_uring_sqe_set_data(sqe, ctx);
   io_uring_backend_defer_submit(backend);
 
   switchpoint_result = backend_await((struct Backend_base *)backend);
@@ -318,7 +316,8 @@ int io_uring_backend_defer_submit_and_await(
     // op was not completed (an exception was raised), so we need to cancel it
     ctx->result = -ECANCELED;
     sqe = io_uring_backend_get_sqe(backend);
-    io_uring_prep_cancel(sqe, (__u64)ctx, 0);
+    io_uring_prep_cancel(sqe, ctx, 0);
+    io_uring_sqe_set_data(sqe, NULL);
     io_uring_backend_immediate_submit(backend);
   }
 
@@ -931,7 +930,6 @@ static inline op_context_t *prepare_double_splice_ctx(Backend_t *backend, int sr
   struct io_uring_sqe *sqe = io_uring_backend_get_sqe(backend);
   io_uring_prep_splice(sqe, src_fd, -1, dest_fd, -1, DOUBLE_SPLICE_MAXLEN, 0);
   io_uring_sqe_set_data(sqe, ctx);
-  io_uring_sqe_set_flags(sqe, IOSQE_ASYNC);
   backend->base.op_count += 1;
   backend->pending_sqes += 1;
 
@@ -941,7 +939,8 @@ static inline op_context_t *prepare_double_splice_ctx(Backend_t *backend, int sr
 static inline void io_uring_backend_cancel(Backend_t *backend, op_context_t *ctx) {
   struct io_uring_sqe *sqe = io_uring_backend_get_sqe(backend);
   ctx->result = -ECANCELED;
-  io_uring_prep_cancel(sqe, (__u64)ctx, 0);
+  io_uring_prep_cancel(sqe, ctx, 0);
+  io_uring_sqe_set_data(sqe, NULL);
 }
 
 VALUE double_splice_safe(struct double_splice_ctx *ctx) {
@@ -1212,7 +1211,8 @@ VALUE Backend_timeout_ensure(VALUE arg) {
     timeout_ctx->ctx->result = -ECANCELED;
     // op was not completed, so we need to cancel it
     sqe = io_uring_get_sqe(&timeout_ctx->backend->ring);
-    io_uring_prep_cancel(sqe, (__u64)timeout_ctx->ctx, 0);
+    io_uring_prep_cancel(sqe, timeout_ctx->ctx, 0);
+    io_uring_sqe_set_data(sqe, NULL);
     io_uring_backend_immediate_submit(timeout_ctx->backend);
   }
   context_store_release(&timeout_ctx->backend->store, timeout_ctx->ctx);
@@ -1330,7 +1330,8 @@ VALUE Backend_wait_event(VALUE self, VALUE raise) {
     // last fiber to use the eventfd, so we cancel the ongoing poll
     struct io_uring_sqe *sqe;
     sqe = io_uring_backend_get_sqe(backend);
-    io_uring_prep_cancel(sqe, (__u64)backend->event_fd_ctx, 0);
+    io_uring_prep_cancel(sqe, backend->event_fd_ctx, 0);
+    io_uring_sqe_set_data(sqe, NULL);
     io_uring_backend_immediate_submit(backend);
     backend->event_fd_ctx = NULL;
   }
@@ -1429,14 +1430,13 @@ VALUE Backend_chain(int argc,VALUE *argv, VALUE self) {
 
       if (sqe_count) {
         struct io_uring_sqe *sqe;
-        
         io_uring_sqe_set_data(last_sqe, ctx);
-        io_uring_sqe_set_flags(last_sqe, IOSQE_ASYNC);
 
         ctx->ref_count = sqe_count;
         ctx->result = -ECANCELED;
         sqe = io_uring_backend_get_sqe(backend);
-        io_uring_prep_cancel(sqe, (__u64)ctx, 0);
+        io_uring_prep_cancel(sqe, ctx, 0);
+        io_uring_sqe_set_data(sqe, NULL);
         io_uring_backend_immediate_submit(backend);
       }
       else {
@@ -1447,7 +1447,7 @@ VALUE Backend_chain(int argc,VALUE *argv, VALUE self) {
     }
 
     io_uring_sqe_set_data(last_sqe, ctx);
-    flags = (i == (argc - 1)) ? IOSQE_ASYNC : IOSQE_ASYNC | IOSQE_IO_LINK;
+    flags = (i == (argc - 1)) ? 0 : IOSQE_IO_LINK;
     io_uring_sqe_set_flags(last_sqe, flags);
     sqe_count++;
   }
@@ -1466,7 +1466,8 @@ VALUE Backend_chain(int argc,VALUE *argv, VALUE self) {
     // op was not completed (an exception was raised), so we need to cancel it
     ctx->result = -ECANCELED;
     sqe = io_uring_backend_get_sqe(backend);
-    io_uring_prep_cancel(sqe, (__u64)ctx, 0);
+    io_uring_prep_cancel(sqe, ctx, 0);
+    io_uring_sqe_set_data(sqe, NULL);
     io_uring_backend_immediate_submit(backend);
     RAISE_IF_EXCEPTION(resume_value);
     return resume_value;
@@ -1532,7 +1533,8 @@ static inline void splice_chunks_cancel(Backend_t *backend, op_context_t *ctx) {
   
   ctx->result = -ECANCELED;
   sqe = io_uring_backend_get_sqe(backend);
-  io_uring_prep_cancel(sqe, (__u64)ctx, 0);
+  io_uring_prep_cancel(sqe, ctx, 0);
+  io_uring_sqe_set_data(sqe, NULL);
   io_uring_backend_immediate_submit(backend);
 }
 
@@ -1699,7 +1701,7 @@ void Backend_unpark_fiber(VALUE self, VALUE fiber) {
   backend_base_unpark_fiber(&backend->base, fiber);
 }
 
-void Init_Backend() {
+void Init_Backend(void) {
   VALUE cBackend = rb_define_class_under(mPolyphony, "Backend", rb_cObject);
   rb_define_alloc_func(cBackend, Backend_allocate);
 

data/ext/polyphony/backend_libev.c

@@ -107,7 +107,7 @@ void break_async_callback(struct ev_loop *ev_loop, struct ev_async *ev_async, in
   // of a *blocking* event loop (waking it up) in a thread-safe, signal-safe manner
 }
 
-inline struct ev_loop *libev_new_loop() {
+inline struct ev_loop *libev_new_loop(void) {
   #ifdef POLYPHONY_USE_PIDFD_OPEN
     return ev_loop_new(EVFLAG_NOSIGMASK);
   #else
@@ -1474,7 +1474,7 @@ void Backend_unpark_fiber(VALUE self, VALUE fiber) {
   backend_base_unpark_fiber(&backend->base, fiber);
 }
 
-void Init_Backend() {
+void Init_Backend(void) {
   ev_set_allocator(xrealloc);
 
   VALUE cBackend = rb_define_class_under(mPolyphony, "Backend", rb_cObject);

data/ext/polyphony/event.c

@@ -79,7 +79,7 @@ VALUE Event_await(VALUE self) {
   return switchpoint_result;
 }
 
-void Init_Event() {
+void Init_Event(void) {
   cEvent = rb_define_class_under(mPolyphony, "Event", rb_cObject);
   rb_define_alloc_func(cEvent, Event_allocate);
 

data/ext/polyphony/extconf.rb

@@ -15,11 +15,13 @@ def get_config
   raise "Could not parse Linux kernel information (#{kernel_info.inspect})" if !m
 
   version, major_revision, distribution = m[1].to_i, m[2].to_i, m[4]
-  config[:pidfd_open] = (version == 5) && (major_revision >= 3)
+
+  combined_version = version.to_i * 100 + major_revision.to_i
+
+  config[:pidfd_open] = combined_version > 503
 
   force_libev = ENV['POLYPHONY_LIBEV'] != nil
-  config[:io_uring] = !force_libev &&
-    (version == 5) && (major_revision >= 6) && (distribution != 'linuxkit')
+  config[:io_uring] = !force_libev && (combined_version >= 506) && (distribution != 'linuxkit')
   config
 end
 

data/ext/polyphony/fiber.c

@@ -38,24 +38,16 @@ inline VALUE Fiber_auto_watcher(VALUE self) {
   return watcher;
 }
 
-void Fiber_make_runnable(VALUE fiber, VALUE value) {
+inline void Fiber_make_runnable(VALUE fiber, VALUE value) {
   VALUE thread = rb_ivar_get(fiber, ID_ivar_thread);
-  if (thread == Qnil) {
-    rb_raise(rb_eRuntimeError, "No thread set for fiber");
-    // rb_warn("No thread set for fiber");
-    return;
-  }
+  if (thread == Qnil) rb_raise(rb_eRuntimeError, "No thread set for fiber");
 
   Thread_schedule_fiber(thread, fiber, value);
 }
 
-void Fiber_make_runnable_with_priority(VALUE fiber, VALUE value) {
+inline void Fiber_make_runnable_with_priority(VALUE fiber, VALUE value) {
   VALUE thread = rb_ivar_get(fiber, ID_ivar_thread);
-  if (thread == Qnil) {
-    rb_raise(rb_eRuntimeError, "No thread set for fiber");
-    // rb_warn("No thread set for fiber");
-    return;
-  }
+  if (thread == Qnil) rb_raise(rb_eRuntimeError, "No thread set for fiber");
 
   Thread_schedule_fiber_with_priority(thread, fiber, value);
 }
@@ -130,7 +122,7 @@ VALUE Fiber_parked_p(VALUE self) {
   return rb_ivar_get(self, ID_ivar_parked);
 }
 
-void Init_Fiber() {
+void Init_Fiber(void) {
   VALUE cFiber = rb_const_get(rb_cObject, rb_intern("Fiber"));
   rb_define_method(cFiber, "safe_transfer", Fiber_safe_transfer, -1);
   rb_define_method(cFiber, "schedule", Fiber_schedule, -1);

data/ext/polyphony/io_extensions.c

@@ -631,7 +631,7 @@ VALUE IO_http1_splice_chunked(VALUE self, VALUE src, VALUE dest, VALUE maxlen) {
   return self;
 }
 
-void Init_IOExtensions() {
+void Init_IOExtensions(void) {
   rb_define_singleton_method(rb_cIO, "gzip", IO_gzip, -1);
   rb_define_singleton_method(rb_cIO, "gunzip", IO_gunzip, -1);
   rb_define_singleton_method(rb_cIO, "deflate", IO_deflate, 2);

data/ext/polyphony/pipe.c

@@ -96,7 +96,7 @@ VALUE Pipe_fds(VALUE self) {
   return rb_ary_new_from_args(2, INT2FIX(pipe->fds[0]), INT2FIX(pipe->fds[1]));
 }
 
-void Init_Pipe() {
+void Init_Pipe(void) {
   cPipe = rb_define_class_under(mPolyphony, "Pipe", rb_cObject);
   cClosedPipeError = rb_define_class_under(cPipe, "ClosedPipeError", rb_eRuntimeError);
 

data/ext/polyphony/polyphony.c

@@ -170,7 +170,7 @@ VALUE Polyphony_raw_buffer_size(VALUE self, VALUE buffer) {
 //   return Backend_close(BACKEND(), io);
 // }
 
-void Init_Polyphony() {
+void Init_Polyphony(void) {
   mPolyphony = rb_define_module("Polyphony");
 
   // backend methods

data/ext/polyphony/polyphony_ext.c

@@ -15,7 +15,7 @@ void Init_SocketExtensions();
 extern void playground();
 #endif
 
-void Init_polyphony_ext() {
+void Init_polyphony_ext(void) {
   Init_Polyphony();
 
   Init_Backend();

data/ext/polyphony/queue.c

@@ -315,7 +315,7 @@ end:
   return self;
 }
 
-void Init_Queue() {
+void Init_Queue(void) {
   cClosedQueueError = rb_const_get(rb_cObject, rb_intern("ClosedQueueError"));
   cThreadError = rb_const_get(rb_cObject, rb_intern("ThreadError"));
 

data/ext/polyphony/ring_buffer.c

@@ -10,6 +10,7 @@ void ring_buffer_init(ring_buffer *buffer) {
 }
 
 void ring_buffer_free(ring_buffer *buffer) {
+  // printf("ring_buffer_free ring_buffer: %p entries: %p\n", buffer, buffer->entries);
   free(buffer->entries);
 }
 

data/ext/polyphony/socket_extensions.c

@@ -16,7 +16,7 @@ VALUE Socket_double_chevron(VALUE self, VALUE msg) {
   return self;
 }
 
-void Init_SocketExtensions() {
+void Init_SocketExtensions(void) {
   VALUE cSocket;
   VALUE cTCPSocket;
 

data/ext/polyphony/thread.c

@@ -60,7 +60,7 @@ VALUE Thread_class_backend(VALUE _self) {
   return rb_ivar_get(rb_thread_current(), ID_ivar_backend);
 }
 
-void Init_Thread() {
+void Init_Thread(void) {
   rb_define_method(rb_cThread, "setup_fiber_scheduling", Thread_setup_fiber_scheduling, 0);
   rb_define_method(rb_cThread, "schedule_and_wakeup", Thread_fiber_schedule_and_wakeup, 2);
   rb_define_method(rb_cThread, "switch_fiber", Thread_switch_fiber, 0);

data/lib/polyphony/adapters/readline.rb

@@ -7,11 +7,13 @@ require 'readline'
 # thread pool. That way, the reactor loop can keep running while waiting for
 # readline to return
 module ::Readline
-  alias_method :orig_readline, :readline
+  class << self
+    alias_method :orig_readline, :readline
 
-  Worker = Polyphony::ThreadPool.new(1)
+    Worker = Polyphony::ThreadPool.new(1)
 
-  def readline(*args)
-    Worker.process { orig_readline(*args) }
+    def readline(*args)
+      Worker.process { orig_readline(*args) }
+    end
   end
 end