clogger 0.3.0 → 0.3.1

data/ext/clogger_ext/clogger.c

@@ -188,7 +188,7 @@ static VALUE clogger_alloc(VALUE klass)
 {
 	struct clogger *c;
 
-	return Data_Make_Struct(klass, struct clogger, clogger_mark, 0, c);
+	return Data_Make_Struct(klass, struct clogger, clogger_mark, -1, c);
 }
 
 static struct clogger *clogger_get(VALUE self)

data/test/test_clogger.rb

@@ -587,4 +587,22 @@ class TestClogger < Test::Unit::TestCase
     assert ! cl.reentrant?
   end
 
+  # so we don't  care about the portability of this test
+  # if it doesn't leak on Linux, it won't leak anywhere else
+  # unless your C compiler or platform is otherwise broken
+  LINUX_PROC_PID_STATUS = "/proc/self/status"
+  def test_memory_leak
+    app = lambda { |env| [ 0, {}, [] ] }
+    clogger = Clogger.new(app, :logger => $stderr)
+    match_rss = /^VmRSS:\s+(\d+)/
+    if File.read(LINUX_PROC_PID_STATUS) =~ match_rss
+      before = $1.to_i
+      1000000.times { clogger.dup }
+      File.read(LINUX_PROC_PID_STATUS) =~ match_rss
+      after = $1.to_i
+      diff = after - before
+      assert(diff < 10000, "memory grew more than 10M: #{diff}")
+    end
+  end if RUBY_PLATFORM =~ /linux/ && test(?r, LINUX_PROC_PID_STATUS)
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: clogger
 version: !ruby/object:Gem::Version 
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors: 
 - cloggers
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-02-09 00:00:00 +00:00
+date: 2010-02-13 00:00:00 +00:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency