make Barrier constructor private and apply PR suggestions

src/System/include/BipedalLocomotion/System/Barrier.h

@@ -24,10 +24,9 @@ class Barrier
 {
 public:
     /**
-     * Constructor.
-     * @param counter initial value of the expected counter
+     * Calls the constructor. It creates a new Barrier with the given counter.
      */
-    explicit Barrier(const std::size_t counter);
+    static std::shared_ptr<Barrier> create(const std::size_t counter);
 
     /**
      * Blocks this thread at the phase synchronization point until its phase completion step is run
@@ -40,6 +39,12 @@ class Barrier
     std::size_t m_initialCount;
     std::size_t m_count;
     std::size_t m_generation;
+
+    /**
+     * Constructor.
+     * @param counter initial value of the expected counter
+     */
+    explicit Barrier(const std::size_t counter);
 };
 } // namespace System
 } // namespace BipedalLocomotion

src/System/src/Barrier.cpp

@@ -5,10 +5,11 @@
  * distributed under the terms of the BSD-3-Clause license.
  */
 
-#include "BipedalLocomotion/TextLogging/Logger.h"
+#include <memory>
 #include <mutex>
 
 #include <BipedalLocomotion/System/Barrier.h>
+#include <BipedalLocomotion/TextLogging/Logger.h>
 
 using namespace BipedalLocomotion::System;
 
@@ -19,6 +20,11 @@ Barrier::Barrier(const std::size_t counter)
 {
 }
 
+std::shared_ptr<Barrier> Barrier::create(const std::size_t counter)
+{
+    return std::shared_ptr<Barrier>(new Barrier(counter));
+}
+
 void Barrier::wait()
 {
     constexpr auto logPrefix = "[Barrier::wait]";
@@ -34,7 +40,7 @@ void Barrier::wait()
         m_count = m_initialCount;
 
         // notify the other threads
-        log()->info("{} All threads reached the barrier.", logPrefix);
+        log()->debug("{} All threads reached the barrier.", logPrefix);
         m_cond.notify_all();
     } else
     {

src/System/tests/AdvanceableRunnerTest.cpp

@@ -121,7 +121,7 @@ TEST_CASE("Test Block")
     SECTION("With synchronization")
     {
         constexpr std::size_t numberOfRunners = 2;
-        auto barrier = std::make_shared<Barrier>(numberOfRunners);
+        auto barrier = Barrier::create(numberOfRunners);
 
         // run the block
         auto thread0 = runner0.run(barrier);