update thread safe code

Author: 7bitcoder

Include/SevenBit/DI/Details/Core/Impl/ServiceInstanceProviderRoot.hpp

@@ -10,12 +10,16 @@ namespace sb::di::details
         : ServiceInstanceProvider(*this, options), _descriptorsMap(options.checkServiceGlobalUniqueness),
           _singletons(options.strongDestructionOrder)
     {
-        _descriptorsMap.seal();
     }
 
     INLINE void ServiceInstanceProviderRoot::init(ServiceProvider &serviceProvider)
     {
         ServiceInstanceProvider::init(serviceProvider);
+        _descriptorsMap.seal();
+        if (getOptions().threadSafe)
+        {
+            _mutex = std::make_unique<std::recursive_mutex>();
+        }
         if (getOptions().prebuildSingletons)
         {
             prebuildSingletons();

Include/SevenBit/DI/Details/Core/ServiceInstanceProvider.hpp

@@ -18,9 +18,9 @@ namespace sb::di::details
     class EXPORT ServiceInstanceProvider : public IServiceInstanceProvider
     {
         ServiceProviderOptions _options;
+        IServiceInstanceProviderRoot &_root;
         ServiceInstancesCreator _instancesCreator;
         ServiceAliasesCreator _aliasesCreator;
-        IServiceInstanceProviderRoot &_root;
         ServiceInstancesMap _scoped;
 
       public:

Include/SevenBit/DI/Details/Core/ServiceInstanceProviderRoot.hpp

@@ -27,11 +27,6 @@ namespace sb::di::details
               _descriptorsMap(begin, end, options.checkServiceGlobalUniqueness),
               _singletons(options.strongDestructionOrder)
         {
-            if (options.threadSafe)
-            {
-                _mutex = std::make_unique<std::recursive_mutex>();
-            }
-            _descriptorsMap.seal();
         }
 
         void init(ServiceProvider &serviceProvider) override;

Include/SevenBit/DI/ServiceProvider.hpp

@@ -32,11 +32,11 @@ namespace sb::di
             _syncMutex = _instanceProvider->tryGetSyncMutex();
         }
 
-        ServiceProvider(const ServiceProvider &parent) = delete;
+        ServiceProvider(const ServiceProvider &) = delete;
         ServiceProvider(ServiceProvider &&) = delete;
 
-        ServiceProvider &operator=(const ServiceProvider &parent) = delete;
-        ServiceProvider &operator=(ServiceProvider &&parent) = delete;
+        ServiceProvider &operator=(const ServiceProvider &) = delete;
+        ServiceProvider &operator=(ServiceProvider &&) = delete;
 
         /**
          * @brief Returns inner service instance provider