dbg_tags 1.1.1 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d23d3bfe9f58ffa0b26481b6aa296208c6febff871649c197506ee836a54f36e
-  data.tar.gz: 669f9963e7dcdb9bffceabff6c7f45ba48d7b75c0d7d7293d9c65d16d772eb1d
+  metadata.gz: 269e2882bb82fb4a015a17417744067bcbdbd6b98a6f5e799f4f57a332b6eadf
+  data.tar.gz: 288f736d512ba40960086f54413d149b2b7558e5223cbf5ff545e15dab79b8f6
 SHA512:
-  metadata.gz: 1b2f84b3808a68011ae0a489c294e20f81d10ebc8a99f28d620191dd862b9ae399046ddfd5e9952334730377a11c6b49c8912aaca3a91c9617b24838899b5785
-  data.tar.gz: db37f12455aef1520cd5a0c1e93dbe2f352b07f3cd345fc92b367e26da649b6eb817e66550b90d64a46ebc9fa7feeb971edd0344292854b756f7e2bff23fab1d
+  metadata.gz: 072f1320bae3b4900d895370e17f717f125a721318a6e5a5bc393abef86507b18ca4de3c7b0f39ee4d12aa050dbd6829ceae3eb43bcdd44fe592b39da510114c
+  data.tar.gz: cfa63312d3ed1e16ab82d7e73737f6e87b5fe6df12dac8a11d8cec3ad6f0731c6029d43a223ed2d73e7cd2af9e07299175e2c5662854e6d725afaa51cce694ea

data/lib/dbg_tags.rb

@@ -187,14 +187,31 @@ module Tag
 
   class << self
 
+    private # class methods of Tag
+
+    # @param value [Bool] Value to set
+    # Primarily for rspec stuff. Use {disable_fiber_local_state!} instead
+    def no_fiber_local_state= value
+      @no_fiber_local_state = value
+    end # Tag::no_fiber_local_state=
+
     public # class methods of Tag
 
-    # @return [GlobalState] Thread local data 
+    # @return [Bool,nil] True if we should store global state inside the Tag class itself.
+    attr :no_fiber_local_state
+    alias no_fiber_local_state? no_fiber_local_state
+
+    # @return [Bool] True (the default) if we should store global state
+    # in each fiber/thread of the application.
+    def use_fiber_local_state?; !no_fiber_local_state? end
+
+    # @return [GlobalState] Either fiber local data (default) or truly global.
     def global_state
-      if gs = Thread.current[:dbg_tags_global_state]
-        gs
+      if @no_fiber_local_state # testing undefined ivar here. But no warnings... That is good,
+          # at least for performance
+        @global_state ||= GlobalState.new
       else
-        Thread.current[:dbg_tags_global_state] = GlobalState.new
+        Thread.current[:dbg_tags_global_state] ||= GlobalState.new
       end
     end # Tag::global_state
 
@@ -278,5 +295,20 @@ module Tag
     # @return [bool] Reflects explicit enable calls only. The :all feature is IGNORED
     def enabled? feature; (global_state.enabled[feature] || NONE) > NONE; end
 
+    # shortcut for no_fiber_local_state := true
+    # By default fiber local state is enabled but this means that
+    # 'enabling' of tags in some fiber does not work in others.
+    # Or more specific, that changing the Tag state outside of any fiber
+    # does not effect any fibers (already) created.
+    # Now it is possible to transfer data into the fiber using 'resume'
+    # but this is a hassle.
+    # So: when using threads leave this enabled, as it will cause race conditions.
+    # when using fibers, but no threads it is probably convenient to disable it.
+    def disable_fiber_local_state; self.no_fiber_local_state = true end
+
+    # shortcut for no_fiber_local_state := false
+    # For rspec use mostly.
+    def enable_fiber_local_state; self.no_fiber_local_state = false end
+
   end # singleton class Tag
 end # module Tag

