libuv 2.0.3 → 2.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c8d3076693c01bb76953ce8b883c5978bb4169a8
-  data.tar.gz: e5410efdde18367e7b78c3efdcd8f1db8a619e88
+  metadata.gz: dc82e51b4569c6be6d73639176fbb4779a5b4d62
+  data.tar.gz: 711888d208a4f2719efaa20c52d3815645202ed4
 SHA512:
-  metadata.gz: 90c9bee7837dcef7d431ad174f0e096938ec8866261d6fab695146930cbe9e8e503df4ae82ae9e0809c4583cd56cb0dbac8fa36e6565f1edb647820d93028e6d
-  data.tar.gz: 0efd4cb5ae7ce63058fc76ed1e2a00ce842f2643df776b38a171aa1ecad3cbfefbb3b4bc40c7c1fe68946423065e5e9e0ca58af96b4e7ad91ef045d9040d70f9
+  metadata.gz: a6a084029d33260ce7671932c32be0ec450d5c0c9f01e3ea991b2e685b73c89912fe724cec39d4b4e81cd7665eb0d017aab9e3de0c2c547177ab9a23d76e809a
+  data.tar.gz: 8a77e498c41b77c819f4eeff6baee7048e053b3ff36a9373f14b7e62c743dba92ccc3d40f609389e1cb8ee8d0a1ecd6e7e004b5ee7a4eb18a779f44e3cba290a

data/lib/libuv/mixins/listener.rb

@@ -15,7 +15,7 @@ module Libuv
             end
 
             def define_callback(function:, params: [:pointer], ret_val: :void, lookup: :default_lookup)
-                @callback_funcs[function] = FFI::Function.new(ret_val, params) do |*args|
+                @callback_funcs[function] = ::FFI::Function.new(ret_val, params) do |*args|
                     dispatch_callback(function, lookup, args)
                 end
             end
@@ -41,8 +41,8 @@ module Libuv
 
         def self.included(base)
             base.instance_variable_set(:@callback_funcs, {})
-            base.instance_variable_set(:@callback_lookup, ThreadSafe::Cache.new)
-            base.instance_variable_set(:@callback_lock, Mutex.new)
+            base.instance_variable_set(:@callback_lookup, ::ThreadSafe::Cache.new)
+            base.instance_variable_set(:@callback_lock, ::Mutex.new)
             base.extend(ClassMethods)
         end
 

data/lib/libuv/mixins/stream.rb

@@ -70,7 +70,7 @@ module Libuv
                     assert_type(String, data, WRITE_ERROR)
 
                     buffer1 = ::FFI::MemoryPointer.from_string(data)
-                    buffer  = ::Libuv::Ext.buf_init(buffer1, data.respond_to?(:bytesize) ? data.bytesize : data.size)
+                    buffer  = ::Libuv::Ext.buf_init(buffer1, data.bytesize)
 
                     # local as this variable will be available until the handle is closed
                     @write_callbacks ||= {}
@@ -174,6 +174,7 @@ module Libuv
         end
 
         def on_shutdown(req, status)
+            cleanup_callbacks(req.address)
             ::Libuv::Ext.free(req)
             @close_error = check_result(status)
             close

data/lib/libuv/q.rb

@@ -156,11 +156,11 @@ module Libuv
             
             
             def pending
-                @defer.instance_eval { @pending }
+                @defer.pending
             end
             
             def value
-                @defer.instance_eval { @value }
+                @defer.value
             end
         end
         
@@ -226,6 +226,8 @@ module Libuv
                 @value = nil
                 @loop = loop
             end
+
+            attr_reader :pending, :value
             
             #
             # resolves the derived promise with the value. If the value is a rejection constructed via

data/lib/libuv/version.rb

@@ -1,3 +1,3 @@
 module Libuv
-    VERSION = '2.0.3'
+    VERSION = '2.0.4'
 end

data/spec/async_spec.rb

@@ -46,6 +46,7 @@ describe Libuv::Async do
 					@loop.stop
 				end
 				stopper.start(1000)
+				callback.close
 			end
 
 			@loop.work(proc {

data/spec/zen_spec.rb

@@ -0,0 +1,35 @@
+require 'libuv'
+
+
+describe Libuv::Listener do
+	it "should ensure there are no remaining object references in callbacks" do
+		require 'objspace'
+
+		checked = []
+
+		# These are created by loop objects and are never cleaned up
+		# This is OK as the loops are expected to execute for the life of the application
+		except = [::Libuv::Async, ::Libuv::Timer]
+
+		ObjectSpace.each_object(Class) do |cls|
+			next unless cls.ancestors.include? ::Libuv::Handle
+			next if checked.include? cls
+			checked << cls
+
+			values = cls.callback_lookup.values
+			values.select! {|val| except.include?(val.class) ? false : val.class }
+
+			if values.length > 0
+				puts "\nMemory Leak in #{cls} with #{values.length} left over objects"
+				puts "\nChecked #{checked.length} classes"
+				puts "\n\nObjects are:"
+				values.each do |val|
+					puts "\n#{val}\n"
+				end
+				raise 'Free the machines!'
+			end
+		end
+
+		expect(checked.length).to be > 3
+	end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: libuv
 version: !ruby/object:Gem::Version
-  version: 2.0.3
+  version: 2.0.4
 platform: ruby
 authors:
 - Bulat Shakirzyanov
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-20 00:00:00.000000000 Z
+date: 2015-08-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -490,6 +490,7 @@ files:
 - spec/pipe_spec.rb
 - spec/tcp_spec.rb
 - spec/udp_spec.rb
+- spec/zen_spec.rb
 homepage: https://github.com/cotag/libuv
 licenses:
 - MIT
@@ -525,4 +526,5 @@ test_files:
 - spec/pipe_spec.rb
 - spec/tcp_spec.rb
 - spec/udp_spec.rb
+- spec/zen_spec.rb
 has_rdoc: