vernier 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 97eadc619870ae964e5bf61e925791d4399c0480775aadc5af31597535ae0374
-  data.tar.gz: 5327fd25cfac20752905f89126e8f6a4422e8cc90c134bab98a47b7132f7909d
+  metadata.gz: 9d826cc1605b081bf6784ff46e1e49b09958f3c2626637d3d994b967bc52b391
+  data.tar.gz: 743a02e32629c961795ab0a4b62f839e5bf905e3a4f339f6534157cd257f2e7a
 SHA512:
-  metadata.gz: f0fd5a598a03f41a0d5984d4652f3f85dece310ca83c015b554b85271cda220ca2535c198d4d62dad39532861ff38c858cb1048e46c29a43ce25c4fcb318ba7f
-  data.tar.gz: d7e0462723a56f1e7a1cef5c96f2a86958df9b129442defa7814c2b5dc8d2844a1507abf0ea486d7a0458257347e9cf243d6f369568258bbdde0fa374a9d7268
+  metadata.gz: ac4dc6ea1e3f9c32ba223e96aa7eae35c62f4a4e3d503efdee643b44984f5d03638b42a4d2e25fe991e8bc49240f75541da0c0791b525782f827969f1926d50c
+  data.tar.gz: 4321e93946c0c2155f79326f9bf29f00bf0fab24785c1c7238b8fdf46570706d8906bf70e80241a7dadd1d1531f8c44e12b4b47584b6024cf7d47fa9fa0a432d

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    vernier (0.1.0)
+    vernier (0.1.1)
 
 GEM
   remote: https://rubygems.org/

data/ext/vernier/vernier.cc

@@ -4,6 +4,7 @@
 #include <algorithm>
 #include <sstream>
 #include <unordered_map>
+#include <unordered_set>
 
 #include "vernier.hh"
 #include "stack.hh"
@@ -19,6 +20,7 @@ struct retained_collector {
     int allocated_objects = 0;
     int freed_objects = 0;
 
+    std::unordered_set<VALUE> unique_frames;
     std::unordered_map<VALUE, std::unique_ptr<Stack>> object_frames;
 };
 
@@ -54,6 +56,10 @@ newobj_i(VALUE tpval, void *data) {
     int lines_buffer[2048];
     int n = rb_profile_frames(0, 2048, frames_buffer, lines_buffer);
 
+    for (int i = 0; i < n; i++) {
+        collector->unique_frames.insert(frames_buffer[i]);
+    }
+
     collector->object_frames.emplace(
             tp.obj,
             make_unique<Stack>(frames_buffer, lines_buffer, n)
@@ -166,12 +172,8 @@ retained_collector_mark(void *data) {
     // We don't mark the objects, but we MUST mark the frames, otherwise they
     // can be garbage collected.
     // This may lead to method entries being unnecessarily retained.
-    for (auto& it: collector->object_frames) {
-        const Stack &stack = *it.second;
-
-        for (int i = 0; i < stack.size(); i++) {
-            rb_gc_mark(stack.frames[i]);
-        }
+    for (VALUE frame: collector->unique_frames) {
+        rb_gc_mark(frame);
     }
 }
 

data/lib/vernier/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Vernier
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vernier
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - John Hawthorn