Fix clang-tidy warnings in iotop.

* Declare explicit conversion constructors.
* Use const reference type for parameter to avoid unnecessary copy.

Bug: 28341362
Bug: 30407689
Change-Id: Ia95a041125703ecbfac128709fd6ea0591df2103
Test: build with WITH_TIDY=1

3 files changed, 3 insertions, 3 deletions

diff --git a/iotop/iotop.cpp b/iotop/iotop.cpp
index 828fb32c..e2582e85 100644
--- a/iotop/iotop.cpp
+++ b/iotop/iotop.cpp
@@ -53,7 +53,7 @@ static void usage(char* myname) {
 }
 
 using Sorter = std::function<void(std::vector<TaskStatistics>&)>;
-static Sorter GetSorter(const std::string field) {
+static Sorter GetSorter(const std::string& field) {
   // Generic comparator
   static auto comparator = [](auto& lhs, auto& rhs, auto field, bool ascending) -> bool {
     auto a = (lhs.*field)();
diff --git a/iotop/tasklist.cpp b/iotop/tasklist.cpp
index f81143fb..07087414 100644
--- a/iotop/tasklist.cpp
+++ b/iotop/tasklist.cpp
@@ -28,7 +28,7 @@
 #include "tasklist.h"
 
 template<typename Func>
-static bool ScanPidsInDir(std::string name, Func f) {
+static bool ScanPidsInDir(const std::string& name, Func f) {
   std::unique_ptr<DIR, decltype(&closedir)> dir(opendir(name.c_str()), closedir);
   if (!dir) {
     return false;
diff --git a/iotop/taskstats.h b/iotop/taskstats.h
index f5ad2d20..bbf7e9e2 100644
--- a/iotop/taskstats.h
+++ b/iotop/taskstats.h
@@ -25,7 +25,7 @@ struct taskstats;
 
 class TaskStatistics {
 public:
-  TaskStatistics(const taskstats&);
+  explicit TaskStatistics(const taskstats&);
   TaskStatistics() = default;
   TaskStatistics(const TaskStatistics&) = default;
   void AddPidToTgid(const TaskStatistics&);