Confirm that resource usage is still 0 before retiring. (#17)

socketry · Apr 22, 2024 · ed9050c · ed9050c
1 parent ee098ef
commit ed9050c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/.mailmap b/.mailmap
@@ -0,0 +1 @@
+Thomas Morgan <[email protected]>
diff --git a/lib/async/pool/controller.rb b/lib/async/pool/controller.rb
@@ -169,10 +169,12 @@ def prune(retain = 0)
 
 				# It's okay for this to context switch:
 				unused.each do |resource|
-					if block_given?
-						yield resource
-					else
-						retire(resource)
+					if usage = @resources[resource] and usage.zero?
+						if block_given?
+							yield resource
+						else
+							retire(resource)
+						end
 					end
 
 					break if @resources.size <= retain