smart_initializer 0.9.0 → 0.10.0

data/lib/smart_core/initializer/attribute/list.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 class SmartCore::Initializer::Attribute::List
   # @since 0.1.0
   include Enumerable
@@ -10,9 +11,10 @@ class SmartCore::Initializer::Attribute::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def initialize
     @attributes = {}
-    @lock = SmartCore::Engine::Lock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @param attribute_name [Symbol]
@@ -22,15 +24,16 @@ class SmartCore::Initializer::Attribute::List
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def fetch(attribute_name)
-    thread_safe do
-      attributes.fetch(attribute_name)
-    rescue ::KeyError
+    @lock.read_sync do
       raise(
         ::SmartCore::Initializer::UndefinedAttributeError,
         "Attribute with `#{attribute_name}` name is not defined in your constructor. " \
         "Please, check your attribute definitions inside your class."
-      )
+      ) unless attributes.key?(attribute_name)
+
+      attributes[attribute_name]
     end
   end
   alias_method :[], :fetch
@@ -40,8 +43,9 @@ class SmartCore::Initializer::Attribute::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def add(attribute)
-    thread_safe { attributes[attribute.name] = attribute }
+    @lock.write_sync { attributes[attribute.name] = attribute }
   end
   alias_method :<<, :add
 
@@ -50,8 +54,9 @@ class SmartCore::Initializer::Attribute::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def concat(list)
-    thread_safe do
+    @lock.write_sync do
       list.each { |attribute| add(attribute.dup) }
     end
   end
@@ -62,7 +67,7 @@ class SmartCore::Initializer::Attribute::List
   # @api private
   # @since 0.1.0
   def include?(attribute)
-    thread_safe { attributes.key?(attribute.name) }
+    @lock.read_sync { attributes.key?(attribute.name) }
   end
 
   # @param block [Block]
@@ -70,8 +75,9 @@ class SmartCore::Initializer::Attribute::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def each(&block)
-    thread_safe do
+    @lock.read_sync do
       block_given? ? attributes.values.each(&block) : attributes.values.each
     end
   end
@@ -80,8 +86,9 @@ class SmartCore::Initializer::Attribute::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def size
-    thread_safe { attributes.size }
+    @lock.read_sync { attributes.size }
   end
 
   # @param block [Block]
@@ -89,8 +96,9 @@ class SmartCore::Initializer::Attribute::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def count(&block)
-    thread_safe { attributes.values.count(&block) }
+    @lock.read_sync { attributes.values.count(&block) }
   end
 
   private
@@ -100,13 +108,4 @@ class SmartCore::Initializer::Attribute::List
   # @api private
   # @since 0.1.0
   attr_reader :attributes
-
-  # @param block [Block]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe(&block)
-    @lock.synchronize(&block)
-  end
 end

data/lib/smart_core/initializer/constructor/definer.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 # rubocop:disable Metrics/ClassLength
 class SmartCore::Initializer::Constructor::Definer
   # @param klass [Class]
@@ -9,9 +10,10 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def initialize(klass)
     @klass = klass
-    @lock = SmartCore::Engine::Lock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @param block [Proc]
@@ -19,8 +21,9 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def define_init_extension(block)
-    thread_safe do
+    @lock.write_sync do
       add_init_extension(build_init_extension(block))
     end
   end
@@ -37,7 +40,7 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def define_parameter(
     name,
     type,
@@ -48,7 +51,7 @@ class SmartCore::Initializer::Constructor::Definer
     mutable,
     as
   )
-    thread_safe do
+    @lock.write_sync do
       attribute = build_param_attribute(
         name,
         type,
@@ -71,9 +74,9 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def define_parameters(*names, mutable:, privacy:)
-    thread_safe do
+    @lock.write_sync do
       names.map do |name|
         build_param_attribute(
           name,
@@ -107,7 +110,7 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def define_option(
     name,
     type,
@@ -120,7 +123,7 @@ class SmartCore::Initializer::Constructor::Definer
     default,
     optional
   )
-    thread_safe do
+    @lock.write_sync do
       attribute = build_option_attribute(
         name,
         type,
@@ -145,9 +148,9 @@ class SmartCore::Initializer::Constructor::Definer
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def define_options(*names, mutable:, privacy:)
-    thread_safe do
+    @lock.write_sync do
       names.map do |name|
         build_option_attribute(
           name,
@@ -352,14 +355,5 @@ class SmartCore::Initializer::Constructor::Definer
       ERROR_MESSAGE
     end
   end
-
-  # @param block [Block]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe(&block)
-    @lock.synchronize(&block)
-  end
 end
 # rubocop:enable Metrics/ClassLength

data/lib/smart_core/initializer/dsl.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 module SmartCore::Initializer::DSL
   require_relative 'dsl/inheritance'
 
@@ -11,13 +12,13 @@ module SmartCore::Initializer::DSL
     #
     # @api private
     # @since 0.1.0
+    # @version 0.10.0
     def extended(base_klass)
       base_klass.instance_eval do
         instance_variable_set(:@__params__, SmartCore::Initializer::Attribute::List.new)
         instance_variable_set(:@__options__, SmartCore::Initializer::Attribute::List.new)
         instance_variable_set(:@__init_extensions__, SmartCore::Initializer::Extensions::List.new)
         instance_variable_set(:@__definer__, SmartCore::Initializer::Constructor::Definer.new(self))
-        instance_variable_set(:@__deflock__, SmartCore::Engine::Lock.new)
         instance_variable_set(:@__initializer_settings__, SmartCore::Initializer::Settings.new)
       end
       base_klass.extend(ClassMethods)
@@ -27,20 +28,20 @@ module SmartCore::Initializer::DSL
 
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   module ClassInheritance
     # @param child_klass [Class]
     # @return [void]
     #
     # @api private
     # @since 0.1.0
-    # @version 0.3.2
+    # @version 0.10.0
     def inherited(child_klass)
       child_klass.instance_exec(__initializer_settings__) do |init_settings|
         instance_variable_set(:@__params__, SmartCore::Initializer::Attribute::List.new)
         instance_variable_set(:@__options__, SmartCore::Initializer::Attribute::List.new)
         instance_variable_set(:@__init_extensions__, SmartCore::Initializer::Extensions::List.new)
         instance_variable_set(:@__definer__, SmartCore::Initializer::Constructor::Definer.new(self))
-        instance_variable_set(:@__deflock__, SmartCore::Engine::Lock.new)
         instance_variable_set(:@__initializer_settings__, init_settings.dup)
       end
       child_klass.extend(ClassMethods)

data/lib/smart_core/initializer/extensions/list.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 class SmartCore::Initializer::Extensions::List
   # @since 0.1.0
   include Enumerable
@@ -10,9 +11,10 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def initialize
     @extensions = []
-    @lock = SmartCore::Engine::Lock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @param extension [SmartCore::Initializer::Extensions::Abstract]
@@ -20,8 +22,9 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def add(extension)
-    thread_safe { extensions << extension }
+    @lock.write_sync { extensions << extension }
   end
   alias_method :<<, :add
 
@@ -30,8 +33,9 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def concat(list)
-    thread_safe do
+    @lock.write_sync do
       list.each { |extension| add(extension.dup) }
     end
   end
@@ -41,8 +45,9 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def each(&block)
-    thread_safe do
+    @lock.read_sync do
       block_given? ? extensions.each(&block) : extensions.each
     end
   end
@@ -51,8 +56,9 @@ class SmartCore::Initializer::Extensions::List
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def size
-    thread_safe { extensions.size }
+    @lock.read_sync { extensions.size }
   end
 
   private
@@ -62,13 +68,4 @@ class SmartCore::Initializer::Extensions::List
   # @api private
   # @since 0.1.0
   attr_reader :extensions
-
-  # @param block [Block]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe(&block)
-    @lock.synchronize(&block)
-  end
 end

data/lib/smart_core/initializer/plugins/abstract.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 class SmartCore::Initializer::Plugins::Abstract
   class << self
     # @param child_klass [Class]
@@ -9,9 +10,10 @@ class SmartCore::Initializer::Plugins::Abstract
     #
     # @api private
     # @since 0.1.0
+    # @version 0.10.0
     def inherited(child_klass)
       child_klass.instance_variable_set(:@__loaded__, false)
-      child_klass.instance_variable_set(:@__lock__, Mutex.new)
+      child_klass.instance_variable_set(:@__lock__, SmartCore::Engine::Lock.new)
       super
     end
 

data/lib/smart_core/initializer/plugins/registry.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 class SmartCore::Initializer::Plugins::Registry
   # @since 0.1.0
   include Enumerable
@@ -10,9 +11,10 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def initialize
     @plugin_set = {}
-    @access_lock = Mutex.new
+    @access_lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @param plugin_name [Symbol, String]
@@ -20,8 +22,9 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def [](plugin_name)
-    thread_safe { fetch(plugin_name) }
+    @access_lock.read_sync { fetch(plugin_name) }
   end
 
   # @param plugin_name [Symbol, String]
@@ -30,8 +33,9 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def register(plugin_name, plugin_module)
-    thread_safe { apply(plugin_name, plugin_module) }
+    @access_lock.write_sync { apply(plugin_name, plugin_module) }
   end
   alias_method :[]=, :register
 
@@ -39,16 +43,18 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def names
-    thread_safe { plugin_names }
+    @access_lock.read_sync { plugin_names }
   end
 
   # @return [Hash<String,Class<SmartCore::Initializer::Plugins::Abstract>>]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def loaded
-    thread_safe { loaded_plugins }
+    @access_lock.read_sync { loaded_plugins }
   end
 
   # @param block [Block]
@@ -56,8 +62,9 @@ class SmartCore::Initializer::Plugins::Registry
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def each(&block)
-    thread_safe { iterate(&block) }
+    @access_lock.read_sync { iterate(&block) }
   end
 
   private
@@ -74,14 +81,6 @@ class SmartCore::Initializer::Plugins::Registry
   # @since 0.1.0
   attr_reader :access_lock
 
-  # @return [void]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe
-    access_lock.synchronize { yield if block_given? }
-  end
-
   # @return [Array<String>]
   #
   # @api private

data/lib/smart_core/initializer/plugins/registry_interface.rb

@@ -2,6 +2,7 @@
 
 # @api private
 # @since 0.1.0
+# @version 0.10.0
 module SmartCore::Initializer::Plugins::RegistryInterface
   class << self
     # @param base_module [Class, Module]
@@ -9,12 +10,13 @@ module SmartCore::Initializer::Plugins::RegistryInterface
     #
     # @api private
     # @since 0.1.0
+    # @version 0.10.0
     def extended(base_module)
       base_module.instance_variable_set(
         :@plugin_registry, SmartCore::Initializer::Plugins::Registry.new
       )
       base_module.instance_variable_set(
-        :@access_lock, SmartCore::Engine::Lock.new
+        :@access_lock, SmartCore::Engine::ReadWriteLock.new
       )
     end
   end
@@ -24,24 +26,27 @@ module SmartCore::Initializer::Plugins::RegistryInterface
   #
   # @api public
   # @since 0.1.0
+  # @version 0.10.0
   def load(plugin_name)
-    thread_safe { plugin_registry[plugin_name].load! }
+    @access_lock.read_sync { plugin_registry[plugin_name].load! }
   end
 
   # @return [Array<String>]
   #
   # @api public
   # @since 0.1.0
+  # @version 0.10.0
   def loaded_plugins
-    thread_safe { plugin_registry.loaded.keys }
+    @access_lock.read_sync { plugin_registry.loaded.keys }
   end
 
   # @return [Array<String>]
   #
   # @api public
   # @since 0.1.0
+  # @version 0.10.0
   def names
-    thread_safe { plugin_registry.names }
+    @access_lock.read_sync { plugin_registry.names }
   end
 
   # @param plugin_name [Symbol, String]
@@ -49,8 +54,9 @@ module SmartCore::Initializer::Plugins::RegistryInterface
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def register_plugin(plugin_name, plugin_module)
-    thread_safe { plugin_registry[plugin_name] = plugin_module }
+    @access_lock.write_sync { plugin_registry[plugin_name] = plugin_module }
   end
 
   private
@@ -60,18 +66,4 @@ module SmartCore::Initializer::Plugins::RegistryInterface
   # @api private
   # @since 0.1.0
   attr_reader :plugin_registry
-
-  # @return [SmartCore::Engine::Lock]
-  #
-  # @api private
-  # @since 0.1.0
-  attr_reader :access_lock
-
-  # @return [void]
-  #
-  # @api private
-  # @since 0.1.0
-  def thread_safe
-    access_lock.synchronize { yield if block_given? }
-  end
 end

data/lib/smart_core/initializer/settings/auto_cast.rb

@@ -2,22 +2,15 @@
 
 # @api private
 # @since 0.8.0
+# @version 0.10.0
 class SmartCore::Initializer::Settings::AutoCast < SmartCore::Initializer::Settings::Base
-  # @return [void]
-  #
-  # @api private
-  # @since 0.8.0
-  def initialize
-    @value = nil
-    @lock = SmartCore::Engine::Lock.new
-  end
-
   # @return [Boolean]
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def resolve
-    thread_safe do
+    @lock.read_sync do
       @value == nil ? SmartCore::Initializer::Configuration[:auto_cast] : @value
     end
   end
@@ -27,8 +20,9 @@ class SmartCore::Initializer::Settings::AutoCast < SmartCore::Initializer::Setti
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def assign(value)
-    thread_safe do
+    @lock.write_sync do
       raise(
         SmartCore::Initializer::SettingArgumentError,
         ":auto_cast setting should be a type of boolean (got: `#{value.class}`)"

data/lib/smart_core/initializer/settings/base.rb

@@ -2,14 +2,16 @@
 
 # @api private
 # @since 0.8.0
+# @version 0.10.0
 class SmartCore::Initializer::Settings::Base
   # @return [void]
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def initialize
     @value = nil
-    @lock = SmartCore::Engine::Lock.new
+    @lock = SmartCore::Engine::ReadWriteLock.new
   end
 
   # @!method resolve
@@ -28,22 +30,12 @@ class SmartCore::Initializer::Settings::Base
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def dup
-    thread_safe do
+    @lock.write_sync do
       self.class.new.tap do |duplicate|
         duplicate.instance_variable_set(:@value, @value)
       end
     end
   end
-
-  private
-
-  # @param block [Block]
-  # @return [Any]
-  #
-  # @api private
-  # @since 0.8.0
-  def thread_safe(&block)
-    @lock.synchronize(&block)
-  end
 end

data/lib/smart_core/initializer/settings/strict_options.rb

@@ -2,22 +2,15 @@
 
 # @api private
 # @since 0.8.0
+# @version 0.10.0
 class SmartCore::Initializer::Settings::StrictOptions < SmartCore::Initializer::Settings::Base
-  # @return [void]
-  #
-  # @api private
-  # @since 0.8.0
-  def initialize
-    @value = nil
-    @lock = SmartCore::Engine::Lock.new
-  end
-
   # @return [Boolean]
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def resolve
-    thread_safe do
+    @lock.read_sync do
       @value == nil ? SmartCore::Initializer::Configuration[:strict_options] : @value
     end
   end
@@ -27,8 +20,9 @@ class SmartCore::Initializer::Settings::StrictOptions < SmartCore::Initializer::
   #
   # @api private
   # @since 0.8.0
+  # @version 0.10.0
   def assign(value)
-    thread_safe do
+    @lock.write_sync do
       raise(
         SmartCore::Initializer::SettingArgumentError,
         ":strict_options setting should be a type of boolean (got: `#{value.class}`)"

data/lib/smart_core/initializer/settings/type_system.rb

@@ -2,24 +2,15 @@
 
 # @api private
 # @since 0.1.0
-# @version 0.8.0
+# @version 0.10.0
 class SmartCore::Initializer::Settings::TypeSystem < SmartCore::Initializer::Settings::Base
-  # @return [void]
-  #
-  # @api private
-  # @since 0.1.0
-  # @version 0.8.0
-  def initialize
-    @value = nil
-    @lock = SmartCore::Engine::Lock.new
-  end
-
   # @return [Any]
   #
   # @api private
   # @since 0.1.0
+  # @version 0.10.0
   def generic_type_object
-    thread_safe do
+    @lock.read_sync do
       SmartCore::Initializer::TypeSystem.resolve(resolve).generic_type_object
     end
   end
@@ -28,9 +19,9 @@ class SmartCore::Initializer::Settings::TypeSystem < SmartCore::Initializer::Set
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def resolve
-    thread_safe do
+    @lock.read_sync do
       @value == nil ? SmartCore::Initializer::Configuration[:default_type_system] : @value
     end
   end
@@ -40,9 +31,9 @@ class SmartCore::Initializer::Settings::TypeSystem < SmartCore::Initializer::Set
   #
   # @api private
   # @since 0.1.0
-  # @version 0.8.0
+  # @version 0.10.0
   def assign(value)
-    thread_safe do
+    @lock.write_sync do
       # NOTE: type system existence validation
       SmartCore::Initializer::TypeSystem.resolve(value)
       @value = value