async 1.28.3 → 1.28.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 77c63d6ea5723e686a2b72bd7bac1055aa293df4f4583915124a657cd39e6a74
-  data.tar.gz: 3f58014dc1430f1a03bc16a69f19020a8fc3aefb4280e9026fea74fe89118ee1
+  metadata.gz: 0bfd6d656f9d5f891985de61ab428bafe452148e85cdc9d3ff43deec90e17f5e
+  data.tar.gz: 4c402e7fb08a04356ae96672598e5c196774bdc0c95462d6f32620796977cd64
 SHA512:
-  metadata.gz: ca3d285761dd73820c81a56e3bc55a11780e915652d07f611be3723934345f0204d0949b0b5f3a86480b231cdbf37515404eb38152d1771b564932798cb96e70
-  data.tar.gz: fa3cc87e0064c5a63c0f3b3683270c0e1ad9a2db283c131b139bbfd3c337399d8646824377c512b8e2245bc70f6ba213afda6e84391c4caaa4b8764c6587955d
+  metadata.gz: 96b328f71cd892be4d9ecb677911054dad649b2e1115d5c6a92aa6c616295e66c969d964b3127797c9209f38cdb44962e364e4d8a0462e14d5da13bf51b1c4a5
+  data.tar.gz: 919265a3a3c5a8a1e512548860fb8a37641016c0a25f7322330f2560f651a1991144c632ef12c8e87410810f8fe2e42df67a9a5a7c98965e37f414ad515baddb

data/lib/async/node.rb

@@ -24,6 +24,7 @@ module Async
 	# A double linked list.
 	class List
 		def initialize
+			# The list behaves like a list node, so @tail points to the next item (the first one) and head points to the previous item (the last one). This may be slightly confusing but it makes the interface more natural.
 			@head = nil
 			@tail = nil
 			@size = 0
@@ -36,21 +37,21 @@ module Async
 		
 		# Inserts an item at the end of the list.
 		def insert(item)
-			unless @head
-				@head = item
+			unless @tail
 				@tail = item
+				@head = item
 				
 				# Consistency:
 				item.head = nil
 				item.tail = nil
 			else
-				@tail.tail = item
-				item.head = @tail
+				@head.tail = item
+				item.head = @head
 				
 				# Consistency:
 				item.tail = nil
 				
-				@tail = item
+				@head = item
 			end
 			
 			@size += 1
@@ -59,14 +60,14 @@ module Async
 		end
 		
 		def delete(item)
-			if @head.equal?(item)
-				@head = @head.tail
+			if @tail.equal?(item)
+				@tail = @tail.tail
 			else
 				item.head.tail = item.tail
 			end
 			
-			if @tail.equal?(item)
-				@tail = @tail.head
+			if @head.equal?(item)
+				@head = @head.head
 			else
 				item.tail.head = item.head
 			end
@@ -79,15 +80,17 @@ module Async
 			return self
 		end
 		
-		def each
+		def each(&block)
 			return to_enum unless block_given?
 			
-			item = @head
-			while item
-				# We store the tail pointer so we can remove the current item from the linked list:
-				tail = item.tail
-				yield item
-				item = tail
+			current = self
+			while node = current.tail
+				yield node
+				
+				# If the node has deleted itself or any subsequent node, it will no longer be the next node, so don't use it for continued traversal:
+				if current.tail.equal?(node)
+					current = node
+				end
 			end
 		end
 		
@@ -100,19 +103,19 @@ module Async
 		end
 		
 		def first
-			@head
+			@tail
 		end
 		
 		def last
-			@tail
+			@head
 		end
 		
 		def empty?
-			@head.nil?
+			@tail.nil?
 		end
 		
 		def nil?
-			@head.nil?
+			@tail.nil?
 		end
 	end
 	
@@ -276,6 +279,8 @@ module Async
 						if child.finished?
 							delete_child(child)
 						else
