diff options
| author | Kane York <kanepyork@gmail.com> | 2020-07-24T01·59-0700 |
|---|---|---|
| committer | kanepyork <rikingcoding@gmail.com> | 2020-07-24T02·40+0000 |
| commit | 3ce4a6cc18b233da69b5e2e193ba88677baed34f (patch) | |
| tree | 01c6fdc20745b9c68086b9590e21373171a40274 /third_party/nix/src/libstore | |
| parent | fba4d145f8e8945e9fd2438e14075f3945da9fe3 (diff) | |
fix(3p/nix): fix more clang-tidy warnings r/1451
Change-Id: I1e8a4852ffa450c552a4cdfc709554799854934c Reviewed-on: https://cl.tvl.fyi/c/depot/+/1416 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi>
Diffstat (limited to 'third_party/nix/src/libstore')
| -rw-r--r-- | third_party/nix/src/libstore/download.cc | 2 | ||||
| -rw-r--r-- | third_party/nix/src/libstore/local-store.cc | 3 |
2 files changed, 4 insertions, 1 deletions
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 |