From 43677021e3c285c2ced2075b918da947e13fcb00 Mon Sep 17 00:00:00 2001 From: Vincent Ambo Date: Wed, 20 May 2020 22:58:43 +0100 Subject: refactor(3p/nix): Apply clang-tidy's performance-* fixes This applies the performance fixes listed here: https://clang.llvm.org/extra/clang-tidy/checks/list.html --- third_party/nix/src/libexpr/common-eval-args.cc | 2 +- third_party/nix/src/libexpr/eval.cc | 2 +- third_party/nix/src/libexpr/eval.hh | 2 +- third_party/nix/src/libexpr/get-drvs.cc | 2 +- third_party/nix/src/libexpr/get-drvs.hh | 2 +- third_party/nix/src/libexpr/primops.cc | 7 ++-- third_party/nix/src/libexpr/primops.hh | 2 +- third_party/nix/src/libmain/common-args.cc | 2 +- third_party/nix/src/libmain/shared.cc | 12 ++++--- third_party/nix/src/libmain/shared.hh | 7 ++-- third_party/nix/src/libstore/binary-cache-store.cc | 2 +- third_party/nix/src/libstore/binary-cache-store.hh | 2 +- third_party/nix/src/libstore/build.cc | 40 ++++++++++++---------- third_party/nix/src/libstore/crypto.cc | 4 +-- third_party/nix/src/libstore/derivations.cc | 4 +-- third_party/nix/src/libstore/derivations.hh | 2 +- third_party/nix/src/libstore/download.cc | 6 ++-- third_party/nix/src/libstore/download.hh | 2 +- third_party/nix/src/libstore/export-import.cc | 3 +- third_party/nix/src/libstore/globals.cc | 7 ++-- third_party/nix/src/libstore/local-fs-store.cc | 2 +- third_party/nix/src/libstore/machines.cc | 12 +++---- third_party/nix/src/libstore/machines.hh | 10 +++--- third_party/nix/src/libstore/misc.cc | 5 +-- third_party/nix/src/libstore/nar-accessor.cc | 8 ++--- third_party/nix/src/libstore/nar-accessor.hh | 4 +-- third_party/nix/src/libstore/nar-info.cc | 2 +- third_party/nix/src/libstore/profiles.cc | 7 ++-- third_party/nix/src/libstore/profiles.hh | 7 ++-- third_party/nix/src/libstore/remote-fs-accessor.cc | 5 +-- third_party/nix/src/libstore/remote-fs-accessor.hh | 4 +-- third_party/nix/src/libstore/store-api.cc | 17 ++++----- third_party/nix/src/libstore/store-api.hh | 6 ++-- third_party/nix/src/libutil/archive.cc | 2 +- third_party/nix/src/libutil/args.cc | 2 +- third_party/nix/src/libutil/config.cc | 4 +-- third_party/nix/src/libutil/serialise.cc | 4 +-- third_party/nix/src/libutil/serialise.hh | 3 +- third_party/nix/src/libutil/util.cc | 14 ++++---- third_party/nix/src/libutil/util.hh | 4 +-- third_party/nix/src/nix-build/nix-build.cc | 2 +- .../src/nix-collect-garbage/nix-collect-garbage.cc | 2 +- third_party/nix/src/nix-daemon/nix-daemon.cc | 6 ++-- third_party/nix/src/nix-env/nix-env.cc | 4 +-- third_party/nix/src/nix-store/dotgraph.cc | 2 +- third_party/nix/src/nix-store/dotgraph.hh | 2 +- third_party/nix/src/nix-store/graphml.cc | 2 +- third_party/nix/src/nix-store/graphml.hh | 2 +- third_party/nix/src/nix-store/nix-store.cc | 3 ++ third_party/nix/src/nix/cat.cc | 2 +- third_party/nix/src/nix/command.hh | 22 ++++++------ third_party/nix/src/nix/doctor.cc | 2 +- third_party/nix/src/nix/hash.cc | 4 +-- third_party/nix/src/nix/installables.cc | 29 +++++++++------- third_party/nix/src/nix/ls.cc | 2 +- third_party/nix/src/nix/repl.cc | 15 ++++---- third_party/nix/src/nix/run.cc | 4 +-- third_party/nix/src/nix/search.cc | 8 ++--- third_party/nix/src/nix/upgrade-nix.cc | 4 +-- third_party/nix/src/nix/verify.cc | 4 +-- 60 files changed, 189 insertions(+), 166 deletions(-) (limited to 'third_party') diff --git a/third_party/nix/src/libexpr/common-eval-args.cc b/third_party/nix/src/libexpr/common-eval-args.cc index 7a87841c9b77..33319cbabb17 100644 --- a/third_party/nix/src/libexpr/common-eval-args.cc +++ b/third_party/nix/src/libexpr/common-eval-args.cc @@ -29,7 +29,7 @@ MixEvalArgs::MixEvalArgs() { "add a path to the list of locations used to look up <...> file " "names") .label("path") - .handler([&](std::string s) { searchPath.push_back(s); }); + .handler([&](const std::string& s) { searchPath.push_back(s); }); } Bindings* MixEvalArgs::getAutoArgs(EvalState& state) { diff --git a/third_party/nix/src/libexpr/eval.cc b/third_party/nix/src/libexpr/eval.cc index 4be301167be9..4053dffe5185 100644 --- a/third_party/nix/src/libexpr/eval.cc +++ b/third_party/nix/src/libexpr/eval.cc @@ -298,7 +298,7 @@ static Strings parseNixPath(const string& s) { return res; } -EvalState::EvalState(const Strings& _searchPath, ref store) +EvalState::EvalState(const Strings& _searchPath, const ref& store) : sWith(symbols.create("")), sOutPath(symbols.create("outPath")), sDrvPath(symbols.create("drvPath")), diff --git a/third_party/nix/src/libexpr/eval.hh b/third_party/nix/src/libexpr/eval.hh index 48c67b62295d..48400d5f08e2 100644 --- a/third_party/nix/src/libexpr/eval.hh +++ b/third_party/nix/src/libexpr/eval.hh @@ -105,7 +105,7 @@ class EvalState { std::unordered_map resolvedPaths; public: - EvalState(const Strings& _searchPath, ref store); + EvalState(const Strings& _searchPath, const ref& store); ~EvalState(); void addToSearchPath(const string& s); diff --git a/third_party/nix/src/libexpr/get-drvs.cc b/third_party/nix/src/libexpr/get-drvs.cc index c4afc0fb45eb..39870abcbdfa 100644 --- a/third_party/nix/src/libexpr/get-drvs.cc +++ b/third_party/nix/src/libexpr/get-drvs.cc @@ -15,7 +15,7 @@ namespace nix { DrvInfo::DrvInfo(EvalState& state, string attrPath, Bindings* attrs) : state(&state), attrs(attrs), attrPath(std::move(attrPath)) {} -DrvInfo::DrvInfo(EvalState& state, ref store, +DrvInfo::DrvInfo(EvalState& state, const ref& store, const std::string& drvPathWithOutputs) : state(&state), attrPath("") { auto spec = parseDrvPathWithOutputs(drvPathWithOutputs); diff --git a/third_party/nix/src/libexpr/get-drvs.hh b/third_party/nix/src/libexpr/get-drvs.hh index ef6ecd253e6a..f0de0f67b8da 100644 --- a/third_party/nix/src/libexpr/get-drvs.hh +++ b/third_party/nix/src/libexpr/get-drvs.hh @@ -34,7 +34,7 @@ struct DrvInfo { DrvInfo(EvalState& state) : state(&state){}; DrvInfo(EvalState& state, string attrPath, Bindings* attrs); - DrvInfo(EvalState& state, ref store, + DrvInfo(EvalState& state, const ref& store, const std::string& drvPathWithOutputs); string queryName() const; diff --git a/third_party/nix/src/libexpr/primops.cc b/third_party/nix/src/libexpr/primops.cc index 7b73bec03525..307459396eae 100644 --- a/third_party/nix/src/libexpr/primops.cc +++ b/third_party/nix/src/libexpr/primops.cc @@ -34,7 +34,7 @@ namespace nix { name>. */ std::pair decodeContext(const string& s) { if (s.at(0) == '!') { - size_t index = s.find("!", 1); + size_t index = s.find('!', 1); return std::pair(string(s, index + 1), string(s, 1, index - 1)); } @@ -2172,7 +2172,7 @@ static void prim_splitVersion(EvalState& state, const Pos& pos, Value** args, unsigned int n = 0; for (auto& component : components) { auto listElem = v.listElems()[n++] = state.allocValue(); - mkString(*listElem, std::move(component)); + mkString(*listElem, component); } } @@ -2246,7 +2246,8 @@ static void prim_fetchTarball(EvalState& state, const Pos& pos, Value** args, RegisterPrimOp::PrimOps* RegisterPrimOp::primOps; -RegisterPrimOp::RegisterPrimOp(std::string name, size_t arity, PrimOpFun fun) { +RegisterPrimOp::RegisterPrimOp(const std::string& name, size_t arity, + PrimOpFun fun) { if (primOps == nullptr) { primOps = new PrimOps; } diff --git a/third_party/nix/src/libexpr/primops.hh b/third_party/nix/src/libexpr/primops.hh index 8e674509c7c4..6abd0508a09b 100644 --- a/third_party/nix/src/libexpr/primops.hh +++ b/third_party/nix/src/libexpr/primops.hh @@ -11,7 +11,7 @@ struct RegisterPrimOp { /* You can register a constant by passing an arity of 0. fun will get called during EvalState initialization, so there may be primops not yet added and builtins is not yet sorted. */ - RegisterPrimOp(std::string name, size_t arity, PrimOpFun fun); + RegisterPrimOp(const std::string& name, size_t arity, PrimOpFun fun); }; /* These primops are disabled without enableNativeCode, but plugins diff --git a/third_party/nix/src/libmain/common-args.cc b/third_party/nix/src/libmain/common-args.cc index 9d6f6445c0e7..3d9b8ebfaefd 100644 --- a/third_party/nix/src/libmain/common-args.cc +++ b/third_party/nix/src/libmain/common-args.cc @@ -26,7 +26,7 @@ MixCommonArgs::MixCommonArgs(const string& programName) .shortName('j') .label("jobs") .description("maximum number of parallel builds") - .handler([=](std::string s) { settings.set("max-jobs", s); }); + .handler([=](const std::string& s) { settings.set("max-jobs", s); }); std::string cat = "config"; globalConfig.convertToArgs(*this, cat); diff --git a/third_party/nix/src/libmain/shared.cc b/third_party/nix/src/libmain/shared.cc index 5c2084691144..f6c80cae30ef 100644 --- a/third_party/nix/src/libmain/shared.cc +++ b/third_party/nix/src/libmain/shared.cc @@ -36,7 +36,7 @@ void printGCWarning() { } } -void printMissing(ref store, const PathSet& paths) { +void printMissing(const ref& store, const PathSet& paths) { unsigned long long downloadSize; unsigned long long narSize; PathSet willBuild; @@ -48,7 +48,7 @@ void printMissing(ref store, const PathSet& paths) { narSize); } -void printMissing(ref store, const PathSet& willBuild, +void printMissing(const ref& store, const PathSet& willBuild, const PathSet& willSubstitute, const PathSet& unknown, unsigned long long downloadSize, unsigned long long narSize) { if (!willBuild.empty()) { @@ -260,14 +260,15 @@ void parseCmdLine( int argc, char** argv, std::function parseArg) { - parseCmdLine(baseNameOf(argv[0]), argvToStrings(argc, argv), parseArg); + parseCmdLine(baseNameOf(argv[0]), argvToStrings(argc, argv), + std::move(parseArg)); } void parseCmdLine( const string& programName, const Strings& args, std::function parseArg) { - LegacyArgs(programName, parseArg).parseCmdline(args); + LegacyArgs(programName, std::move(parseArg)).parseCmdline(args); } void printVersion(const string& programName) { @@ -298,7 +299,8 @@ void showManPage(const string& name) { throw SysError(format("command 'man %1%' failed") % name.c_str()); } -int handleExceptions(const string& programName, std::function fun) { +int handleExceptions(const string& programName, + const std::function& fun) { ReceiveInterrupts receiveInterrupts; // FIXME: need better place for this string error = ANSI_RED "error:" ANSI_NORMAL " "; diff --git a/third_party/nix/src/libmain/shared.hh b/third_party/nix/src/libmain/shared.hh index db236a701778..da1c9cc66ac3 100644 --- a/third_party/nix/src/libmain/shared.hh +++ b/third_party/nix/src/libmain/shared.hh @@ -18,7 +18,8 @@ class Exit : public std::exception { virtual ~Exit(); }; -int handleExceptions(const string& programName, std::function fun); +int handleExceptions(const string& programName, + const std::function& fun); /* Don't forget to call initPlugins() after settings are initialized! */ void initNix(); @@ -40,9 +41,9 @@ void printGCWarning(); class Store; -void printMissing(ref store, const PathSet& paths); +void printMissing(const ref& store, const PathSet& paths); -void printMissing(ref store, const PathSet& willBuild, +void printMissing(const ref& store, const PathSet& willBuild, const PathSet& willSubstitute, const PathSet& unknown, unsigned long long downloadSize, unsigned long long narSize); diff --git a/third_party/nix/src/libstore/binary-cache-store.cc b/third_party/nix/src/libstore/binary-cache-store.cc index b0d8c81d80e4..487c12b5313e 100644 --- a/third_party/nix/src/libstore/binary-cache-store.cc +++ b/third_party/nix/src/libstore/binary-cache-store.cc @@ -98,7 +98,7 @@ Path BinaryCacheStore::narInfoFileFor(const Path& storePath) { return storePathToHash(storePath) + ".narinfo"; } -void BinaryCacheStore::writeNarInfo(ref narInfo) { +void BinaryCacheStore::writeNarInfo(const ref& narInfo) { auto narInfoFile = narInfoFileFor(narInfo->path); upsertFile(narInfoFile, narInfo->to_string(), "text/x-nix-narinfo"); diff --git a/third_party/nix/src/libstore/binary-cache-store.hh b/third_party/nix/src/libstore/binary-cache-store.hh index cd88ccdd8433..f5bd66bbd6e1 100644 --- a/third_party/nix/src/libstore/binary-cache-store.hh +++ b/third_party/nix/src/libstore/binary-cache-store.hh @@ -65,7 +65,7 @@ class BinaryCacheStore : public Store { std::string narInfoFileFor(const Path& storePath); - void writeNarInfo(ref narInfo); + void writeNarInfo(const ref& narInfo); public: bool isValidPathUncached(const Path& path) override; diff --git a/third_party/nix/src/libstore/build.cc b/third_party/nix/src/libstore/build.cc index 389c99f06c01..fe6463584759 100644 --- a/third_party/nix/src/libstore/build.cc +++ b/third_party/nix/src/libstore/build.cc @@ -145,7 +145,7 @@ class Goal : public std::enable_shared_from_this { public: virtual void work() = 0; - void addWaitee(GoalPtr waitee); + void addWaitee(const GoalPtr& waitee); virtual void waiteeDone(GoalPtr waitee, ExitCode result); @@ -280,10 +280,10 @@ class Worker { RepairFlag repair = NoRepair); /* Remove a dead goal. */ - void removeGoal(GoalPtr goal); + void removeGoal(const GoalPtr& goal); /* Wake up a goal (i.e., there is something for it to do). */ - void wakeUp(GoalPtr goal); + void wakeUp(const GoalPtr& goal); /* Return the number of local build and substitution processes currently running (but not remote builds via the build @@ -292,7 +292,7 @@ class Worker { /* Registers a running child process. `inBuildSlot' means that the process counts towards the jobs limit. */ - void childStarted(GoalPtr goal, const set& fds, bool inBuildSlot, + void childStarted(const GoalPtr& goal, const set& fds, bool inBuildSlot, bool respectTimeouts); /* Unregisters a running child process. `wakeSleepers' should be @@ -303,7 +303,7 @@ class Worker { /* Put `goal' to sleep until a build slot becomes available (which might be right away). */ - void waitForBuildSlot(GoalPtr goal); + void waitForBuildSlot(const GoalPtr& goal); /* Wait for any goal to finish. Pretty indiscriminate way to wait for some resource that some other goal is holding. */ @@ -332,7 +332,7 @@ class Worker { ////////////////////////////////////////////////////////////////////// -void addToWeakGoals(WeakGoals& goals, GoalPtr p) { +void addToWeakGoals(WeakGoals& goals, const GoalPtr& p) { // FIXME: necessary? // FIXME: O(n) for (auto& i : goals) { @@ -343,7 +343,7 @@ void addToWeakGoals(WeakGoals& goals, GoalPtr p) { goals.push_back(p); } -void Goal::addWaitee(GoalPtr waitee) { +void Goal::addWaitee(const GoalPtr& waitee) { waitees.insert(waitee); addToWeakGoals(waitee->waiters, shared_from_this()); } @@ -445,7 +445,9 @@ void handleDiffHook(uid_t uid, uid_t gid, Path tryA, Path tryB, Path drvPath, auto diffHook = settings.diffHook; if (diffHook != "" && settings.runDiffHook) { try { - RunOptions diffHookOptions(diffHook, {tryA, tryB, drvPath, tmpDir}); + RunOptions diffHookOptions( + diffHook, {std::move(tryA), std::move(tryB), std::move(drvPath), + std::move(tmpDir)}); diffHookOptions.searchPath = true; diffHookOptions.uid = uid; diffHookOptions.gid = gid; @@ -979,7 +981,7 @@ class DerivationGoal : public Goal { void done(BuildResult::Status status, const string& msg = ""); - PathSet exportReferences(PathSet storePaths); + PathSet exportReferences(const PathSet& storePaths); }; const Path DerivationGoal::homeDir = "/homeless-shelter"; @@ -1520,7 +1522,7 @@ void DerivationGoal::tryToBuild() { started(); } -void replaceValidPath(const Path& storePath, const Path tmpPath) { +void replaceValidPath(const Path& storePath, const Path& tmpPath) { /* We can't atomically replace storePath (the original) with tmpPath (the replacement), so we have to move it out of the way first. We'd better not be interrupted here, because if @@ -1840,7 +1842,7 @@ int childEntry(void* arg) { return 1; } -PathSet DerivationGoal::exportReferences(PathSet storePaths) { +PathSet DerivationGoal::exportReferences(const PathSet& storePaths) { PathSet paths; for (auto storePath : storePaths) { @@ -4455,7 +4457,7 @@ GoalPtr Worker::makeSubstitutionGoal(const Path& path, RepairFlag repair) { return goal; } -static void removeGoal(GoalPtr goal, WeakGoalMap& goalMap) { +static void removeGoal(const GoalPtr& goal, WeakGoalMap& goalMap) { /* !!! inefficient */ for (auto i = goalMap.begin(); i != goalMap.end();) { if (i->second.lock() == goal) { @@ -4469,7 +4471,7 @@ static void removeGoal(GoalPtr goal, WeakGoalMap& goalMap) { } } -void Worker::removeGoal(GoalPtr goal) { +void Worker::removeGoal(const GoalPtr& goal) { nix::removeGoal(goal, derivationGoals); nix::removeGoal(goal, substitutionGoals); if (topGoals.find(goal) != topGoals.end()) { @@ -4492,15 +4494,15 @@ void Worker::removeGoal(GoalPtr goal) { waitingForAnyGoal.clear(); } -void Worker::wakeUp(GoalPtr goal) { +void Worker::wakeUp(const GoalPtr& goal) { goal->trace("woken up"); addToWeakGoals(awake, goal); } unsigned Worker::getNrLocalBuilds() { return nrLocalBuilds; } -void Worker::childStarted(GoalPtr goal, const set& fds, bool inBuildSlot, - bool respectTimeouts) { +void Worker::childStarted(const GoalPtr& goal, const set& fds, + bool inBuildSlot, bool respectTimeouts) { Child child; child.goal = goal; child.goal2 = goal.get(); @@ -4542,7 +4544,7 @@ void Worker::childTerminated(Goal* goal, bool wakeSleepers) { } } -void Worker::waitForBuildSlot(GoalPtr goal) { +void Worker::waitForBuildSlot(const GoalPtr& goal) { DLOG(INFO) << "wait for build slot"; if (getNrLocalBuilds() < settings.maxBuildJobs) { wakeUp(goal); /* we can do it right away */ @@ -4553,12 +4555,12 @@ void Worker::waitForBuildSlot(GoalPtr goal) { void Worker::waitForAnyGoal(GoalPtr goal) { DLOG(INFO) << "wait for any goal"; - addToWeakGoals(waitingForAnyGoal, goal); + addToWeakGoals(waitingForAnyGoal, std::move(goal)); } void Worker::waitForAWhile(GoalPtr goal) { DLOG(INFO) << "wait for a while"; - addToWeakGoals(waitingForAWhile, goal); + addToWeakGoals(waitingForAWhile, std::move(goal)); } void Worker::run(const Goals& _topGoals) { diff --git a/third_party/nix/src/libstore/crypto.cc b/third_party/nix/src/libstore/crypto.cc index 2fbc30fb3161..90580b8dc902 100644 --- a/third_party/nix/src/libstore/crypto.cc +++ b/third_party/nix/src/libstore/crypto.cc @@ -104,12 +104,12 @@ PublicKeys getDefaultPublicKeys() { // FIXME: filter duplicates - for (auto s : settings.trustedPublicKeys.get()) { + for (const auto& s : settings.trustedPublicKeys.get()) { PublicKey key(s); publicKeys.emplace(key.name, key); } - for (auto secretKeyFile : settings.secretKeyFiles.get()) { + for (const auto& secretKeyFile : settings.secretKeyFiles.get()) { try { SecretKey secretKey(readFile(secretKeyFile)); publicKeys.emplace(secretKey.name, secretKey.toPublicKey()); diff --git a/third_party/nix/src/libstore/derivations.cc b/third_party/nix/src/libstore/derivations.cc index 029c14640b7a..9c71d7209e37 100644 --- a/third_party/nix/src/libstore/derivations.cc +++ b/third_party/nix/src/libstore/derivations.cc @@ -38,7 +38,7 @@ bool BasicDerivation::isBuiltin() const { return string(builder, 0, 8) == "builtin:"; } -Path writeDerivation(ref store, const Derivation& drv, +Path writeDerivation(const ref& store, const Derivation& drv, const string& name, RepairFlag repair) { PathSet references; references.insert(drv.inputSrcs.begin(), drv.inputSrcs.end()); @@ -355,7 +355,7 @@ Hash hashDerivationModulo(Store& store, Derivation drv) { } DrvPathWithOutputs parseDrvPathWithOutputs(const string& s) { - size_t n = s.find("!"); + size_t n = s.find('!'); return n == std::string::npos ? DrvPathWithOutputs(s, std::set()) : DrvPathWithOutputs( diff --git a/third_party/nix/src/libstore/derivations.hh b/third_party/nix/src/libstore/derivations.hh index 11277c813e20..52b951f5e9d9 100644 --- a/third_party/nix/src/libstore/derivations.hh +++ b/third_party/nix/src/libstore/derivations.hh @@ -67,7 +67,7 @@ struct Derivation : BasicDerivation { class Store; /* Write a derivation to the Nix store, and return its path. */ -Path writeDerivation(ref store, const Derivation& drv, +Path writeDerivation(const ref& store, const Derivation& drv, const string& name, RepairFlag repair = NoRepair); /* Read a derivation from a file. */ diff --git a/third_party/nix/src/libstore/download.cc b/third_party/nix/src/libstore/download.cc index 752f546a584b..7476dd50b579 100644 --- a/third_party/nix/src/libstore/download.cc +++ b/third_party/nix/src/libstore/download.cc @@ -130,7 +130,7 @@ struct CurlDownloader : public Downloader { } } - void failEx(std::exception_ptr ex) { + void failEx(const std::exception_ptr& ex) { assert(!done); done = true; callback.rethrow(ex); @@ -663,7 +663,7 @@ struct CurlDownloader : public Downloader { } } - void enqueueItem(std::shared_ptr item) { + void enqueueItem(const std::shared_ptr& item) { if (item->request.data && !hasPrefix(item->request.uri, "http://") && !hasPrefix(item->request.uri, "https://")) { throw nix::Error("uploading to '%s' is not supported", item->request.uri); @@ -858,7 +858,7 @@ void Downloader::download(DownloadRequest&& request, Sink& sink) { } CachedDownloadResult Downloader::downloadCached( - ref store, const CachedDownloadRequest& request) { + const ref& store, const CachedDownloadRequest& request) { auto url = resolveUri(request.uri); auto name = request.name; diff --git a/third_party/nix/src/libstore/download.hh b/third_party/nix/src/libstore/download.hh index 5e7bbf4ad669..9ddbdfc15955 100644 --- a/third_party/nix/src/libstore/download.hh +++ b/third_party/nix/src/libstore/download.hh @@ -108,7 +108,7 @@ struct Downloader { and is more recent than ‘tarball-ttl’ seconds. Otherwise, use the recorded ETag to verify if the server has a more recent version, and if so, download it to the Nix store. */ - CachedDownloadResult downloadCached(ref store, + CachedDownloadResult downloadCached(const ref& store, const CachedDownloadRequest& request); enum Error { NotFound, Forbidden, Misc, Transient, Interrupted }; diff --git a/third_party/nix/src/libstore/export-import.cc b/third_party/nix/src/libstore/export-import.cc index 1974773ba135..077b0d539001 100644 --- a/third_party/nix/src/libstore/export-import.cc +++ b/third_party/nix/src/libstore/export-import.cc @@ -55,7 +55,8 @@ void Store::exportPath(const Path& path, Sink& sink) { << 0; } -Paths Store::importPaths(Source& source, std::shared_ptr accessor, +Paths Store::importPaths(Source& source, + const std::shared_ptr& accessor, CheckSigsFlag checkSigs) { Paths res; while (true) { diff --git a/third_party/nix/src/libstore/globals.cc b/third_party/nix/src/libstore/globals.cc index 57ba8cff6434..066de443653b 100644 --- a/third_party/nix/src/libstore/globals.cc +++ b/third_party/nix/src/libstore/globals.cc @@ -149,17 +149,18 @@ void BaseSetting::convertToArg(Args& args, args.mkFlag() .longName(name) .description("Enable sandboxing.") - .handler([=](std::vector ss) { override(smEnabled); }) + .handler([=](const std::vector& ss) { override(smEnabled); }) .category(category); args.mkFlag() .longName("no-" + name) .description("Disable sandboxing.") - .handler([=](std::vector ss) { override(smDisabled); }) + .handler( + [=](const std::vector& ss) { override(smDisabled); }) .category(category); args.mkFlag() .longName("relaxed-" + name) .description("Enable sandboxing, but allow builds to disable it.") - .handler([=](std::vector ss) { override(smRelaxed); }) + .handler([=](const std::vector& ss) { override(smRelaxed); }) .category(category); } diff --git a/third_party/nix/src/libstore/local-fs-store.cc b/third_party/nix/src/libstore/local-fs-store.cc index 9a5ea153869a..2120afc0ce7c 100644 --- a/third_party/nix/src/libstore/local-fs-store.cc +++ b/third_party/nix/src/libstore/local-fs-store.cc @@ -12,7 +12,7 @@ LocalFSStore::LocalFSStore(const Params& params) : Store(params) {} struct LocalStoreAccessor : public FSAccessor { ref store; - explicit LocalStoreAccessor(ref store) : store(store) {} + explicit LocalStoreAccessor(const ref& store) : store(store) {} Path toRealPath(const Path& path) { Path storePath = store->toStorePath(path); diff --git a/third_party/nix/src/libstore/machines.cc b/third_party/nix/src/libstore/machines.cc index 1b8bc4cf75a5..5b0c7e72ddba 100644 --- a/third_party/nix/src/libstore/machines.cc +++ b/third_party/nix/src/libstore/machines.cc @@ -9,12 +9,12 @@ namespace nix { -Machine::Machine(decltype(storeUri) storeUri, decltype(systemTypes) systemTypes, - decltype(sshKey) sshKey, decltype(maxJobs) maxJobs, - decltype(speedFactor) speedFactor, - decltype(supportedFeatures) supportedFeatures, - decltype(mandatoryFeatures) mandatoryFeatures, - decltype(sshPublicHostKey) sshPublicHostKey) +Machine::Machine(decltype(storeUri)& storeUri, + decltype(systemTypes)& systemTypes, decltype(sshKey)& sshKey, + decltype(maxJobs) maxJobs, decltype(speedFactor) speedFactor, + decltype(supportedFeatures)& supportedFeatures, + decltype(mandatoryFeatures)& mandatoryFeatures, + decltype(sshPublicHostKey)& sshPublicHostKey) : storeUri( // Backwards compatibility: if the URI is a hostname, // prepend ssh://. diff --git a/third_party/nix/src/libstore/machines.hh b/third_party/nix/src/libstore/machines.hh index a5013fe4f151..aef8054c250e 100644 --- a/third_party/nix/src/libstore/machines.hh +++ b/third_party/nix/src/libstore/machines.hh @@ -19,12 +19,12 @@ struct Machine { bool mandatoryMet(const std::set& features) const; - Machine(decltype(storeUri) storeUri, decltype(systemTypes) systemTypes, - decltype(sshKey) sshKey, decltype(maxJobs) maxJobs, + Machine(decltype(storeUri)& storeUri, decltype(systemTypes)& systemTypes, + decltype(sshKey)& sshKey, decltype(maxJobs) maxJobs, decltype(speedFactor) speedFactor, - decltype(supportedFeatures) supportedFeatures, - decltype(mandatoryFeatures) mandatoryFeatures, - decltype(sshPublicHostKey) sshPublicHostKey); + decltype(supportedFeatures)& supportedFeatures, + decltype(mandatoryFeatures)& mandatoryFeatures, + decltype(sshPublicHostKey)& sshPublicHostKey); }; typedef std::vector Machines; diff --git a/third_party/nix/src/libstore/misc.cc b/third_party/nix/src/libstore/misc.cc index 076c691b004c..f95c4bec8b0c 100644 --- a/third_party/nix/src/libstore/misc.cc +++ b/third_party/nix/src/libstore/misc.cc @@ -171,8 +171,9 @@ void Store::queryMissing(const PathSet& targets, PathSet& willBuild_, } }; - auto checkOutput = [&](const Path& drvPath, ref drv, - const Path& outPath, ref> drvState_) { + auto checkOutput = [&](const Path& drvPath, const ref& drv, + const Path& outPath, + const ref>& drvState_) { if (drvState_->lock()->done) { return; } diff --git a/third_party/nix/src/libstore/nar-accessor.cc b/third_party/nix/src/libstore/nar-accessor.cc index 3a8d6b884104..7f6e0377aed4 100644 --- a/third_party/nix/src/libstore/nar-accessor.cc +++ b/third_party/nix/src/libstore/nar-accessor.cc @@ -94,7 +94,7 @@ struct NarAccessor : public FSAccessor { } }; - explicit NarAccessor(ref nar) : nar(nar) { + explicit NarAccessor(const ref& nar) : nar(nar) { NarIndexer indexer(*this, *nar); parseDump(indexer, indexer); } @@ -111,7 +111,7 @@ struct NarAccessor : public FSAccessor { if (type == "directory") { member.type = FSAccessor::Type::tDirectory; for (auto i = v["entries"].begin(); i != v["entries"].end(); ++i) { - std::string name = i.key(); + const std::string& name = i.key(); recurse(member.children[name], i.value()); } } else if (type == "regular") { @@ -225,8 +225,8 @@ ref makeLazyNarAccessor(const std::string& listing, return make_ref(listing, getNarBytes); } -void listNar(JSONPlaceholder& res, ref accessor, const Path& path, - bool recurse) { +void listNar(JSONPlaceholder& res, const ref& accessor, + const Path& path, bool recurse) { auto st = accessor->stat(path); auto obj = res.object(); diff --git a/third_party/nix/src/libstore/nar-accessor.hh b/third_party/nix/src/libstore/nar-accessor.hh index d4bd7366f87d..8b18d4a36e06 100644 --- a/third_party/nix/src/libstore/nar-accessor.hh +++ b/third_party/nix/src/libstore/nar-accessor.hh @@ -23,7 +23,7 @@ class JSONPlaceholder; /* Write a JSON representation of the contents of a NAR (except file contents). */ -void listNar(JSONPlaceholder& res, ref accessor, const Path& path, - bool recurse); +void listNar(JSONPlaceholder& res, const ref& accessor, + const Path& path, bool recurse); } // namespace nix diff --git a/third_party/nix/src/libstore/nar-info.cc b/third_party/nix/src/libstore/nar-info.cc index 6190840ea61c..91cf471554f3 100644 --- a/third_party/nix/src/libstore/nar-info.cc +++ b/third_party/nix/src/libstore/nar-info.cc @@ -122,7 +122,7 @@ std::string NarInfo::to_string() const { res += "System: " + system + "\n"; } - for (auto sig : sigs) { + for (const auto& sig : sigs) { res += "Sig: " + sig + "\n"; } diff --git a/third_party/nix/src/libstore/profiles.cc b/third_party/nix/src/libstore/profiles.cc index 7d16d9f62390..bdb90b7388f7 100644 --- a/third_party/nix/src/libstore/profiles.cc +++ b/third_party/nix/src/libstore/profiles.cc @@ -35,7 +35,7 @@ static int parseName(const string& profileName, const string& name) { return -1; } -Generations findGenerations(Path profile, int& curGen) { +Generations findGenerations(const Path& profile, int& curGen) { Generations gens; Path profileDir = dirOf(profile); @@ -68,7 +68,8 @@ static void makeName(const Path& profile, unsigned int num, Path& outLink) { outLink = prefix + "-link"; } -Path createGeneration(ref store, Path profile, Path outPath) { +Path createGeneration(const ref& store, const Path& profile, + const Path& outPath) { /* The new generation number should be higher than old the previous ones. */ int dummy; @@ -226,7 +227,7 @@ void deleteGenerationsOlderThan(const Path& profile, const string& timeSpec, deleteGenerationsOlderThan(profile, oldTime, dryRun); } -void switchLink(Path link, Path target) { +void switchLink(const Path& link, Path target) { /* Hacky. */ if (dirOf(target) == dirOf(link)) { target = baseNameOf(target); diff --git a/third_party/nix/src/libstore/profiles.hh b/third_party/nix/src/libstore/profiles.hh index b1c3f0948713..ab31cc993af5 100644 --- a/third_party/nix/src/libstore/profiles.hh +++ b/third_party/nix/src/libstore/profiles.hh @@ -19,11 +19,12 @@ typedef list Generations; /* Returns the list of currently present generations for the specified profile, sorted by generation number. */ -Generations findGenerations(Path profile, int& curGen); +Generations findGenerations(const Path& profile, int& curGen); class LocalFSStore; -Path createGeneration(ref store, Path profile, Path outPath); +Path createGeneration(const ref& store, const Path& profile, + const Path& outPath); void deleteGeneration(const Path& profile, unsigned int gen); @@ -40,7 +41,7 @@ void deleteGenerationsOlderThan(const Path& profile, time_t t, bool dryRun); void deleteGenerationsOlderThan(const Path& profile, const string& timeSpec, bool dryRun); -void switchLink(Path link, Path target); +void switchLink(const Path& link, Path target); /* Ensure exclusive access to a profile. Any command that modifies the profile first acquires this lock. */ diff --git a/third_party/nix/src/libstore/remote-fs-accessor.cc b/third_party/nix/src/libstore/remote-fs-accessor.cc index 9aceaa9815b5..ca478c213fde 100644 --- a/third_party/nix/src/libstore/remote-fs-accessor.cc +++ b/third_party/nix/src/libstore/remote-fs-accessor.cc @@ -9,7 +9,8 @@ namespace nix { -RemoteFSAccessor::RemoteFSAccessor(ref store, const Path& cacheDir) +RemoteFSAccessor::RemoteFSAccessor(const ref& store, + const Path& cacheDir) : store(store), cacheDir(cacheDir) { if (!cacheDir.empty()) { createDirs(cacheDir); @@ -23,7 +24,7 @@ Path RemoteFSAccessor::makeCacheFile(const Path& storePath, } void RemoteFSAccessor::addToCache(const Path& storePath, const std::string& nar, - ref narAccessor) { + const ref& narAccessor) { nars.emplace(storePath, narAccessor); if (!cacheDir.empty()) { diff --git a/third_party/nix/src/libstore/remote-fs-accessor.hh b/third_party/nix/src/libstore/remote-fs-accessor.hh index 08985de5291f..32c729f50dcf 100644 --- a/third_party/nix/src/libstore/remote-fs-accessor.hh +++ b/third_party/nix/src/libstore/remote-fs-accessor.hh @@ -20,10 +20,10 @@ class RemoteFSAccessor : public FSAccessor { Path makeCacheFile(const Path& storePath, const std::string& ext); void addToCache(const Path& storePath, const std::string& nar, - ref narAccessor); + const ref& narAccessor); public: - RemoteFSAccessor(ref store, + RemoteFSAccessor(const ref& store, const /* FIXME: use std::optional */ Path& cacheDir = ""); Stat stat(const Path& path) override; diff --git a/third_party/nix/src/libstore/store-api.cc b/third_party/nix/src/libstore/store-api.cc index 39cb5ca9f6c8..50e9b3b6ea86 100644 --- a/third_party/nix/src/libstore/store-api.cc +++ b/third_party/nix/src/libstore/store-api.cc @@ -1,6 +1,7 @@ #include "store-api.hh" #include +#include #include @@ -570,7 +571,7 @@ void Store::buildPaths(const PathSet& paths, BuildMode buildMode) { } } -void copyStorePath(ref srcStore, ref dstStore, +void copyStorePath(ref srcStore, const ref& dstStore, const Path& storePath, RepairFlag repair, CheckSigsFlag checkSigs) { auto srcUri = srcStore->getUri(); @@ -693,7 +694,7 @@ void copyPaths(ref srcStore, ref dstStore, }); } -void copyClosure(ref srcStore, ref dstStore, +void copyClosure(const ref& srcStore, const ref& dstStore, const PathSet& storePaths, RepairFlag repair, CheckSigsFlag checkSigs, SubstituteFlag substitute) { PathSet closure; @@ -824,14 +825,14 @@ void Store::addToStore(const ValidPathInfo& info, Source& narSource, RepairFlag repair, CheckSigsFlag checkSigs, std::shared_ptr accessor) { addToStore(info, make_ref(narSource.drain()), repair, checkSigs, - accessor); + std::move(accessor)); } void Store::addToStore(const ValidPathInfo& info, const ref& nar, RepairFlag repair, CheckSigsFlag checkSigs, std::shared_ptr accessor) { StringSource source(*nar); - addToStore(info, source, repair, checkSigs, accessor); + addToStore(info, source, repair, checkSigs, std::move(accessor)); } } // namespace nix @@ -851,7 +852,7 @@ std::pair splitUriAndParams( Store::Params params; auto q = uri.find('?'); if (q != std::string::npos) { - for (auto s : tokenizeString(uri.substr(q + 1), "&")) { + for (const auto& s : tokenizeString(uri.substr(q + 1), "&")) { auto e = s.find('='); if (e != std::string::npos) { auto value = s.substr(e + 1); @@ -885,7 +886,7 @@ ref openStore(const std::string& uri_, auto params = extraParams; params.insert(uriParams.begin(), uriParams.end()); - for (auto fun : *RegisterStoreImplementation::implementations) { + for (const auto& fun : *RegisterStoreImplementation::implementations) { auto store = fun(uri, params); if (store) { store->warnUnknownSettings(); @@ -952,11 +953,11 @@ std::list> getDefaultSubstituters() { } }; - for (auto uri : settings.substituters.get()) { + for (const auto& uri : settings.substituters.get()) { addStore(uri); } - for (auto uri : settings.extraSubstituters.get()) { + for (const auto& uri : settings.extraSubstituters.get()) { addStore(uri); } diff --git a/third_party/nix/src/libstore/store-api.hh b/third_party/nix/src/libstore/store-api.hh index 17299585edaf..242a3e65621e 100644 --- a/third_party/nix/src/libstore/store-api.hh +++ b/third_party/nix/src/libstore/store-api.hh @@ -564,7 +564,7 @@ class Store : public std::enable_shared_from_this, public Config { the Nix store. Optionally, the contents of the NARs are preloaded into the specified FS accessor to speed up subsequent access. */ - Paths importPaths(Source& source, std::shared_ptr accessor, + Paths importPaths(Source& source, const std::shared_ptr& accessor, CheckSigsFlag checkSigs = CheckSigs); struct Stats { @@ -671,7 +671,7 @@ string storePathToHash(const Path& path); void checkStoreName(const string& name); /* Copy a path from one store to another. */ -void copyStorePath(ref srcStore, ref dstStore, +void copyStorePath(ref srcStore, const ref& dstStore, const Path& storePath, RepairFlag repair = NoRepair, CheckSigsFlag checkSigs = CheckSigs); @@ -686,7 +686,7 @@ void copyPaths(ref srcStore, ref dstStore, SubstituteFlag substitute = NoSubstitute); /* Copy the closure of the specified paths from one store to another. */ -void copyClosure(ref srcStore, ref dstStore, +void copyClosure(const ref& srcStore, const ref& dstStore, const PathSet& storePaths, RepairFlag repair = NoRepair, CheckSigsFlag checkSigs = CheckSigs, SubstituteFlag substitute = NoSubstitute); diff --git a/third_party/nix/src/libutil/archive.cc b/third_party/nix/src/libutil/archive.cc index 6583dc20b899..32bad07f22da 100644 --- a/third_party/nix/src/libutil/archive.cc +++ b/third_party/nix/src/libutil/archive.cc @@ -145,7 +145,7 @@ void dumpString(const std::string& s, Sink& sink) { << "contents" << s << ")"; } -static SerialisationError badArchive(string s) { +static SerialisationError badArchive(const string& s) { return SerialisationError("bad archive: " + s); } diff --git a/third_party/nix/src/libutil/args.cc b/third_party/nix/src/libutil/args.cc index 2b4407480254..48fa715fdf0f 100644 --- a/third_party/nix/src/libutil/args.cc +++ b/third_party/nix/src/libutil/args.cc @@ -175,7 +175,7 @@ Args::FlagMaker& Args::FlagMaker::mkHashTypeFlag(HashType* ht) { arity(1); label("type"); description("hash algorithm ('md5', 'sha1', 'sha256', or 'sha512')"); - handler([ht](std::string s) { + handler([ht](const std::string& s) { *ht = parseHashType(s); if (*ht == htUnknown) { throw UsageError("unknown hash type '%1%'", s); diff --git a/third_party/nix/src/libutil/config.cc b/third_party/nix/src/libutil/config.cc index ef47c167ed49..828ee1811bdb 100644 --- a/third_party/nix/src/libutil/config.cc +++ b/third_party/nix/src/libutil/config.cc @@ -246,12 +246,12 @@ void BaseSetting::convertToArg(Args& args, const std::string& category) { args.mkFlag() .longName(name) .description(description) - .handler([=](std::vector ss) { override(true); }) + .handler([=](const std::vector& ss) { override(true); }) .category(category); args.mkFlag() .longName("no-" + name) .description(description) - .handler([=](std::vector ss) { override(false); }) + .handler([=](const std::vector& ss) { override(false); }) .category(category); } diff --git a/third_party/nix/src/libutil/serialise.cc b/third_party/nix/src/libutil/serialise.cc index 4b12367196f1..34af4e840ab6 100644 --- a/third_party/nix/src/libutil/serialise.cc +++ b/third_party/nix/src/libutil/serialise.cc @@ -157,8 +157,8 @@ size_t StringSource::read(unsigned char* data, size_t len) { #error Coroutines are broken in this version of Boost! #endif -std::unique_ptr sinkToSource(std::function fun, - std::function eof) { +std::unique_ptr sinkToSource(const std::function& fun, + const std::function& eof) { struct SinkToSource : Source { using coro_t = boost::coroutines2::coroutine; diff --git a/third_party/nix/src/libutil/serialise.hh b/third_party/nix/src/libutil/serialise.hh index d8c8b181f411..a5992b50ec6e 100644 --- a/third_party/nix/src/libutil/serialise.hh +++ b/third_party/nix/src/libutil/serialise.hh @@ -209,7 +209,8 @@ struct LambdaSource : Source { /* Convert a function that feeds data into a Sink into a Source. The Source executes the function as a coroutine. */ std::unique_ptr sinkToSource( - std::function fun, std::function eof = []() { + const std::function& fun, + const std::function& eof = []() { throw EndOfFile("coroutine has finished"); }); diff --git a/third_party/nix/src/libutil/util.cc b/third_party/nix/src/libutil/util.cc index 21da06b8ab1e..b13dd7e4f975 100644 --- a/third_party/nix/src/libutil/util.cc +++ b/third_party/nix/src/libutil/util.cc @@ -76,9 +76,9 @@ void clearEnv() { } } -void replaceEnv(std::map newEnv) { +void replaceEnv(const std::map& newEnv) { clearEnv(); - for (auto newEnvVar : newEnv) { + for (const auto& newEnvVar : newEnv) { setenv(newEnvVar.first.c_str(), newEnvVar.second.c_str(), 1); } } @@ -888,9 +888,9 @@ void killUser(uid_t uid) { /* Wrapper around vfork to prevent the child process from clobbering the caller's stack frame in the parent. */ -static pid_t doFork(bool allowVfork, std::function fun) +static pid_t doFork(bool allowVfork, const std::function& fun) __attribute__((noinline)); -static pid_t doFork(bool allowVfork, std::function fun) { +static pid_t doFork(bool allowVfork, const std::function& fun) { #ifdef __linux__ pid_t pid = allowVfork ? vfork() : fork(); #else @@ -944,7 +944,7 @@ std::vector stringsToCharPtrs(const Strings& ss) { return res; } -string runProgram(Path program, bool searchPath, const Strings& args, +string runProgram(const Path& program, bool searchPath, const Strings& args, const std::optional& input) { RunOptions opts(program, args); opts.searchPath = searchPath; @@ -1425,7 +1425,7 @@ string base64Decode(const string& s) { } void callFailure(const std::function& failure, - std::exception_ptr exc) { + const std::exception_ptr& exc) { try { failure(exc); } catch (std::exception& e) { @@ -1516,7 +1516,7 @@ struct InterruptCallbackImpl : InterruptCallback { }; std::unique_ptr createInterruptCallback( - std::function callback) { + const std::function& callback) { auto interruptCallbacks(_interruptCallbacks.lock()); interruptCallbacks->push_back(callback); diff --git a/third_party/nix/src/libutil/util.hh b/third_party/nix/src/libutil/util.hh index 737c4aa77f7a..5ce12f2ede43 100644 --- a/third_party/nix/src/libutil/util.hh +++ b/third_party/nix/src/libutil/util.hh @@ -246,7 +246,7 @@ pid_t startProcess(std::function fun, /* Run a program and return its stdout in a string (i.e., like the shell backtick operator). */ -string runProgram(Path program, bool searchPath = false, +string runProgram(const Path& program, bool searchPath = false, const Strings& args = Strings(), const std::optional& input = {}); @@ -453,7 +453,7 @@ struct InterruptCallback { /* Register a function that gets called on SIGINT (in a non-signal context). */ std::unique_ptr createInterruptCallback( - std::function callback); + const std::function& callback); void triggerInterrupt(); diff --git a/third_party/nix/src/nix-build/nix-build.cc b/third_party/nix/src/nix-build/nix-build.cc index 5090cbe51370..921fc2ba22d1 100644 --- a/third_party/nix/src/nix-build/nix-build.cc +++ b/third_party/nix/src/nix-build/nix-build.cc @@ -299,7 +299,7 @@ static void _main(int argc, char** argv) { if (readStdin) { exprs = {state->parseStdin()}; } else { - for (auto i : left) { + for (const auto& i : left) { if (fromArgs) { exprs.push_back(state->parseExprFromString(i, absPath("."))); } else { diff --git a/third_party/nix/src/nix-collect-garbage/nix-collect-garbage.cc b/third_party/nix/src/nix-collect-garbage/nix-collect-garbage.cc index c3ff068887b5..0050bdda15fb 100644 --- a/third_party/nix/src/nix-collect-garbage/nix-collect-garbage.cc +++ b/third_party/nix/src/nix-collect-garbage/nix-collect-garbage.cc @@ -18,7 +18,7 @@ bool dryRun = false; * Of course, this makes rollbacks to before this point in time * impossible. */ -void removeOldGenerations(std::string dir) { +void removeOldGenerations(const std::string& dir) { if (access(dir.c_str(), R_OK) != 0) { return; } diff --git a/third_party/nix/src/nix-daemon/nix-daemon.cc b/third_party/nix/src/nix-daemon/nix-daemon.cc index 060cc10497b9..9f5e8b7cbedb 100644 --- a/third_party/nix/src/nix-daemon/nix-daemon.cc +++ b/third_party/nix/src/nix-daemon/nix-daemon.cc @@ -194,9 +194,9 @@ struct RetrieveRegularNARSink : ParseSink { } }; -static void performOp(TunnelLogger* logger, ref store, bool trusted, - unsigned int clientVersion, Source& from, Sink& to, - unsigned int op) { +static void performOp(TunnelLogger* logger, const ref& store, + bool trusted, unsigned int clientVersion, Source& from, + Sink& to, unsigned int op) { switch (op) { case wopIsValidPath: { /* 'readStorePath' could raise an error leading to the connection diff --git a/third_party/nix/src/nix-env/nix-env.cc b/third_party/nix/src/nix-env/nix-env.cc index ec2da8f83502..95a395a96fcc 100644 --- a/third_party/nix/src/nix-env/nix-env.cc +++ b/third_party/nix/src/nix-env/nix-env.cc @@ -171,8 +171,8 @@ static void loadSourceExpr(EvalState& state, const Path& path, Value& v) { } } -static void loadDerivations(EvalState& state, Path nixExprPath, - string systemFilter, Bindings& autoArgs, +static void loadDerivations(EvalState& state, const Path& nixExprPath, + const string& systemFilter, Bindings& autoArgs, const string& pathPrefix, DrvInfos& elems) { Value vRoot; loadSourceExpr(state, nixExprPath, vRoot); diff --git a/third_party/nix/src/nix-store/dotgraph.cc b/third_party/nix/src/nix-store/dotgraph.cc index 259ecbd74a5e..0186a7b22ebd 100644 --- a/third_party/nix/src/nix-store/dotgraph.cc +++ b/third_party/nix/src/nix-store/dotgraph.cc @@ -79,7 +79,7 @@ void printClosure(const Path & nePath, const StoreExpr & fs) } #endif -void printDotGraph(ref store, const PathSet& roots) { +void printDotGraph(const ref& store, const PathSet& roots) { PathSet workList(roots); PathSet doneSet; diff --git a/third_party/nix/src/nix-store/dotgraph.hh b/third_party/nix/src/nix-store/dotgraph.hh index c7069451e114..20fc357b1bd4 100644 --- a/third_party/nix/src/nix-store/dotgraph.hh +++ b/third_party/nix/src/nix-store/dotgraph.hh @@ -6,6 +6,6 @@ namespace nix { class Store; -void printDotGraph(ref store, const PathSet& roots); +void printDotGraph(const ref& store, const PathSet& roots); } // namespace nix diff --git a/third_party/nix/src/nix-store/graphml.cc b/third_party/nix/src/nix-store/graphml.cc index 6510b328810d..a1a16cc8f28a 100644 --- a/third_party/nix/src/nix-store/graphml.cc +++ b/third_party/nix/src/nix-store/graphml.cc @@ -37,7 +37,7 @@ static string makeNode(const ValidPathInfo& info) { (isDerivation(info.path) ? "derivation" : "output-path")); } -void printGraphML(ref store, const PathSet& roots) { +void printGraphML(const ref& store, const PathSet& roots) { PathSet workList(roots); PathSet doneSet; std::pair ret; diff --git a/third_party/nix/src/nix-store/graphml.hh b/third_party/nix/src/nix-store/graphml.hh index c330dc6abc43..199421acb264 100644 --- a/third_party/nix/src/nix-store/graphml.hh +++ b/third_party/nix/src/nix-store/graphml.hh @@ -6,6 +6,6 @@ namespace nix { class Store; -void printGraphML(ref store, const PathSet& roots); +void printGraphML(const ref& store, const PathSet& roots); } // namespace nix diff --git a/third_party/nix/src/nix-store/nix-store.cc b/third_party/nix/src/nix-store/nix-store.cc index 802ebd420792..40d6f01cd96c 100644 --- a/third_party/nix/src/nix-store/nix-store.cc +++ b/third_party/nix/src/nix-store/nix-store.cc @@ -28,6 +28,9 @@ using namespace nix; using std::cin; using std::cout; +// TODO(tazjin): clang-tidy's performance lints don't like this, but +// the automatic fixes fail (it seems that some of the ops want to own +// the args for whatever reason) using Operation = void (*)(Strings, Strings); static Path gcRoot; diff --git a/third_party/nix/src/nix/cat.cc b/third_party/nix/src/nix/cat.cc index 9b2da929ac33..2d662a453f54 100644 --- a/third_party/nix/src/nix/cat.cc +++ b/third_party/nix/src/nix/cat.cc @@ -8,7 +8,7 @@ using namespace nix; struct MixCat : virtual Args { std::string path; - void cat(ref accessor) { + void cat(const ref& accessor) { auto st = accessor->stat(path); if (st.type == FSAccessor::Type::tMissing) { throw Error(format("path '%1%' does not exist") % path); diff --git a/third_party/nix/src/nix/command.hh b/third_party/nix/src/nix/command.hh index 829ce080d3a2..4122bdd0e58f 100644 --- a/third_party/nix/src/nix/command.hh +++ b/third_party/nix/src/nix/command.hh @@ -171,21 +171,23 @@ struct RegisterCommand { }; std::shared_ptr parseInstallable(SourceExprCommand& cmd, - ref store, + const ref& store, const std::string& installable, bool useDefaultInstallables); -Buildables build(ref store, RealiseMode mode, - std::vector> installables); +Buildables build(const ref& store, RealiseMode mode, + const std::vector>& installables); -PathSet toStorePaths(ref store, RealiseMode mode, - std::vector> installables); +PathSet toStorePaths( + const ref& store, RealiseMode mode, + const std::vector>& installables); -Path toStorePath(ref store, RealiseMode mode, - std::shared_ptr installable); +Path toStorePath(const ref& store, RealiseMode mode, + const std::shared_ptr& installable); -PathSet toDerivations(ref store, - std::vector> installables, - bool useDeriver = false); +PathSet toDerivations( + const ref& store, + const std::vector>& installables, + bool useDeriver = false); } // namespace nix diff --git a/third_party/nix/src/nix/doctor.cc b/third_party/nix/src/nix/doctor.cc index 95bf8a922e26..44b4bd605d6c 100644 --- a/third_party/nix/src/nix/doctor.cc +++ b/third_party/nix/src/nix/doctor.cc @@ -64,7 +64,7 @@ struct CmdDoctor : StoreCommand { return true; } - static bool checkProfileRoots(ref store) { + static bool checkProfileRoots(const ref& store) { PathSet dirs; for (auto& dir : tokenizeString(getEnv("PATH"), ":")) { diff --git a/third_party/nix/src/nix/hash.cc b/third_party/nix/src/nix/hash.cc index da4b64fe757c..74ff45d0d84e 100644 --- a/third_party/nix/src/nix/hash.cc +++ b/third_party/nix/src/nix/hash.cc @@ -34,7 +34,7 @@ struct CmdHash : Command { } void run() override { - for (auto path : paths) { + for (const auto& path : paths) { Hash h = mode == mFile ? hashFile(ht, path) : hashPath(ht, path).first; if (truncate && h.hashSize > 20) { h = compressHash(h, 20); @@ -73,7 +73,7 @@ struct CmdToBase : Command { } void run() override { - for (auto s : args) { + for (const auto& s : args) { std::cout << fmt("%s\n", Hash(s, ht).to_string(base, base == SRI)); } } diff --git a/third_party/nix/src/nix/installables.cc b/third_party/nix/src/nix/installables.cc index f6cba6d3b636..8d2f93256814 100644 --- a/third_party/nix/src/nix/installables.cc +++ b/third_party/nix/src/nix/installables.cc @@ -198,8 +198,8 @@ std::string attrRegex = R"([A-Za-z_][A-Za-z0-9-_+]*)"; static std::regex attrPathRegex(fmt(R"(%1%(\.%1%)*)", attrRegex)); static std::vector> parseInstallables( - SourceExprCommand& cmd, ref store, std::vector ss, - bool useDefaultInstallables) { + SourceExprCommand& cmd, const ref& store, + std::vector ss, bool useDefaultInstallables) { std::vector> result; if (ss.empty() && useDefaultInstallables) { @@ -213,7 +213,7 @@ static std::vector> parseInstallables( if (s.compare(0, 1, "(") == 0) { result.push_back(std::make_shared(cmd, s)); - } else if (s.find("/") != std::string::npos) { + } else if (s.find('/') != std::string::npos) { auto path = store->toStorePath(store->followLinksToStore(s)); if (store->isStorePath(path)) { @@ -233,7 +233,7 @@ static std::vector> parseInstallables( } std::shared_ptr parseInstallable(SourceExprCommand& cmd, - ref store, + const ref& store, const std::string& installable, bool useDefaultInstallables) { auto installables = parseInstallables(cmd, store, {installable}, false); @@ -241,8 +241,9 @@ std::shared_ptr parseInstallable(SourceExprCommand& cmd, return installables.front(); } -Buildables build(ref store, RealiseMode mode, - std::vector> installables) { +Buildables build( + const ref& store, RealiseMode mode, + const std::vector>& installables) { if (mode != Build) { settings.readOnlyMode = true; } @@ -278,8 +279,9 @@ Buildables build(ref store, RealiseMode mode, return buildables; } -PathSet toStorePaths(ref store, RealiseMode mode, - std::vector> installables) { +PathSet toStorePaths( + const ref& store, RealiseMode mode, + const std::vector>& installables) { PathSet outPaths; for (auto& b : build(store, mode, installables)) { @@ -291,8 +293,8 @@ PathSet toStorePaths(ref store, RealiseMode mode, return outPaths; } -Path toStorePath(ref store, RealiseMode mode, - std::shared_ptr installable) { +Path toStorePath(const ref& store, RealiseMode mode, + const std::shared_ptr& installable) { auto paths = toStorePaths(store, mode, {installable}); if (paths.size() != 1) { @@ -303,9 +305,10 @@ Path toStorePath(ref store, RealiseMode mode, return *paths.begin(); } -PathSet toDerivations(ref store, - std::vector> installables, - bool useDeriver) { +PathSet toDerivations( + const ref& store, + const std::vector>& installables, + bool useDeriver) { PathSet drvPaths; for (auto& i : installables) { diff --git a/third_party/nix/src/nix/ls.cc b/third_party/nix/src/nix/ls.cc index 13b7bbbbb9bd..5b052f8fcc7c 100644 --- a/third_party/nix/src/nix/ls.cc +++ b/third_party/nix/src/nix/ls.cc @@ -74,7 +74,7 @@ struct MixLs : virtual Args, MixJSON { showDirectory); } - void list(ref accessor) { + void list(const ref& accessor) { if (path == "/") { path = ""; } diff --git a/third_party/nix/src/nix/repl.cc b/third_party/nix/src/nix/repl.cc index 81e940dd3aaf..78991645ac07 100644 --- a/third_party/nix/src/nix/repl.cc +++ b/third_party/nix/src/nix/repl.cc @@ -3,6 +3,7 @@ #include #include #include +#include #include @@ -58,10 +59,10 @@ struct NixRepl { const Path historyFile; - NixRepl(const Strings& searchPath, nix::ref store); + NixRepl(const Strings& searchPath, const nix::ref& store); ~NixRepl(); void mainLoop(const std::vector& files); - StringSet completePrefix(string prefix); + StringSet completePrefix(const string& prefix); static bool getLine(string& input, const std::string& prompt); Path getDerivationPath(Value& v); bool processLine(string line); @@ -70,7 +71,7 @@ struct NixRepl { void reloadFiles(); void addAttrsToScope(Value& attrs); void addVarToScope(const Symbol& name, Value& v); - Expr* parseString(string s); + Expr* parseString(const string& s); void evalString(string s, Value& v); using ValuesSeen = set; @@ -129,7 +130,7 @@ string removeWhitespace(string s) { return s; } -NixRepl::NixRepl(const Strings& searchPath, nix::ref store) +NixRepl::NixRepl(const Strings& searchPath, const nix::ref& store) : state(searchPath, store), staticEnv(false, &state.staticBaseEnv), historyFile(getDataDir() + "/nix/repl-history") { @@ -332,7 +333,7 @@ bool NixRepl::getLine(string& input, const std::string& prompt) { return true; } -StringSet NixRepl::completePrefix(string prefix) { +StringSet NixRepl::completePrefix(const string& prefix) { StringSet completions; size_t start = prefix.find_last_of(" \n\r\t(){}[]"); @@ -641,13 +642,13 @@ void NixRepl::addVarToScope(const Symbol& name, Value& v) { varNames.insert((string)name); } -Expr* NixRepl::parseString(string s) { +Expr* NixRepl::parseString(const string& s) { Expr* e = state.parseExprFromString(s, curDir, staticEnv); return e; } void NixRepl::evalString(string s, Value& v) { - Expr* e = parseString(s); + Expr* e = parseString(std::move(s)); e->eval(state, *env, v); state.forceValue(v); } diff --git a/third_party/nix/src/nix/run.cc b/third_party/nix/src/nix/run.cc index 9827360f0aa9..fa471bd542a1 100644 --- a/third_party/nix/src/nix/run.cc +++ b/third_party/nix/src/nix/run.cc @@ -30,7 +30,7 @@ struct CmdRun : InstallablesCommand { .description("command and arguments to be executed; defaults to 'bash'") .labels({"command", "args"}) .arity(ArityAny) - .handler([&](std::vector ss) { + .handler([&](const std::vector& ss) { if (ss.empty()) { throw UsageError("--command requires at least one argument"); } @@ -227,7 +227,7 @@ void chrootHelper(int argc, char** argv) { throw SysError("mounting '%s' on '%s'", realStoreDir, storeDir); } - for (auto entry : readDirectory("/")) { + for (const auto& entry : readDirectory("/")) { auto src = "/" + entry.name; auto st = lstat(src); if (!S_ISDIR(st.st_mode)) { diff --git a/third_party/nix/src/nix/search.cc b/third_party/nix/src/nix/search.cc index 5f6def220291..7fe81adb6990 100644 --- a/third_party/nix/src/nix/search.cc +++ b/third_party/nix/src/nix/search.cc @@ -16,12 +16,12 @@ using namespace nix; -std::string wrap(std::string prefix, std::string s) { +std::string wrap(const std::string& prefix, const std::string& s) { return prefix + s + ANSI_NORMAL; } std::string hilite(const std::string& s, const std::smatch& m, - std::string postfix) { + const std::string& postfix) { return m.empty() ? s : std::string(m.prefix()) + ANSI_RED + std::string(m.str()) + postfix + std::string(m.suffix()); @@ -99,7 +99,7 @@ struct CmdSearch : SourceExprCommand, MixJSON { std::function doExpr; - doExpr = [&](Value* v, std::string attrPath, bool toplevel, + doExpr = [&](Value* v, const std::string& attrPath, bool toplevel, JSONObject* cache) { DLOG(INFO) << "at attribute '" << attrPath << "'"; @@ -262,7 +262,7 @@ struct CmdSearch : SourceExprCommand, MixJSON { } RunPager pager; - for (auto el : results) { + for (const auto& el : results) { std::cout << el.second << "\n"; } } diff --git a/third_party/nix/src/nix/upgrade-nix.cc b/third_party/nix/src/nix/upgrade-nix.cc index b225aac2d725..9b060a7f195a 100644 --- a/third_party/nix/src/nix/upgrade-nix.cc +++ b/third_party/nix/src/nix/upgrade-nix.cc @@ -97,7 +97,7 @@ struct CmdUpgradeNix : MixDryRun, StoreCommand { } /* Return the profile in which Nix is installed. */ - static Path getProfileDir(ref store) { + static Path getProfileDir(const ref& store) { Path where; for (auto& dir : tokenizeString(getEnv("PATH"), ":")) { @@ -143,7 +143,7 @@ struct CmdUpgradeNix : MixDryRun, StoreCommand { } /* Return the store path of the latest stable Nix. */ - Path getLatestNix(ref store) { + Path getLatestNix(const ref& store) { // FIXME: use nixos.org? auto req = DownloadRequest(storePathsUrl); auto res = getDownloader()->download(req); diff --git a/third_party/nix/src/nix/verify.cc b/third_party/nix/src/nix/verify.cc index 63dd181e074c..a773b3a2c8de 100644 --- a/third_party/nix/src/nix/verify.cc +++ b/third_party/nix/src/nix/verify.cc @@ -102,8 +102,8 @@ struct CmdVerify : StorePathsCommand { size_t actualSigsNeeded = std::max(sigsNeeded, (size_t)1); size_t validSigs = 0; - auto doSigs = [&](StringSet sigs) { - for (auto sig : sigs) { + auto doSigs = [&](const StringSet& sigs) { + for (const auto& sig : sigs) { if (sigsSeen.count(sig) != 0u) { continue; } -- cgit 1.4.1