+							# In theory we don't need to do this... because we are throwing away the list. However, if you don't correctly update the list when moving the child to the parent, it foobars the enumeration, and subsequent nodes will be skipped, or in the worst case you might start enumerating the parents nodes.
+							delete_child(child)
 							parent.add_child(child)
 						end
 					end

data/lib/async/reactor.rb

@@ -102,7 +102,7 @@ module Async
 			fiber = Fiber.current
 			
 			if timeout
-				timer = self.after(timeout) do
+				timer = @timers.after(timeout) do
 					if fiber.alive?
 						fiber.resume(false)
 					end
@@ -111,7 +111,7 @@ module Async
 			
 			begin
 				@blocked += 1
-				Fiber.yield
+				Task.yield
 			ensure
 				@blocked -= 1
 			end
@@ -198,7 +198,7 @@ module Async
 		def yield(fiber = Fiber.current)
 			@ready << fiber
 			
-			Fiber.yield
+			Task.yield
 		end
 		
 		def finished?
@@ -223,7 +223,7 @@ module Async
 				@running.clear
 			end
 			
-			unless @blocked.zero?
+			if @unblocked.any?
 				unblocked = Array.new
 				
 				@guard.synchronize do
@@ -347,7 +347,7 @@ module Async
 			timer = @timers.after(timeout) do
 				if fiber.alive?
 					error = exception.new("execution expired")
-					fiber.resume error
+					fiber.resume(error)
 				end
 			end
 			

data/lib/async/scheduler.rb

@@ -34,24 +34,20 @@ module Async
 		
 		def initialize(reactor)
 			@reactor = reactor
-			@wrappers = nil
 		end
 		
+		attr :wrappers
+		
 		def set!
-			@wrappers = {}
 			Fiber.set_scheduler(self)
 		end
 		
 		def clear!
-			# Because these instances are created with `autoclose: false`, this does not close the underlying file descriptor:
-			# @ios&.each_value(&:close)
-			
-			@wrappers = nil
 			Fiber.set_scheduler(nil)
 		end
 		
 		private def from_io(io)
-			@wrappers[io] ||= Wrapper.new(io, @reactor)
+			Wrapper.new(io, @reactor)
 		end
 		
 		def io_wait(io, events, timeout = nil)
@@ -82,12 +78,12 @@ module Async
 		# @returns [Process::Status] A process status instance.
 		def process_wait(pid, flags)
 			Thread.new do
-				Process::Status.wait(pid, flags)
+				::Process::Status.wait(pid, flags)
 			end.value
 		end
 		
 		def kernel_sleep(duration)
-			@reactor.sleep(duration)
+			self.block(nil, duration)
 		end
 		
 		def block(blocker, timeout)

data/lib/async/task.rb

@@ -72,14 +72,14 @@ module Async
 		# Create a new task.
 		# @param reactor [Async::Reactor] the reactor this task will run within.
 		# @param parent [Async::Task] the parent task.
-		def initialize(reactor, parent = Task.current?, logger: nil, **options, &block)
+		def initialize(reactor, parent = Task.current?, logger: nil, finished: nil, **options, &block)
 			super(parent || reactor, **options)
 			
 			@reactor = reactor
 			
 			@status = :initialized
 			@result = nil
-			@finished = nil
+			@finished = finished
 			
 			@logger = logger
 			

data/lib/async/version.rb

@@ -21,5 +21,5 @@
 # THE SOFTWARE.
 
 module Async
-	VERSION = "1.28.3"
+	VERSION = "1.28.8"
 end

data/lib/kernel/sync.rb

@@ -28,7 +28,10 @@ module Kernel
 		if task = ::Async::Task.current?
 			yield task
 		else
-			::Async::Reactor.run(&block).wait
+			::Async::Reactor.run(
+				finished: ::Async::Condition.new,
+				&block
+			).wait
 		end
 	end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: async
 version: !ruby/object:Gem::Version
-  version: 1.28.3
+  version: 1.28.8
 platform: ruby
 authors:
 - Samuel Williams
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-01-02 00:00:00.000000000 Z
+date: 2021-02-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: console