commit 507db2c022390c6df4808b5bb135c69c575bb807
author Jan Niklas Hasse <jhasse@bixense.com> Sun Dec 06 12:15:55 2020 +0100
committer GitHub <noreply@github.com> Sun Dec 06 12:15:55 2020 +0100
tree af66912f21bf1972923da76f1a21f628ac0d8066
parent e664814b6aa5f897accf88f6ccd06ea323a3dc7c
parent 0351d68f0a6bf326126b8e537c788c486b46aa14

Merge pull request #1791 from neheb/exp

[clang-tidy] add explicit to single argument constructors

src/build_test.cc

diff --git a/src/build_test.cc b/src/build_test.cc
index 078080d..0baabc4 100644
--- a/src/build_test.cc
+++ b/src/build_test.cc
@@ -490,10 +490,9 @@
                 status_(config_) {
   }
 
-  BuildTest(DepsLog* log) : config_(MakeConfig()), command_runner_(&fs_),
-                            builder_(&state_, config_, NULL, log, &fs_),
-                            status_(config_) {
-  }
+  explicit BuildTest(DepsLog* log)
+      : config_(MakeConfig()), command_runner_(&fs_),
+        builder_(&state_, config_, NULL, log, &fs_), status_(config_) {}
 
   virtual void SetUp() {
     StateTestWithBuiltinRules::SetUp();

src/graph.cc

diff --git a/src/graph.cc b/src/graph.cc
index ea11360..78d0d49 100644
--- a/src/graph.cc
+++ b/src/graph.cc
@@ -515,7 +515,7 @@
 }
 
 struct matches {
-  matches(std::vector<StringPiece>::iterator i) : i_(i) {}
+  explicit matches(std::vector<StringPiece>::iterator i) : i_(i) {}
 
   bool operator()(const Node* node) const {
     StringPiece opath = StringPiece(node->path());