diff options
| author | Vincent Ambo <tazjin@google.com> | 2020-05-20T03·33+0100 |
|---|---|---|
| committer | Vincent Ambo <tazjin@google.com> | 2020-05-20T03·33+0100 |
| commit | d331d3a0b5c497a46e2636f308234be66566c04c (patch) | |
| tree | 92526b2f99456c09c5cc81233ed5a4311abe3d2b /third_party/nix/src/libstore/gc.cc | |
| parent | fed31b2c9b364fc1ed0b724c21b068cdedf46ee7 (diff) | |
refactor(3p/nix): Apply clang-tidy's modernize-* fixes r/787
This applies the modernization fixes listed here: https://clang.llvm.org/extra/clang-tidy/checks/list.html The 'modernize-use-trailing-return-type' fix was excluded due to my personal preference (more specifically, I think the 'auto' keyword is misleading in that position).
Diffstat (limited to 'third_party/nix/src/libstore/gc.cc')
| -rw-r--r-- | third_party/nix/src/libstore/gc.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/third_party/nix/src/libstore/gc.cc b/third_party/nix/src/libstore/gc.cc index 2241c58c41..a0c84dc0a5 100644 --- a/third_party/nix/src/libstore/gc.cc +++ b/third_party/nix/src/libstore/gc.cc @@ -1,11 +1,11 @@ #include <algorithm> +#include <cerrno> #include <climits> #include <functional> #include <queue> #include <random> #include <regex> -#include <errno.h> #include <fcntl.h> #include <sys/stat.h> #include <sys/statvfs.h> @@ -147,7 +147,7 @@ void LocalStore::addTempRoot(const Path& path) { /* Create the temporary roots file for this process. */ if (!state->fdTempRoots) { - while (1) { + while (true) { AutoCloseFD fdGCLock = openGCLock(ltRead); if (pathExists(fnTempRoots)) { @@ -505,7 +505,8 @@ struct LocalStore::GCState { unsigned long long bytesInvalidated; bool moveToTrash = true; bool shouldDelete; - explicit GCState(GCResults& results_) : results(results_), bytesInvalidated(0) {} + explicit GCState(GCResults& results_) + : results(results_), bytesInvalidated(0) {} }; bool LocalStore::isActiveTempFile(const GCState& state, const Path& path, |