allocations 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b60f6178ed9431369c8bc4445f31d5ff1196fd0f
-  data.tar.gz: 5299855c4bf14ff61c4abc047d42acdbee83eb39
+  metadata.gz: eb4fcbc39ec1eab5fe4831d66b372a71ea0d0d4a
+  data.tar.gz: 5abee9e95a64a77c97efce6ee84fc3423f83f4a5
 SHA512:
-  metadata.gz: 31565e3a8fc1a26582491b14885041aaf37016b93962ed3149ce15e316f4aad7499c430af7c538cf4ef5672c71645e0e2ed55da74eba6eb4f179c55bc4ce0ae4
-  data.tar.gz: 516095b00c662b3a3ab40f30d1f9c77d76c3e275cb2497691c2d405605a3fab4f8cb2a66fc97757ee69790cc82cb30f548f5b552cd139097388b7a521c0894a1
+  metadata.gz: c03ec7031bd717eac4bec3ff0939fec83549bec9eafa8696d76898fbe66d5a1d2809a63fce601b0007a5b4723d3029565eee001b48bcd76a9029144a51b9fe4e
+  data.tar.gz: d3eaf46688c14434715dc5b567a4da61eb7435e2e4c4bd904d27e357311a20088567d70d517fe0665148e0329bfe374513682f51f5dab4bc5f70edcdf0617c47

data/ext/liballocations/liballocations.c

@@ -2,8 +2,6 @@
 
 st_table *object_counts;
 
-VALUE mutex;
-
 VALUE allocation_tracer;
 VALUE free_tracer;
 
@@ -25,12 +23,8 @@ void newobj_callback(VALUE tracepoint, void* data) {
         return;
     }
 
-    rb_mutex_lock(mutex);
-
     st_lookup(object_counts, (st_data_t) klass, &count);
     st_insert(object_counts, (st_data_t) klass, count + 1);
-
-    rb_mutex_unlock(mutex);
 }
 
 /**
@@ -47,8 +41,6 @@ void freeobj_callback(VALUE tracepoint, void* data) {
     VALUE obj   = rb_tracearg_object(trace_arg);
     VALUE klass = RBASIC_CLASS(obj);
 
-    rb_mutex_lock(mutex);
-
     if ( st_lookup(object_counts, (st_data_t) klass, &count) ) {
         if ( count > 0 && (count - 1) > 0) {
             st_insert(object_counts, (st_data_t) klass, count - 1);
@@ -58,8 +50,6 @@ void freeobj_callback(VALUE tracepoint, void* data) {
             st_delete(object_counts, (st_data_t*) &klass, NULL);
         }
     }
-
-    rb_mutex_unlock(mutex);
 }
 
 /**
@@ -84,18 +74,12 @@ VALUE allocations_to_hash(VALUE self) {
     st_table *local_counts;
     VALUE hash;
 
-    rb_mutex_lock(mutex);
-
     if ( !object_counts ) {
-        rb_mutex_unlock(mutex);
-
         return rb_hash_new();
     }
 
     local_counts = st_copy(object_counts);
 
-    rb_mutex_unlock(mutex);
-
     hash = rb_hash_new();
 
     st_foreach(local_counts, each_count, (st_data_t) hash);
@@ -112,11 +96,7 @@ VALUE allocations_to_hash(VALUE self) {
  *     Allocations.start -> nil
  */
 VALUE allocations_start(VALUE self) {
-    rb_mutex_lock(mutex);
-
     if ( rb_ivar_get(self, id_enabled) == Qtrue ) {
-        rb_mutex_unlock(mutex);
-
         return Qnil;
     }
 
@@ -124,8 +104,6 @@ VALUE allocations_start(VALUE self) {
 
     rb_ivar_set(self, id_enabled, Qtrue);
 
-    rb_mutex_unlock(mutex);
-
     rb_tracepoint_enable(allocation_tracer);
     rb_tracepoint_enable(free_tracer);
 
@@ -139,11 +117,7 @@ VALUE allocations_start(VALUE self) {
  *     Allocations.stop -> nil
  */
 VALUE allocations_stop(VALUE self) {
-    rb_mutex_lock(mutex);
-
     if ( rb_ivar_get(self, id_enabled) != Qtrue ) {
-        rb_mutex_unlock(mutex);
-
         return Qnil;
     }
 
@@ -158,8 +132,6 @@ VALUE allocations_stop(VALUE self) {
 
     rb_ivar_set(self, id_enabled, Qfalse);
 
-    rb_mutex_unlock(mutex);
-
     return Qnil;
 }
 
@@ -172,14 +144,10 @@ VALUE allocations_stop(VALUE self) {
 VALUE allocations_enabled_p(VALUE self) {
     VALUE enabled = Qfalse;
 
-    rb_mutex_lock(mutex);
-
     if ( rb_ivar_get(self, id_enabled) == Qtrue ) {
         enabled = Qtrue;
     }
 
-    rb_mutex_unlock(mutex);
-
     return enabled;
 }
 
@@ -192,8 +160,6 @@ void Init_liballocations() {
     free_tracer = rb_tracepoint_new(Qnil, RUBY_INTERNAL_EVENT_FREEOBJ,
                                     freeobj_callback, NULL);
 
-    mutex = rb_mutex_new();
-
     id_enabled = rb_intern("enabled");
 
     rb_define_singleton_method(mAllocations, "to_hash", allocations_to_hash, 0);
@@ -201,7 +167,6 @@ void Init_liballocations() {
     rb_define_singleton_method(mAllocations, "stop", allocations_stop, 0);
     rb_define_singleton_method(mAllocations, "enabled?", allocations_enabled_p, 0);
 
-    rb_define_const(mAllocations, "MUTEX", mutex);
     rb_define_const(mAllocations, "ALLOCATION_TRACER", allocation_tracer);
     rb_define_const(mAllocations, "FREE_TRACER", free_tracer);
 }

data/lib/allocations/version.rb

@@ -1,3 +1,3 @@
 module Allocations
-  VERSION = '1.0.2'
+  VERSION = '1.0.3'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: allocations
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Yorick Peterse
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-17 00:00:00.000000000 Z
+date: 2015-12-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -101,7 +101,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.2.5
 signing_key: 
 specification_version: 4
 summary: Tracking of retained objects in CRuby