about summary refs log tree commit diff
path: root/third_party/nix/src/nix-env
diff options
context:
space:
mode:
authorVincent Ambo <tazjin@google.com>2020-05-20T21·27+0100
committerVincent Ambo <tazjin@google.com>2020-05-20T21·27+0100
commit689ef502f5b0655c9923ed77da2ae3504630f473 (patch)
tree3e331c153646f136875f047cc3b9f0aad8c86341 /third_party/nix/src/nix-env
parentd331d3a0b5c497a46e2636f308234be66566c04c (diff)
refactor(3p/nix): Apply clang-tidy's readability-* fixes r/788
This applies the readability fixes listed here:

https://clang.llvm.org/extra/clang-tidy/checks/list.html
Diffstat (limited to 'third_party/nix/src/nix-env')
-rw-r--r--third_party/nix/src/nix-env/nix-env.cc78
-rw-r--r--third_party/nix/src/nix-env/user-env.cc27
2 files changed, 57 insertions, 48 deletions
diff --git a/third_party/nix/src/nix-env/nix-env.cc b/third_party/nix/src/nix-env/nix-env.cc
index 86340d5c9138..ec2da8f83502 100644
--- a/third_party/nix/src/nix-env/nix-env.cc
+++ b/third_party/nix/src/nix-env/nix-env.cc
@@ -319,7 +319,7 @@ static void queryInstSources(EvalState& state, InstallSourceInfo& instSource,
                              const Strings& args, DrvInfos& elems,
                              bool newestOnly) {
   InstallSourceType type = instSource.type;
-  if (type == srcUnknown && args.size() > 0 && isPath(args.front())) {
+  if (type == srcUnknown && !args.empty() && isPath(args.front())) {
     type = srcStorePaths;
   }
 
@@ -352,7 +352,8 @@ static void queryInstSources(EvalState& state, InstallSourceInfo& instSource,
 
       for (auto& i : args) {
         Expr* eFun = state.parseExprFromString(i, absPath("."));
-        Value vFun, vTmp;
+        Value vFun;
+        Value vTmp;
         state.eval(eFun, vFun);
         mkApp(vTmp, vFun, vArg);
         getDerivations(state, vTmp, "", *instSource.autoArgs, elems, true);
@@ -420,7 +421,7 @@ static void printMissing(EvalState& state, DrvInfos& elems) {
   PathSet targets;
   for (auto& i : elems) {
     Path drvPath = i.queryDrvPath();
-    if (drvPath != "") {
+    if (!drvPath.empty()) {
       targets.insert(drvPath);
     } else {
       targets.insert(i.queryOutPath());
@@ -437,7 +438,8 @@ static void installDerivations(Globals& globals, const Strings& args,
   DLOG(INFO) << "installing derivations";
 
   /* Get the set of user environment elements to be installed. */
-  DrvInfos newElems, newElemsTmp;
+  DrvInfos newElems;
+  DrvInfos newElemsTmp;
   queryInstSources(*globals.state, globals.instSource, args, newElemsTmp, true);
 
   /* If --prebuilt-only is given, filter out source-only packages. */
@@ -453,7 +455,7 @@ static void installDerivations(Globals& globals, const Strings& args,
        one-click installs, namely those where the name used in the
        path is not the one we want (e.g., `java-front' versus
        `java-front-0.9pre15899'). */
-    if (globals.forceName != "") {
+    if (!globals.forceName.empty()) {
       i.setName(globals.forceName);
     }
     newNames.insert(DrvName(i.queryName()).name);
@@ -644,7 +646,7 @@ static void setMetaFlag(EvalState& state, DrvInfo& drv, const string& name,
 }
 
 static void opSetFlag(Globals& globals, Strings opFlags, Strings opArgs) {
-  if (opFlags.size() > 0) {
+  if (!opFlags.empty()) {
     throw UsageError(format("unknown flag '%1%'") % opFlags.front());
   }
   if (opArgs.size() < 2) {
@@ -708,18 +710,18 @@ static void opSet(Globals& globals, Strings opFlags, Strings opArgs) {
 
   DrvInfo& drv(elems.front());
 
-  if (globals.forceName != "") {
+  if (!globals.forceName.empty()) {
     drv.setName(globals.forceName);
   }
 
-  if (drv.queryDrvPath() != "") {
+  if (!drv.queryDrvPath().empty()) {
     PathSet paths = {drv.queryDrvPath()};
     printMissing(globals.state->store, paths);
     if (globals.dryRun) {
       return;
     }
     globals.state->store->buildPaths(
-        paths, globals.state->repair ? bmRepair : bmNormal);
+        paths, globals.state->repair != 0u ? bmRepair : bmNormal);
   } else {
     printMissing(globals.state->store, {drv.queryOutPath()});
     if (globals.dryRun) {
@@ -774,7 +776,7 @@ static void uninstallDerivations(Globals& globals, Strings& selectors,
 }
 
 static void opUninstall(Globals& globals, Strings opFlags, Strings opArgs) {
-  if (opFlags.size() > 0) {
+  if (!opFlags.empty()) {
     throw UsageError(format("unknown flag '%1%'") % opFlags.front());
   }
   uninstallDerivations(globals, opArgs, globals.profile);
@@ -792,7 +794,7 @@ static bool cmpElemByName(const DrvInfo& a, const DrvInfo& b) {
 using Table = list<Strings>;
 
 void printTable(Table& table) {
-  auto nrColumns = table.size() > 0 ? table.front().size() : 0;
+  auto nrColumns = !table.empty() ? table.front().size() : 0;
 
   vector<size_t> widths;
   widths.resize(nrColumns);
@@ -852,7 +854,7 @@ static VersionDiff compareVersionAgainstSet(const DrvInfo& elem,
         version = name2.version;
       } else if (diff != cvGreater && diff != cvEqual && d > 0) {
         diff = cvLess;
-        if (version == "" || compareVersions(version, name2.version) < 0) {
+        if (version.empty() || compareVersions(version, name2.version) < 0) {
           version = name2.version;
         }
       }
@@ -878,7 +880,7 @@ static void queryJSON(Globals& globals, vector<DrvInfo>& elems) {
     for (auto& j : metaNames) {
       auto placeholder = metaObj.placeholder(j);
       Value* v = i.queryMeta(j);
-      if (!v) {
+      if (v == nullptr) {
         LOG(ERROR) << "derivation '" << i.queryName()
                    << "' has invalid meta attribute '" << j << "'";
         placeholder.write(nullptr);
@@ -946,7 +948,8 @@ static void opQuery(Globals& globals, Strings opFlags, Strings opArgs) {
   }
 
   /* Obtain derivation information from the specified source. */
-  DrvInfos availElems, installedElems;
+  DrvInfos availElems;
+  DrvInfos installedElems;
 
   if (source == sInstalled || compareVersions || printStatus) {
     installedElems = queryInstalled(*globals.state, globals.profile);
@@ -983,7 +986,8 @@ static void opQuery(Globals& globals, Strings opFlags, Strings opArgs) {
   }
 
   /* Query which paths have substitutes. */
-  PathSet validPaths, substitutablePaths;
+  PathSet validPaths;
+  PathSet substitutablePaths;
   if (printStatus || globals.prebuiltOnly) {
     PathSet paths;
     for (auto& i : elems) {
@@ -1005,7 +1009,7 @@ static void opQuery(Globals& globals, Strings opFlags, Strings opArgs) {
     return;
   }
 
-  bool tty = isatty(STDOUT_FILENO);
+  bool tty = isatty(STDOUT_FILENO) != 0;
   RunPager pager;
 
   Table table;
@@ -1107,7 +1111,7 @@ static void opQuery(Globals& globals, Strings opFlags, Strings opArgs) {
       }
 
       if (xmlOutput) {
-        if (i.querySystem() != "") {
+        if (!i.querySystem().empty()) {
           attrs["system"] = i.querySystem();
         }
       } else if (printSystem) {
@@ -1117,11 +1121,11 @@ static void opQuery(Globals& globals, Strings opFlags, Strings opArgs) {
       if (printDrvPath) {
         string drvPath = i.queryDrvPath();
         if (xmlOutput) {
-          if (drvPath != "") {
+          if (!drvPath.empty()) {
             attrs["drvPath"] = drvPath;
           }
         } else {
-          columns.push_back(drvPath == "" ? "-" : drvPath);
+          columns.push_back(drvPath.empty() ? "-" : drvPath);
         }
       }
 
@@ -1144,7 +1148,7 @@ static void opQuery(Globals& globals, Strings opFlags, Strings opArgs) {
       if (printDescription) {
         string descr = i.queryMetaString("description");
         if (xmlOutput) {
-          if (descr != "") {
+          if (!descr.empty()) {
             attrs["description"] = descr;
           }
         } else {
@@ -1170,7 +1174,7 @@ static void opQuery(Globals& globals, Strings opFlags, Strings opArgs) {
               XMLAttrs attrs2;
               attrs2["name"] = j;
               Value* v = i.queryMeta(j);
-              if (!v) {
+              if (v == nullptr) {
                 LOG(ERROR) << "derivation '" << i.queryName()
                            << "' has invalid meta attribute '" << j << "'";
               } else {
@@ -1244,7 +1248,7 @@ static void opQuery(Globals& globals, Strings opFlags, Strings opArgs) {
 }
 
 static void opSwitchProfile(Globals& globals, Strings opFlags, Strings opArgs) {
-  if (opFlags.size() > 0) {
+  if (!opFlags.empty()) {
     throw UsageError(format("unknown flag '%1%'") % opFlags.front());
   }
   if (opArgs.size() != 1) {
@@ -1278,9 +1282,8 @@ static void switchGeneration(Globals& globals, int dstGen) {
     if (dstGen == prevGen) {
       throw Error(format("no generation older than the current (%1%) exists") %
                   curGen);
-    } else {
-      throw Error(format("generation %1% does not exist") % dstGen);
     }
+    throw Error(format("generation %1% does not exist") % dstGen);
   }
 
   LOG(INFO) << "switching from generation " << curGen << " to " << dst.number;
@@ -1294,7 +1297,7 @@ static void switchGeneration(Globals& globals, int dstGen) {
 
 static void opSwitchGeneration(Globals& globals, Strings opFlags,
                                Strings opArgs) {
-  if (opFlags.size() > 0) {
+  if (!opFlags.empty()) {
     throw UsageError(format("unknown flag '%1%'") % opFlags.front());
   }
   if (opArgs.size() != 1) {
@@ -1310,10 +1313,10 @@ static void opSwitchGeneration(Globals& globals, Strings opFlags,
 }
 
 static void opRollback(Globals& globals, Strings opFlags, Strings opArgs) {
-  if (opFlags.size() > 0) {
+  if (!opFlags.empty()) {
     throw UsageError(format("unknown flag '%1%'") % opFlags.front());
   }
-  if (opArgs.size() != 0) {
+  if (!opArgs.empty()) {
     throw UsageError(format("no arguments expected"));
   }
 
@@ -1322,10 +1325,10 @@ static void opRollback(Globals& globals, Strings opFlags, Strings opArgs) {
 
 static void opListGenerations(Globals& globals, Strings opFlags,
                               Strings opArgs) {
-  if (opFlags.size() > 0) {
+  if (!opFlags.empty()) {
     throw UsageError(format("unknown flag '%1%'") % opFlags.front());
   }
-  if (opArgs.size() != 0) {
+  if (!opArgs.empty()) {
     throw UsageError(format("no arguments expected"));
   }
 
@@ -1339,7 +1342,7 @@ static void opListGenerations(Globals& globals, Strings opFlags,
 
   for (auto& i : gens) {
     tm t;
-    if (!localtime_r(&i.creationTime, &t)) {
+    if (localtime_r(&i.creationTime, &t) == nullptr) {
       throw Error("cannot convert time");
     }
     cout << format("%|4|   %|4|-%|02|-%|02| %|02|:%|02|:%|02|   %||\n") %
@@ -1351,7 +1354,7 @@ static void opListGenerations(Globals& globals, Strings opFlags,
 
 static void opDeleteGenerations(Globals& globals, Strings opFlags,
                                 Strings opArgs) {
-  if (opFlags.size() > 0) {
+  if (!opFlags.empty()) {
     throw UsageError(format("unknown flag '%1%'") % opFlags.front());
   }
 
@@ -1390,7 +1393,8 @@ static void opVersion(Globals& globals, Strings opFlags, Strings opArgs) {
 
 static int _main(int argc, char** argv) {
   {
-    Strings opFlags, opArgs;
+    Strings opFlags;
+    Strings opArgs;
     Operation op = nullptr;
     RepairFlag repair = NoRepair;
     string file;
@@ -1482,7 +1486,7 @@ static int _main(int argc, char** argv) {
         opArgs.push_back(*arg);
       }
 
-      if (oldOp && oldOp != op) {
+      if ((oldOp != nullptr) && oldOp != op) {
         throw UsageError("only one operation may be specified");
       }
 
@@ -1493,7 +1497,7 @@ static int _main(int argc, char** argv) {
 
     initPlugins();
 
-    if (!op) {
+    if (op == nullptr) {
       throw UsageError("no operation specified");
     }
 
@@ -1503,17 +1507,17 @@ static int _main(int argc, char** argv) {
         std::shared_ptr<EvalState>(new EvalState(myArgs.searchPath, store));
     globals.state->repair = repair;
 
-    if (file != "") {
+    if (!file.empty()) {
       globals.instSource.nixExprPath = lookupFileArg(*globals.state, file);
     }
 
     globals.instSource.autoArgs = myArgs.getAutoArgs(*globals.state);
 
-    if (globals.profile == "") {
+    if (globals.profile.empty()) {
       globals.profile = getEnv("NIX_PROFILE", "");
     }
 
-    if (globals.profile == "") {
+    if (globals.profile.empty()) {
       Path profileLink = getHome() + "/.nix-profile";
       try {
         if (!pathExists(profileLink)) {
diff --git a/third_party/nix/src/nix-env/user-env.cc b/third_party/nix/src/nix-env/user-env.cc
index d013f34b08fa..bf15543c3f95 100644
--- a/third_party/nix/src/nix-env/user-env.cc
+++ b/third_party/nix/src/nix-env/user-env.cc
@@ -31,13 +31,14 @@ bool createUserEnv(EvalState& state, DrvInfos& elems, const Path& profile,
      exist already. */
   PathSet drvsToBuild;
   for (auto& i : elems) {
-    if (i.queryDrvPath() != "") {
+    if (!i.queryDrvPath().empty()) {
       drvsToBuild.insert(i.queryDrvPath());
     }
   }
 
   DLOG(INFO) << "building user environment dependencies";
-  state.store->buildPaths(drvsToBuild, state.repair ? bmRepair : bmNormal);
+  state.store->buildPaths(drvsToBuild,
+                          state.repair != 0u ? bmRepair : bmNormal);
 
   /* Construct the whole top level derivation. */
   PathSet references;
@@ -61,7 +62,7 @@ bool createUserEnv(EvalState& state, DrvInfos& elems, const Path& profile,
       mkString(*state.allocAttr(v, state.sSystem), system);
     }
     mkString(*state.allocAttr(v, state.sOutPath), i.queryOutPath());
-    if (drvPath != "") {
+    if (!drvPath.empty()) {
       mkString(*state.allocAttr(v, state.sDrvPath), i.queryDrvPath());
     }
 
@@ -90,7 +91,7 @@ bool createUserEnv(EvalState& state, DrvInfos& elems, const Path& profile,
     StringSet metaNames = i.queryMetaNames();
     for (auto& j : metaNames) {
       Value* v = i.queryMeta(j);
-      if (!v) {
+      if (v == nullptr) {
         continue;
       }
       vMeta.attrs->push_back(Attr(state.symbols.create(j), v));
@@ -98,7 +99,7 @@ bool createUserEnv(EvalState& state, DrvInfos& elems, const Path& profile,
     vMeta.attrs->sort();
     v.attrs->sort();
 
-    if (drvPath != "") {
+    if (!drvPath.empty()) {
       references.insert(drvPath);
     }
   }
@@ -115,7 +116,8 @@ bool createUserEnv(EvalState& state, DrvInfos& elems, const Path& profile,
 
   /* Construct a Nix expression that calls the user environment
      builder with the manifest as argument. */
-  Value args, topLevel;
+  Value args;
+  Value topLevel;
   state.mkAttrs(args, 3);
   mkString(*state.allocAttr(args, state.symbols.create("manifest")),
            manifestFile, {manifestFile});
@@ -128,15 +130,18 @@ bool createUserEnv(EvalState& state, DrvInfos& elems, const Path& profile,
   state.forceValue(topLevel);
   PathSet context;
   Attr& aDrvPath(*topLevel.attrs->find(state.sDrvPath));
-  Path topLevelDrv = state.coerceToPath(aDrvPath.pos ? *(aDrvPath.pos) : noPos,
-                                        *(aDrvPath.value), context);
+  Path topLevelDrv =
+      state.coerceToPath(aDrvPath.pos != nullptr ? *(aDrvPath.pos) : noPos,
+                         *(aDrvPath.value), context);
   Attr& aOutPath(*topLevel.attrs->find(state.sOutPath));
-  Path topLevelOut = state.coerceToPath(aOutPath.pos ? *(aOutPath.pos) : noPos,
-                                        *(aOutPath.value), context);
+  Path topLevelOut =
+      state.coerceToPath(aOutPath.pos != nullptr ? *(aOutPath.pos) : noPos,
+                         *(aOutPath.value), context);
 
   /* Realise the resulting store expression. */
   DLOG(INFO) << "building user environment";
-  state.store->buildPaths({topLevelDrv}, state.repair ? bmRepair : bmNormal);
+  state.store->buildPaths({topLevelDrv},
+                          state.repair != 0u ? bmRepair : bmNormal);
 
   /* Switch the current user environment to the output path. */
   auto store2 = state.store.dynamic_pointer_cast<LocalFSStore>();