rage-iodine 4.0.0 → 4.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 26936e113c659cee25877d312515e6fe50ca5ed56ff5e8195e6f5274292f2145
-  data.tar.gz: 40b7aa9c791a5bc08513e5e0c478c02274a39c81d9be0fab7736f8478157ffab
+  metadata.gz: 4262f702902096226a678e94c8b73acf61798fb5441c8cfbb04a831cbea7a738
+  data.tar.gz: fdede223e570118defae54723fe52744cc01548e1c0c71d303dfc4760448d76a
 SHA512:
-  metadata.gz: 1a789b92abed8d78b010ace73cb41d1688ff50c4fc6b590b6705a717196c212721039e252461ba8834a8f0bcd43679fc9c611a9f4d4690a826f3ac212bb3b9e5
-  data.tar.gz: 446a4ecf05a3b371266ea71a92cb514d9a510d2c0fb6dc72e212ff26253e60c343bfc1675a68e6f3c9985c267b9940e61492706f4e61d89e6cb73dbba6c05a2c
+  metadata.gz: 2cf0070ab89370aa02970d80d920ac5b498172e5dfc2ef78b460a1d2d131886ef033a16f96107b3510f4b1652622e133942dcd16d438adead001962bcba54111
+  data.tar.gz: 85a2937799e604dbd0079668b5432bd785b3550d7e3e5dd94bae41f0b0c7b58ed68c036aad6cbcb79dadaddfb6534a31c0f0b34e34bfb5773c8dbfab84a0656d

data/CHANGELOG.md

@@ -6,6 +6,10 @@ Please notice that this change log contains changes for upcoming releases as wel
 
 ## Changes:
 
+#### Change log v.4.1.0 (2025-02-07)
+
+**Update**: Correctly handle `0` timeouts.
+
 #### Change log v.4.0.0 (2024-09-11)
 
 **Update**: Stop disconnecting from the DB on fork.

data/ext/iodine/scheduler.c

@@ -17,6 +17,7 @@ static uint8_t ATTACH_ON_READ_READY_CALLBACK;
 static uint8_t ATTACH_ON_WRITE_READY_CALLBACK;
 static VALUE e_timeout_args[1];
 static VALUE e_closed_args[1];
+static VALUE e_not_ready_args[1];
 
 /* *****************************************************************************
 Fiber Scheduler API
@@ -58,6 +59,25 @@ static void iodine_scheduler_task_perform(intptr_t uuid, fio_protocol_s *fio_pro
   (void)uuid;
 }
 
+static void iodine_scheduler_task_not_ready(void *uuid, void *fio_protocol) {
+  scheduler_protocol_s *protocol = (scheduler_protocol_s *)fio_protocol;
+
+  if (!protocol->fulfilled) {
+    IodineCaller.call2(protocol->block, call_id, 1, e_not_ready_args);
+    protocol->fulfilled = 1;
+  }
+
+  (void)uuid;
+}
+
+static void iodine_scheduler_task_deferred_not_ready(intptr_t uuid, fio_protocol_s *fio_protocol) {
+  // if there's a read event, `fio_defer` will give it time to fulfill the protocol first;
+  // otherwise, the fiber will be resumed with `false` indicating the IO is not ready
+  fio_defer(iodine_scheduler_task_not_ready, (void *)uuid, (void *)fio_protocol);
+
+  (void)uuid;
+}
+
 static void iodine_scheduler_task_timeout(intptr_t uuid, fio_protocol_s *fio_protocol) {
   scheduler_protocol_s *protocol = (scheduler_protocol_s *)fio_protocol;
 
@@ -74,8 +94,11 @@ static VALUE iodine_scheduler_attach(VALUE self, VALUE r_fd, VALUE r_waittype, V
   Check_Type(r_waittype, T_FIXNUM);
   size_t waittype = FIX2UINT(r_waittype);
 
-  Check_Type(r_timeout, T_FIXNUM);
-  size_t timeout = FIX2UINT(r_timeout);
+  size_t timeout;
+  if (r_timeout != Qnil) {
+    Check_Type(r_timeout, T_FIXNUM);
+    timeout = FIX2UINT(r_timeout);
+  }
 
   fio_set_non_block(fd);
 
@@ -112,8 +135,14 @@ static VALUE iodine_scheduler_attach(VALUE self, VALUE r_fd, VALUE r_waittype, V
   }
 
   intptr_t uuid = fio_fd2uuid(fd);
-  if (timeout) {
+
+  if (r_timeout == Qnil) {
+    fio_timeout_set(uuid, 0);
+  } else if (timeout) {
     fio_timeout_set(uuid, timeout);
+  } else {
+    // timeout was explicitly set to 0 - return right away
+    protocol->p.on_ready = iodine_scheduler_task_deferred_not_ready;
   }
 
   fio_watch(uuid, (fio_protocol_s *)protocol);
@@ -185,6 +214,7 @@ void iodine_scheduler_initialize(void) {
   call_id = rb_intern2("call", 4);
   e_timeout_args[0] = INT2NUM(-ETIMEDOUT);
   e_closed_args[0] = INT2NUM(-EIO);
+  e_not_ready_args[0] = Qfalse;
 
   VALUE SchedulerModule = rb_define_module_under(IodineModule, "Scheduler");
 

data/lib/iodine/version.rb

@@ -1,3 +1,3 @@
 module Iodine
-  VERSION = '4.0.0'.freeze
+  VERSION = '4.1.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rage-iodine
 version: !ruby/object:Gem::Version
-  version: 4.0.0
+  version: 4.1.0
 platform: ruby
 authors:
 - Boaz Segev
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-09-11 00:00:00.000000000 Z
+date: 2025-02-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake