backtracie 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 887db5ff6ff55d3cf758d7ad192d5a01400a6c31ba6e46067073cf4bff0d34ac
-  data.tar.gz: 4b6b131ebfa4071b5dc222ca9758b30b22f8175c373de2813c85c2bd732a8a5c
+  metadata.gz: 54e3870e49e076db276f5db8ce6fdd4d3678a7dee52b5d7c11c047425cea931a
+  data.tar.gz: da7e2f00f3f569b5e1867c88343b29a68124f3cf005eca650bd32a5d45c3732b
 SHA512:
-  metadata.gz: 36e6e23f5eeeef37f14ca586891b9bdd06c44b5dc1b0ab984858f298ff87fb94f38eff9702521fb163587dd9771b73d3f208f98ce059d018a055830320fcc00d
-  data.tar.gz: 1b598ed425ae8ee01a51228c15886dc246c4324a8d4371c54fa36b49d6b66e08ffebff26af201cf3d756867aa7f3ae27dac791454984ebddb641ae0c693b021e
+  metadata.gz: f481358789f4a4577a29fb143c5951bc75cf47f69304094ba3b40361a1c1dce0f4736bda189a547222f65c88a5afb0bc13e7cfdc6bd723a5e21ceb327c32e1f0
+  data.tar.gz: 385384d64fe51ed93d084afa16d10e07565ed7d8f7d2bc25250bfe62c5962f607a061ad385d93203ab9768b878d29bd4ece827ca31d3bd30162ba93c4713d9b2

data/README.adoc

@@ -47,7 +47,7 @@ This gem is versioned according to http://semver.org/spec/v2.0.0.html[Semantic V
 Currently, `backtracie` exposes two APIs:
 
 * `Backtracie.backtrace_locations(thread)`: Returns an array representing the backtrace of the given `thread`. Similar to `Thread#backtrace_locations`.
-* `Backtracie.caller_locations`: Returns an array representing the backtrace of the current thread, starting from the caller of the current method. the Similar to `Kernel#caller_locations`.
+* `Backtracie.caller_locations`: Returns an array representing the backtrace of the current thread, starting from the caller of the current method. Similar to `Kernel#caller_locations`.
 
 These methods, inspired by their original Ruby counterparts, return arrays of `Backtracie::Location` items. These items return A LOT more information than Ruby usually exposes:
 

data/ext/backtracie_native_extension/backtracie.c

@@ -85,6 +85,7 @@ static VALUE collect_backtrace_locations(VALUE self, VALUE thread, int ignored_s
       stack_depth = backtracie_rb_profile_frames(MAX_STACK_DEPTH, raw_locations);
     } else {
       stack_depth = backtracie_rb_profile_frames_for_thread(thread, MAX_STACK_DEPTH, raw_locations);
+      if (stack_depth == 0 && !backtracie_is_thread_alive(thread)) return Qnil;
     }
   #else
     VALUE current_thread = rb_funcall(rb_cThread, current_id, 0);

data/ext/backtracie_native_extension/ruby_shards.c

@@ -238,12 +238,20 @@ int backtracie_rb_profile_frames(int limit, raw_location *raw_locations) {
   return backtracie_rb_profile_frames_for_execution_context(GET_EC(), limit, raw_locations);
 }
 
-int backtracie_rb_profile_frames_for_thread(VALUE thread, int limit, raw_location *raw_locations) {
+bool backtracie_is_thread_alive(VALUE thread) {
   // In here we're assuming that what we got is really a Thread or its subclass. This assumption NEEDS to be verified by
   // the caller, otherwise I see a segfault in your future.
   rb_thread_t *thread_pointer = (rb_thread_t*) DATA_PTR(thread);
 
-  if (thread_pointer->to_kill || thread_pointer->status == THREAD_KILLED) return Qnil;
+  return !(thread_pointer->to_kill || thread_pointer->status == THREAD_KILLED);
+}
+
+int backtracie_rb_profile_frames_for_thread(VALUE thread, int limit, raw_location *raw_locations) {
+  if (!backtracie_is_thread_alive(thread)) return 0;
+
+  // In here we're assuming that what we got is really a Thread or its subclass. This assumption NEEDS to be verified by
+  // the caller, otherwise I see a segfault in your future.
+  rb_thread_t *thread_pointer = (rb_thread_t*) DATA_PTR(thread);
 
   return backtracie_rb_profile_frames_for_execution_context(thread_pointer->ec, limit, raw_locations);
 }

data/ext/backtracie_native_extension/ruby_shards.h

@@ -131,6 +131,7 @@ typedef struct {
   #ifndef PRE_MJIT_RUBY
     int backtracie_rb_profile_frames(int limit, raw_location *raw_locations);
     int backtracie_rb_profile_frames_for_thread(VALUE thread, int limit, raw_location *raw_locations);
+    bool backtracie_is_thread_alive(VALUE thread);
   #endif
 VALUE backtracie_called_id(raw_location *the_location);
 VALUE backtracie_defined_class(raw_location *the_location);

data/lib/backtracie/version.rb

@@ -19,5 +19,5 @@
 # along with backtracie.  If not, see <http://www.gnu.org/licenses/>.
 
 module Backtracie
-  VERSION = "0.2.0"
+  VERSION = "0.2.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: backtracie
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Ivo Anjo
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-06-29 00:00:00.000000000 Z
+date: 2021-07-01 00:00:00.000000000 Z
 dependencies: []
 description: Ruby gem for beautiful backtraces
 email: