Mark more constructors 'explicit'

Since c++17 the 'explicit' keyword has use also for constructors
with more than one argument and it is recommended to use it by
default whereever implicit conversions are not expected bahaviour.

1 files changed, 4 insertions, 4 deletions

diff --git a/src/buildtool/execution_engine/traverser/traverser.hpp b/src/buildtool/execution_engine/traverser/traverser.hpp
index 81f2e348..e99c2565 100644
--- a/src/buildtool/execution_engine/traverser/traverser.hpp
+++ b/src/buildtool/execution_engine/traverser/traverser.hpp
@@ -46,10 +46,10 @@ concept Runnable = requires(T const r,
 template <Runnable Executor>
 class Traverser {
   public:
-    Traverser(Executor const& r,
-              DependencyGraph const& graph,
-              std::size_t jobs,
-              gsl::not_null<std::atomic<bool>*> const& fail_flag)
+    explicit Traverser(Executor const& r,
+                       DependencyGraph const& graph,
+                       std::size_t jobs,
+                       gsl::not_null<std::atomic<bool>*> const& fail_flag)
         : runner_{r}, graph_{graph}, failed_{fail_flag}, tasker_{jobs} {}
     Traverser() = delete;
     Traverser(Traverser const&) = delete;