maintenance: remove unused capture, member, and argument

SilKit/IntegrationTests/ITest_MessageAggregation.cpp

@@ -105,8 +105,8 @@ TEST_F(ITest_MessageAggregation, timeout_in_case_of_deadlock_when_using_async_si
         });
 
         timeSyncService->SetSimulationStepHandlerAsync(
-            [dataPublisher, lifecycleService, timeSyncService, &msgReceived](std::chrono::nanoseconds /*now*/,
-                                                                             std::chrono::nanoseconds /*duration*/) {
+            [dataPublisher, lifecycleService, &msgReceived](std::chrono::nanoseconds /*now*/,
+                                                                  std::chrono::nanoseconds /*duration*/) {
             // send ping
             std::vector<uint8_t> ping(1, '?');
             dataPublisher->Publish(std::move(ping));

SilKit/source/core/participant/Participant_impl.hpp

@@ -242,8 +242,7 @@ void Participant<SilKitConnectionT>::SetupMetrics()
         SilKit::Config::InternalController config;
         config.name = "MetricsReceiver";
         config.network = "default";
-        CreateController<VSilKit::MetricsReceiver>(config, std::move(supplementalData), true, true, *_logger,
-                                                   processor);
+        CreateController<VSilKit::MetricsReceiver>(config, std::move(supplementalData), true, true, processor);
     }
 }
 

SilKit/source/core/vasio/VAsioRegistry.cpp

@@ -333,7 +333,7 @@ void VAsioRegistry::SetupMetrics()
 
     if (_vasioConfig->experimental.metrics.collectFromRemote)
     {
-        auto metricsReceiver = std::make_unique<VSilKit::MetricsReceiver>(nullptr, *_logger, processor);
+        auto metricsReceiver = std::make_unique<VSilKit::MetricsReceiver>(nullptr, processor);
 
         SilKit::Core::SupplementalData supplementalData;
         supplementalData[SilKit::Core::Discovery::controllerType] =

SilKit/source/services/metrics/MetricsReceiver.cpp

@@ -13,10 +13,8 @@ namespace Log = SilKit::Services::Logging;
 namespace VSilKit {
 
 
-MetricsReceiver::MetricsReceiver(SilKit::Core::IParticipantInternal *, SilKit::Services::Logging::ILogger &logger,
-                                 IMetricsReceiverListener &listener)
-    : _logger{&logger}
-    , _listener{&listener}
+MetricsReceiver::MetricsReceiver(SilKit::Core::IParticipantInternal *, IMetricsReceiverListener &listener)
+    : _listener{&listener}
 {
     _serviceDescriptor.SetNetworkName("default");
 }

SilKit/source/services/metrics/MetricsReceiver.hpp

@@ -26,8 +26,7 @@ class MetricsReceiver
     , public SilKit::Core::IServiceEndpoint
 {
 public:
-    MetricsReceiver(SilKit::Core::IParticipantInternal*, SilKit::Services::Logging::ILogger& logger,
-                    IMetricsReceiverListener& listener);
+    MetricsReceiver(SilKit::Core::IParticipantInternal*, IMetricsReceiverListener& listener);
 
     // NB: The first constructor argument is present to enable using the CreateController function template. It is
     //     allowed to be nullptr.
@@ -40,7 +39,6 @@ class MetricsReceiver
     auto GetServiceDescriptor() const -> const SilKit::Core::ServiceDescriptor& override;
 
 private:
-    SilKit::Services::Logging::ILogger* _logger{nullptr};
     IMetricsReceiverListener* _listener{nullptr};
 
     SilKit::Core::ServiceDescriptor _serviceDescriptor;

SilKit/source/services/orchestration/TimeSyncService.hpp

@@ -148,7 +148,6 @@ class TimeSyncService
 
     std::vector<std::string> _requiredParticipants;
 
-    bool _isRunning{false};
     bool _isSynchronizingVirtualTime{false};
     bool _timeSyncConfigured{false};