Fix some clang-tidy warnings

Incorporated some fixes by Daniel Chabrowski (#467)

src/indexer.cc

@@ -663,7 +663,7 @@ public:
           rd->isInvalidDecl() || !validateRecord(rd))
         offset = -1;
       for (FieldDecl *fd : rd->fields()) {
-        int offset1 = offset < 0 ? -1 : offset + ctx->getFieldOffset(fd);
+        int offset1 = offset < 0 ? -1 : int(offset + ctx->getFieldOffset(fd));
         if (fd->getIdentifier())
           type.def.vars.emplace_back(getUsr(fd), offset1);
         else if (const auto *rt1 = fd->getType()->getAs<RecordType>()) {

src/lsp.cc

@@ -33,7 +33,7 @@ void reflect(JsonWriter &visitor, RequestId &value) {
     visitor.null_();
     break;
   case RequestId::kInt:
-    visitor.int_(atoll(value.value.c_str()));
+    visitor.int64(atoll(value.value.c_str()));
     break;
   case RequestId::kString:
     visitor.string(value.value.c_str(), value.value.size());
@@ -51,7 +51,7 @@ void DocumentUri::setPath(const std::string &path) {
   // file:///c%3A/Users/jacob/Desktop/superindex/indexer/full_tests
   raw_uri = path;
 
-  size_t index = raw_uri.find(":");
+  size_t index = raw_uri.find(':');
   if (index == 1) { // widows drive letters must always be 1 char
     raw_uri.replace(raw_uri.begin() + index, raw_uri.begin() + index + 1,
                     "%3A");

src/message_handler.cc

@@ -109,10 +109,8 @@ void ReplyOnce::replyLocationLink(std::vector<LocationLink> &result) {
   if (g_config->client.linkSupport) {
     (*this)(result);
   } else {
-    std::vector<Location> result1;
+    (*this)(std::vector<Location>(std::make_move_iterator(result.begin()),
-    for (auto &loc : result)
+                                  std::make_move_iterator(result.end())));
-      result1.emplace_back(std::move(loc));
-    (*this)(result1);
   }
 }
 

src/message_handler.hh

@@ -20,8 +20,9 @@ struct WorkingFile;
 struct WorkingFiles;
 
 namespace pipeline {
-void reply(RequestId id, const std::function<void(JsonWriter &)> &fn);
+void reply(const RequestId &id, const std::function<void(JsonWriter &)> &fn);
-void replyError(RequestId id, const std::function<void(JsonWriter &)> &fn);
+void replyError(const RequestId &id,
+                const std::function<void(JsonWriter &)> &fn);
 } // namespace pipeline
 
 struct CodeActionParam {

src/messages/initialize.cc

@@ -370,7 +370,7 @@ void do_initialize(MessageHandler *m, InitializeParam &param,
   // may take a long time. Indexer threads will emit status/progress
   // reports.
   if (g_config->index.threads == 0)
-    g_config->index.threads = std::thread::hardware_concurrency();
+    g_config->index.threads = (int)std::thread::hardware_concurrency();
 
   LOG_S(INFO) << "start " << g_config->index.threads << " indexers";
   for (int i = 0; i < g_config->index.threads; i++)

src/messages/textDocument_formatting.cc

@@ -32,8 +32,8 @@ std::vector<TextEdit> replacementsToEdits(std::string_view code,
                                           const tooling::Replacements &repls) {
   std::vector<TextEdit> ret;
   int i = 0, line = 0, col = 0;
-  auto move = [&](int p) {
+  auto move = [&](unsigned p) {
-    for (; i < p; i++)
+    for (; i < (int)p; i++)
       if (code[i] == '\n')
         line++, col = 0;
       else {

src/messages/workspace.cc

@@ -89,7 +89,7 @@ void MessageHandler::workspace_didChangeWorkspaceFolders(
     if (folder == real)
       real.clear();
     LOG_S(INFO) << "add workspace folder " << wf.name << ": "
-                << (real.empty() ? folder : folder + " -> " + real);
+                << (real.empty() ? folder : (folder + " -> ").append(real));
     workspaceFolders.emplace_back();
     auto it = workspaceFolders.end() - 1;
     for (; it != workspaceFolders.begin() && folder < it[-1].first; --it)

src/pipeline.cc

@@ -605,13 +605,13 @@ void mainLoop() {
 
   SemaManager manager(
       &project, &wfiles,
-      [&](std::string path, std::vector<Diagnostic> diagnostics) {
+      [](const std::string &path, std::vector<Diagnostic> diagnostics) {
         PublishDiagnosticParam params;
         params.uri = DocumentUri::fromPath(path);
-        params.diagnostics = diagnostics;
+        params.diagnostics = std::move(diagnostics);
         notify("textDocument/publishDiagnostics", params);
       },
-      [](RequestId id) {
+      [](const RequestId &id) {
         if (id.valid()) {
           ResponseError err;
           err.code = ErrorCode::InternalError;
@@ -706,8 +706,9 @@ void standalone(const std::string &root) {
   WorkingFiles wfiles;
   VFS vfs;
   SemaManager manager(
-      nullptr, nullptr, [&](std::string, std::vector<Diagnostic>) {},
+      nullptr, nullptr,
-      [](RequestId id) {});
+      [](const std::string &, const std::vector<Diagnostic> &) {},
+      [](const RequestId &id) {});
   IncludeComplete complete(&project);
 
   MessageHandler handler;
@@ -745,7 +746,7 @@ void standalone(const std::string &root) {
 void index(const std::string &path, const std::vector<const char *> &args,
            IndexMode mode, bool must_exist, RequestId id) {
   pending_index_requests++;
-  index_request->pushBack({path, args, mode, must_exist, id},
+  index_request->pushBack({path, args, mode, must_exist, std::move(id)},
                           mode != IndexMode::Background);
 }
 
@@ -789,7 +790,7 @@ void notifyOrRequest(const char *method, bool request,
   for_stdout->pushBack(output.GetString());
 }
 
-static void reply(RequestId id, const char *key,
+static void reply(const RequestId &id, const char *key,
                   const std::function<void(JsonWriter &)> &fn) {
   rapidjson::StringBuffer output;
   rapidjson::Writer<rapidjson::StringBuffer> w(output);
@@ -802,7 +803,7 @@ static void reply(RequestId id, const char *key,
     w.Null();
     break;
   case RequestId::kInt:
-    w.Int(atoll(id.value.c_str()));
+    w.Int64(atoll(id.value.c_str()));
     break;
   case RequestId::kString:
     w.String(id.value.c_str(), id.value.size());
@@ -817,11 +818,12 @@ static void reply(RequestId id, const char *key,
   for_stdout->pushBack(output.GetString());
 }
 
-void reply(RequestId id, const std::function<void(JsonWriter &)> &fn) {
+void reply(const RequestId &id, const std::function<void(JsonWriter &)> &fn) {
   reply(id, "result", fn);
 }
 
-void replyError(RequestId id, const std::function<void(JsonWriter &)> &fn) {
+void replyError(const RequestId &id,
+                const std::function<void(JsonWriter &)> &fn) {
   reply(id, "error", fn);
 }
 } // namespace pipeline

src/pipeline.hh

@@ -68,10 +68,11 @@ template <typename T> void request(const char *method, T &result) {
   notifyOrRequest(method, true, [&](JsonWriter &w) { reflect(w, result); });
 }
 
-void reply(RequestId id, const std::function<void(JsonWriter &)> &fn);
+void reply(const RequestId &id, const std::function<void(JsonWriter &)> &fn);
 
-void replyError(RequestId id, const std::function<void(JsonWriter &)> &fn);
+void replyError(const RequestId &id,
-template <typename T> void replyError(RequestId id, T &result) {
+                const std::function<void(JsonWriter &)> &fn);
+template <typename T> void replyError(const RequestId &id, T &result) {
   replyError(id, [&](JsonWriter &w) { reflect(w, result); });
 }
 } // namespace pipeline

src/project.cc

@@ -237,7 +237,7 @@ bool appendToCDB(const std::vector<const char *> &args) {
   return args.size() && StringRef("%compile_commands.json") == args[0];
 }
 
-std::vector<const char *> getFallback(const std::string path) {
+std::vector<const char *> getFallback(const std::string &path) {
   std::vector<const char *> argv{"clang"};
   if (sys::path::extension(path) == ".h")
     argv.push_back("-xobjective-c++-header");
@@ -595,7 +595,7 @@ Project::Entry Project::findEntry(const std::string &path, bool can_redirect,
   return ret;
 }
 
-void Project::index(WorkingFiles *wfiles, RequestId id) {
+void Project::index(WorkingFiles *wfiles, const RequestId &id) {
   auto &gi = g_config->index;
   GroupMatch match(gi.whitelist, gi.blacklist),
       match_i(gi.initialWhitelist, gi.initialBlacklist);

src/project.hh

@@ -64,7 +64,7 @@ struct Project {
   void setArgsForFile(const std::vector<const char *> &args,
                       const std::string &path);
 
-  void index(WorkingFiles *wfiles, RequestId id);
+  void index(WorkingFiles *wfiles, const RequestId &id);
   void indexRelated(const std::string &path);
 };
 } // namespace ccls

src/sema_manager.cc

@@ -604,7 +604,7 @@ void *diagnosticMain(void *manager_) {
         ret.severity = 1;
         break;
       }
-      ret.code = d.category;
+      ret.code = (int)d.category;
       return ret;
     };
 
@@ -673,8 +673,10 @@ std::shared_ptr<PreambleData> Session::getPreamble() {
 
 SemaManager::SemaManager(Project *project, WorkingFiles *wfiles,
                          OnDiagnostic on_diagnostic, OnDropped on_dropped)
-    : project_(project), wfiles(wfiles), on_diagnostic_(on_diagnostic),
+    : project_(project), wfiles(wfiles),
-      on_dropped_(on_dropped), pch(std::make_shared<PCHContainerOperations>()) {
+      on_diagnostic_(std::move(on_diagnostic)),
+      on_dropped_(std::move(on_dropped)),
+      pch(std::make_shared<PCHContainerOperations>()) {
   spawnThread(ccls::preambleMain, this);
   spawnThread(ccls::completionMain, this);
   spawnThread(ccls::diagnosticMain, this);

src/serializer.cc

@@ -13,6 +13,7 @@
 
 #include <llvm/ADT/CachedHashString.h>
 #include <llvm/ADT/DenseSet.h>
+#include <llvm/ADT/STLExtras.h>
 #include <llvm/Support/Allocator.h>
 
 #include <mutex>
@@ -24,7 +25,7 @@ bool gTestOutputMode = false;
 
 namespace ccls {
 
-void JsonReader::iterArray(std::function<void()> fn) {
+void JsonReader::iterArray(llvm::function_ref<void()> fn) {
   if (!m->IsArray())
     throw std::invalid_argument("array");
   // Use "0" to indicate any element for now.
@@ -37,7 +38,7 @@ void JsonReader::iterArray(std::function<void()> fn) {
   }
   path_.pop_back();
 }
-void JsonReader::member(const char *name, std::function<void()> fn) {
+void JsonReader::member(const char *name, llvm::function_ref<void()> fn) {
   path_.push_back(name);
   auto it = m->FindMember(name);
   if (it != m->MemberEnd()) {
@@ -70,7 +71,7 @@ void JsonWriter::startObject() { m->StartObject(); }
 void JsonWriter::endObject() { m->EndObject(); }
 void JsonWriter::key(const char *name) { m->Key(name); }
 void JsonWriter::null_() { m->Null(); }
-void JsonWriter::int_(int v) { m->Int(v); }
+void JsonWriter::int64(int64_t v) { m->Int64(v); }
 void JsonWriter::string(const char *s) { m->String(s); }
 void JsonWriter::string(const char *s, size_t len) { m->String(s, len); }
 

src/serializer.hh

@@ -22,6 +22,7 @@
 namespace llvm {
 class CachedHashStringRef;
 class StringRef;
+template <typename Fn> class function_ref;
 } // namespace llvm
 
 namespace ccls {
@@ -36,8 +37,8 @@ struct JsonReader {
   JsonReader(rapidjson::Value *m) : m(m) {}
   void startObject() {}
   void endObject() {}
-  void iterArray(std::function<void()> fn);
+  void iterArray(llvm::function_ref<void()> fn);
-  void member(const char *name, std::function<void()> fn);
+  void member(const char *name, llvm::function_ref<void()> fn);
   bool isNull();
   std::string getString();
   std::string getPath() const;
@@ -57,7 +58,7 @@ struct JsonWriter {
   void endObject();
   void key(const char *name);
   void null_();
-  void int_(int v);
+  void int64(int64_t v);
   void string(const char *s);
   void string(const char *s, size_t len);
 };

src/working_files.cc

@@ -131,7 +131,7 @@ int alignColumn(const std::string &a, int column, std::string b, bool is_end) {
   if (column < head)
     return column;
   if ((int)a.size() - tail < column)
-    return column + b.size() - a.size();
+    return column + (int)b.size() - (int)a.size();
   if (std::max(a.size(), b.size()) - head - tail >= kMaxColumnAlignSize)
     return std::min(column, (int)b.size());