io-event 1.14.0 → 1.14.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 53a9942daf957580fe332adea530a3bba659c0ec5405560078233ead0b893d11
-  data.tar.gz: d6eea90381836e4f1bf62430ae95ea0f7f2b1693683d2a114b5193f0c7a1d3f8
+  metadata.gz: 5db83dbc5cd7f63abe723a5ddd79821592d47090ece9c0c940f539550ec3b84e
+  data.tar.gz: 9e15a94bd293c8af581e58d5178ffaace83aa52c93b5aaf5a2e2aca34e4e136d
 SHA512:
-  metadata.gz: 9bfd04d670380038ec6a57a212f0f927b3fd748f93ac95c26e04adb1ed829a999fc02ebca56a27812c84be402b9fef576ba5d53be0df7b8f5e06bb516c70f2b6
-  data.tar.gz: ca71eac94412be0bbb953d32eb60ef1654fe24535012bc3f6f9a1a5375377175a71641c28e5ab92a5f3d44418ab50c3ef177a552cc0abe57f633e718cb2b4bc4
+  metadata.gz: 219307268126c7d27129722c34e1b1f5572683f6cdad99425449573045b9591de3b926bc4bd510208584e149b1e94a8cc15cd5c5739d3d141e8d55ef12964a83
+  data.tar.gz: 07d411e20aca99e29344bb113da4f849d5093ee3ed6b6785460bc4b6fe2d7ee298b4c0e9147b6cffce919088c7367c5298c6e1d5704ca795dc98af9e22740f32

data/ext/io/event/worker_pool.c

@@ -86,6 +86,19 @@ static void worker_pool_free(void *ptr) {
 	}
 }
 
+static void worker_pool_mark(void *ptr)
+{
+	struct IO_Event_WorkerPool *pool = (struct IO_Event_WorkerPool *)ptr;
+	struct IO_Event_WorkerPool_Worker *worker = pool->workers;
+	while (worker) {
+		struct IO_Event_WorkerPool_Worker *next = worker->next;
+		// We need to mark the thread even though its marked through the VM's ractors because we call `join`
+		// on them after their completion. They could be freed by then.
+		rb_gc_mark(worker->thread);  // thread objects are currently pinned anyway
+		worker = next;
+	}
+}
+
 // Size functions for Ruby GC
 static size_t worker_pool_size(const void *ptr) {
 	return sizeof(struct IO_Event_WorkerPool);
@@ -94,7 +107,7 @@ static size_t worker_pool_size(const void *ptr) {
 // Ruby TypedData structures
 static const rb_data_type_t IO_Event_WorkerPool_type = {
 	"IO::Event::WorkerPool",
-	{0, worker_pool_free, worker_pool_size,},
+	{worker_pool_mark, worker_pool_free, worker_pool_size,},
 	0, 0, RUBY_TYPED_FREE_IMMEDIATELY
 };
 
@@ -199,22 +212,20 @@ static VALUE worker_thread_func(void *_worker) {
 }
 
 // Create a new worker thread
-static int create_worker_thread(struct IO_Event_WorkerPool *pool) {
-	if (pool->current_worker_count >= pool->maximum_worker_count) {
-		return -1;
-	}
-	
-		struct IO_Event_WorkerPool_Worker *worker = malloc(sizeof(struct IO_Event_WorkerPool_Worker));
-	if (!worker) {
-		return -1;
-	}
-	
-	worker->pool = pool;
-	worker->interrupted = false;
-	worker->current_blocking_operation = NULL;
-	worker->next = pool->workers;
-	
-	worker->thread = rb_thread_create(worker_thread_func, worker);
+static int create_worker_thread(VALUE self, struct IO_Event_WorkerPool *pool) {
+if (pool->current_worker_count >= pool->maximum_worker_count) {
+return -1;
+}
+
+	struct IO_Event_WorkerPool_Worker *worker = malloc(sizeof(struct IO_Event_WorkerPool_Worker));
+if (!worker) {
+return -1;
+}
+@@ -214,7 +227,7 @@ static int create_worker_thread(struct IO_Event_WorkerPool *pool) {
+worker->current_blocking_operation = NULL;
+worker->next = pool->workers;
+
+	RB_OBJ_WRITE(self, &worker->thread, rb_thread_create(worker_thread_func, worker));
 	if (NIL_P(worker->thread)) {
 		free(worker);
 		return -1;
@@ -273,7 +284,7 @@ static VALUE worker_pool_initialize(int argc, VALUE *argv, VALUE self) {
 	
 	// Create initial workers
 	for (size_t i = 0; i < maximum_worker_count; i++) {
-		if (create_worker_thread(pool) != 0) {
+		if (create_worker_thread(self, pool) != 0) {
 			// Just set the maximum_worker_count for debugging, don't fail completely
 			// worker_pool_free(pool);
 			// rb_raise(rb_eRuntimeError, "Failed to create workers");

data/lib/io/event/version.rb

@@ -7,6 +7,6 @@
 class IO
 	# @namespace
 	module Event
-		VERSION = "1.14.0"
+		VERSION = "1.14.1"
 	end
 end

data/releases.md

@@ -22,7 +22,7 @@ heap.concat([5, 2, 8, 1, 9, 3])
 removed = heap.delete(5)  # Returns 5, heap maintains order
 
 # Bulk removal with condition
-count = heap.delete_if { |x| x.even? }  # Removes 2, 8 efficiently
+count = heap.delete_if{|x| x.even?}  # Removes 2, 8 efficiently
 ```
 
 The `delete_if` and `concat` methods are particularly efficient for bulk operations, using bottom-up heapification to maintain the heap property in O(n) time. This provides significant performance improvements:

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: io-event
 version: !ruby/object:Gem::Version
-  version: 1.14.0
+  version: 1.14.1
 platform: ruby
 authors:
 - Samuel Williams
@@ -116,7 +116,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.7.0.dev
+rubygems_version: 3.6.9
 specification_version: 4
 summary: An event loop.
 test_files: []