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 +++ 2 files changed, 4 insertions(+), 1 deletion(-) (limited to 'third_party/nix/src/libstore') 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 -- cgit 1.4.1