data/spec/01_tag_spec.rb

@@ -255,6 +255,7 @@ describe 'tag' do
   end # context 'Nested levels'
 
   it 'has thread local data to prevent mix ups (tag_300)' do
+    Tag.enable_fiber_local_state # NOTE enabled by default, but other examples may botch it
     t1 = Thread.new do
       Tag.enable threads: :trc do
         Tag.trc(:threads) {
@@ -274,44 +275,74 @@ describe 'tag' do
   end # it
 
   it 'each thread has a private tag system (tag_301)' do
+    Tag.enable_fiber_local_state # NOTE enabled by default, but other examples may botch it
+    executed = false
     Tag.enable threads: :trc do
-      expect(Tag.enabled).to eq({threads: 3})
+      expect(Tag.enabled).to eq({threads: Tag::TRC})
       t1 = Thread.new do
         expect(Tag.enabled).to eq({})
         Tag.enable threads: :log do
-          expect(Tag.enabled).to eq({threads: 2})
-          Tag.trc(:threads) {
+          expect(Tag.enabled).to eq({threads: Tag::LOG})
+          Tag.log(:threads) {
             expect(Tag.inside?).to be true
             sleep 1
+            executed = true
             nil
           }
         end
       end 
       t1.join
-      expect(Tag.enabled).to eq({threads: 3})
+      expect(Tag.enabled).to eq({threads: Tag::TRC})
     end
+    expect(executed).to be true
   end # it
 
   it 'each fiber has a private tag system (tag_302)' do
+    Tag.enable_fiber_local_state # NOTE enabled by default, but other examples may botch it
+    executed = false
     Tag.enable threads: :trc do
-      expect(Tag.enabled).to eq({threads: 3})
+      expect(Tag.enabled).to eq({threads: Tag::TRC})
       t1 = Fiber.new do
         expect(Tag.enabled).to eq({})
         Tag.enable threads: :log do
-          expect(Tag.enabled).to eq({threads: 2})
-          Tag.trc(:threads) {
+          expect(Tag.enabled).to eq({threads: Tag::LOG})
+          Tag.log(:threads) {
             expect(Tag.inside?).to be true
-            sleep 1
+            executed = true
+            nil
+          }
+        end
+      end 
+      t1.resume
+      expect(Tag.enabled).to eq({threads: Tag::TRC})
+    end
+    expect(executed).to be true
+  end # it
+
+  it 'allows to disable fiber local state (tag_305)' do
+    Tag.disable_fiber_local_state
+    executed = false
+    Tag.enable threads: :trc do
+      expect(Tag.enabled).to eq({threads: Tag::TRC})
+      t1 = Fiber.new do
+        expect(Tag.enabled).to eq({threads: Tag::TRC})
+        Tag.enable threads: :log do
+          expect(Tag.enabled).to eq({threads: Tag::LOG})
+          Tag.log(:threads) {
+            expect(Tag.inside?).to be true
+            executed = true
             nil
           }
         end
       end 
       t1.resume
-      expect(Tag.enabled).to eq({threads: 3})
+      expect(Tag.enabled).to eq({threads: Tag::TRC})
+      expect(executed).to be true
     end
   end # it
 
   it 'does allow restore_state to transfer state through a Fiber barrier (tag_310)' do
+    Tag.enable_fiber_local_state
     did_something = false
     Tag.enable example: :dtl, fiber: :trc do
       state = Tag.state # same as Tag.enabled
@@ -331,6 +362,7 @@ describe 'tag' do
   end # it
 
   it 'does allow a nil-state to transfer state through a Fiber barrier (tag_311)' do
+    Tag.enable_fiber_local_state
     did_something = false
     state = Tag.state
     expect(state).to eq({})

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dbg_tags
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.1.2
 platform: ruby
 authors:
 - Eugene Brazwick
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-06-04 00:00:00.000000000 Z
+date: 2024-06-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec