fix(3p/nix): revert "apply all clang-tidy fixes" r/1534

This reverts commit ef54f5da9fa30b5c302f2a49595ee5d041f9706a.

Resolved conflicts:
	third_party/nix/src/libexpr/eval.cc
	third_party/nix/src/libstore/builtins/fetchurl.cc
	third_party/nix/src/libstore/references.cc
	third_party/nix/src/libutil/hash.cc
	third_party/nix/src/nix-daemon/nix-daemon.cc

Change-Id: Ib9cf6e96a79a23bde3983579ced3f92e530cb011
Reviewed-on: https://cl.tvl.fyi/c/depot/+/1547
Reviewed-by: glittershark <grfn@gws.fyi>
Tested-by: BuildkiteCI

1 files changed, 10 insertions, 16 deletions

diff --git a/third_party/nix/src/libexpr/primops/fromTOML.cc b/third_party/nix/src/libexpr/primops/fromTOML.cc
index e3d2a4940769..389ac080677a 100644
--- a/third_party/nix/src/libexpr/primops/fromTOML.cc
+++ b/third_party/nix/src/libexpr/primops/fromTOML.cc
@@ -30,12 +30,10 @@ static void prim_fromTOML(EvalState& state, const Pos& pos, Value** args,
         if (auto i2 = i.second->as_table_array()) {
           size_t size2 = i2->get().size();
           state.mkList(v2, size2);
-          for (size_t j = 0; j < size2; ++j) {
+          for (size_t j = 0; j < size2; ++j)
             visit(*((*v2.list)[j] = state.allocValue()), i2->get()[j]);
-          }
-        } else {
+        } else
           visit(v2, i.second);
-        }
       }
     }
 
@@ -44,9 +42,8 @@ static void prim_fromTOML(EvalState& state, const Pos& pos, Value** args,
 
       state.mkList(v, size);
 
-      for (size_t i = 0; i < size; ++i) {
+      for (size_t i = 0; i < size; ++i)
         visit(*((*v.list)[i] = state.allocValue()), t2->get()[i]);
-      }
     }
 
     // Handle cases like 'a = [[{ a = true }]]', which IMHO should be
@@ -58,28 +55,25 @@ static void prim_fromTOML(EvalState& state, const Pos& pos, Value** args,
 
       state.mkList(v, size);
 
-      for (size_t j = 0; j < size; ++j) {
+      for (size_t j = 0; j < size; ++j)
         visit(*((*v.list)[j] = state.allocValue()), t2->get()[j]);
-      }
     }
 
     else if (t->is_value()) {
-      if (auto val = t->as<int64_t>()) {
+      if (auto val = t->as<int64_t>())
         mkInt(v, val->get());
-      } else if (auto val = t->as<NixFloat>()) {
+      else if (auto val = t->as<NixFloat>())
         mkFloat(v, val->get());
-      } else if (auto val = t->as<bool>()) {
+      else if (auto val = t->as<bool>())
         mkBool(v, val->get());
-      } else if (auto val = t->as<std::string>()) {
+      else if (auto val = t->as<std::string>())
         mkString(v, val->get());
-      } else {
+      else
         throw EvalError("unsupported value type in TOML");
-      }
     }
 
-    else {
+    else
       abort();
-    }
   };
 
   try {