Mark some of the single argument constructors as explicit

Author: vil02

include/CXXGraph/Edge/DirectedWeightedEdge_decl.h

@@ -63,7 +63,7 @@ class DirectedWeightedEdge : public DirectedEdge<T>, public Weighted {
       const double weight);
   DirectedWeightedEdge(const DirectedEdge<T> &edge, const double weight);
   DirectedWeightedEdge(const Edge<T> &edge, const double weight);
-  DirectedWeightedEdge(const DirectedEdge<T> &edge);
+  explicit DirectedWeightedEdge(const DirectedEdge<T> &edge);
   DirectedWeightedEdge(const Edge<T> &edge);
   DirectedWeightedEdge(const UndirectedWeightedEdge<T> &edge);
   ~DirectedWeightedEdge() override = default;

include/CXXGraph/Edge/UndirectedWeightedEdge_decl.h

@@ -65,7 +65,7 @@ class UndirectedWeightedEdge : public UndirectedEdge<T>, public Weighted {
       const double weight);
   UndirectedWeightedEdge(const UndirectedEdge<T> &edge, const double weight);
   UndirectedWeightedEdge(const Edge<T> &edge, const double weight);
-  UndirectedWeightedEdge(const UndirectedEdge<T> &edge);
+  explicit UndirectedWeightedEdge(const UndirectedEdge<T> &edge);
   UndirectedWeightedEdge(const Edge<T> &edge);
   UndirectedWeightedEdge(const DirectedWeightedEdge<T> &edge);
   ~UndirectedWeightedEdge() override = default;

include/CXXGraph/Graph/Graph_decl.h

@@ -204,7 +204,7 @@ class Graph {
 
  public:
   Graph();
-  Graph(const T_EdgeSet<T> &edgeSet);
+  explicit Graph(const T_EdgeSet<T> &edgeSet);
   virtual ~Graph() = default;
 
   /**

include/CXXGraph/Partitioning/CoordinatedPartitionState.hpp

@@ -62,7 +62,7 @@ class CoordinatedPartitionState : public PartitionState<T> {
   std::shared_ptr<std::mutex> record_map_mutex = nullptr;
   // DatWriter out; //to print the final partition of each edge
  public:
-  CoordinatedPartitionState(const Globals &G);
+  explicit CoordinatedPartitionState(const Globals &G);
   ~CoordinatedPartitionState() override;
 
   std::shared_ptr<Record<T>> getRecord(CXXGraph::id_t x) override;

include/CXXGraph/Partitioning/Utility/Globals.hpp

@@ -33,11 +33,11 @@ namespace Partitioning {
 class Globals {
  private:
  public:
-  Globals(const int numberOfPartiton,
-          const PartitionAlgorithm algorithm = PartitionAlgorithm::HDRF_ALG,
-          const double param1 = 1, const double param2 = 1,
-          const double param3 = 1,
-          const unsigned int threads = std::thread::hardware_concurrency());
+  explicit Globals(
+      const int numberOfPartiton,
+      const PartitionAlgorithm algorithm = PartitionAlgorithm::HDRF_ALG,
+      const double param1 = 1, const double param2 = 1, const double param3 = 1,
+      const unsigned int threads = std::thread::hardware_concurrency());
   ~Globals();
 
   const std::string print() const;