From 3ce4a6cc18b233da69b5e2e193ba88677baed34f Mon Sep 17 00:00:00 2001 From: Kane York Date: Thu, 23 Jul 2020 18:59:01 -0700 Subject: fix(3p/nix): fix more clang-tidy warnings Change-Id: I1e8a4852ffa450c552a4cdfc709554799854934c Reviewed-on: https://cl.tvl.fyi/c/depot/+/1416 Tested-by: BuildkiteCI Reviewed-by: glittershark --- third_party/nix/src/libstore/download.cc | 2 +- third_party/nix/src/libstore/local-store.cc | 3 +++ third_party/nix/src/tests/language-tests.cc | 6 +++--- 3 files changed, 7 insertions(+), 4 deletions(-) (limited to 'third_party/nix') diff --git a/third_party/nix/src/libstore/download.cc b/third_party/nix/src/libstore/download.cc index f30642d69daf..e73181e73109 100644 --- a/third_party/nix/src/libstore/download.cc +++ b/third_party/nix/src/libstore/download.cc @@ -835,7 +835,7 @@ void Downloader::download(DownloadRequest&& request, Sink& sink) { state.wait(state->avail); } - chunk = std::move(state->data); + chunk = state->data; state->request.notify_one(); } diff --git a/third_party/nix/src/libstore/local-store.cc b/third_party/nix/src/libstore/local-store.cc index 995bc4f998b4..5ae38535d841 100644 --- a/third_party/nix/src/libstore/local-store.cc +++ b/third_party/nix/src/libstore/local-store.cc @@ -73,6 +73,9 @@ LocalStore::LocalStore(const Params& params) } } + // TODO(kanepyork): migrate to external constructor, this bypasses virtual + // dispatch + // NOLINTNEXTLINE clang-analyzer-optin.cplusplus.VirtualCall createUser(getUserName(), getuid()); /* Optionally, create directories and set permissions for a diff --git a/third_party/nix/src/tests/language-tests.cc b/third_party/nix/src/tests/language-tests.cc index 98f2927911b2..9fb453e5e537 100644 --- a/third_party/nix/src/tests/language-tests.cc +++ b/third_party/nix/src/tests/language-tests.cc @@ -212,13 +212,13 @@ TEST_P(EvalSuccessTest, Fails) { EvalState state({}, ref(store)); auto path = GetParam(); - Expr* expr; - EXPECT_NO_THROW(expr = state.parseExprFromFile(GetParam().string())) + Expr* expr = nullptr; + ASSERT_NO_THROW(expr = state.parseExprFromFile(GetParam().string())) << path.stem().string() << ": should parse successfully"; Value result; - EXPECT_NO_THROW({ + ASSERT_NO_THROW({ state.eval(expr, result); state.forceValueDeep(result); }) << path.stem().string() -- cgit 1.4.1