guix-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

62/64: nix-store --optimise: Remove bogus statistics


From: Ludovic Courtès
Subject: 62/64: nix-store --optimise: Remove bogus statistics
Date: Mon, 05 Jan 2015 16:39:13 +0000

civodul pushed a commit to branch nix
in repository guix.

commit 84813af5b938246d9a4a785dfb08b86383ef62ae
Author: Eelco Dolstra <address@hidden>
Date:   Thu May 15 11:33:46 2014 +0200

    nix-store --optimise: Remove bogus statistics
---
 src/libstore/local-store.hh    |    6 ++----
 src/libstore/optimise-store.cc |   11 +++++------
 src/nix-store/nix-store.cc     |    6 ++----
 3 files changed, 9 insertions(+), 14 deletions(-)

diff --git a/src/libstore/local-store.hh b/src/libstore/local-store.hh
index 487bb71..54331e4 100644
--- a/src/libstore/local-store.hh
+++ b/src/libstore/local-store.hh
@@ -34,14 +34,12 @@ struct Derivation;
 
 struct OptimiseStats
 {
-    unsigned long totalFiles;
-    unsigned long sameContents;
     unsigned long filesLinked;
     unsigned long long bytesFreed;
     unsigned long long blocksFreed;
     OptimiseStats()
     {
-        totalFiles = sameContents = filesLinked = 0;
+        filesLinked = 0;
         bytesFreed = blocksFreed = 0;
     }
 };
@@ -315,7 +313,7 @@ private:
 #endif
 
     InodeHash loadInodeHash();
-    Strings readDirectoryIgnoringInodes(const Path & path, const InodeHash & 
inodeHash, OptimiseStats & stats);
+    Strings readDirectoryIgnoringInodes(const Path & path, const InodeHash & 
inodeHash);
     void optimisePath_(OptimiseStats & stats, const Path & path, InodeHash & 
inodeHash);
 
     // Internal versions that are not wrapped in retry_sqlite.
diff --git a/src/libstore/optimise-store.cc b/src/libstore/optimise-store.cc
index 39b9e58..e43dbf3 100644
--- a/src/libstore/optimise-store.cc
+++ b/src/libstore/optimise-store.cc
@@ -39,6 +39,7 @@ struct MakeReadOnly
     }
 };
 
+
 LocalStore::InodeHash LocalStore::loadInodeHash()
 {
     printMsg(lvlDebug, "loading hash inodes in memory");
@@ -60,7 +61,8 @@ LocalStore::InodeHash LocalStore::loadInodeHash()
     return inodeHash;
 }
 
-Strings LocalStore::readDirectoryIgnoringInodes(const Path & path, const 
InodeHash & inodeHash, OptimiseStats & stats)
+
+Strings LocalStore::readDirectoryIgnoringInodes(const Path & path, const 
InodeHash & inodeHash)
 {
     Strings names;
 
@@ -73,7 +75,6 @@ Strings LocalStore::readDirectoryIgnoringInodes(const Path & 
path, const InodeHa
 
         if (inodeHash.count(dirent->d_ino)) {
             printMsg(lvlDebug, format("`%1%' is already linked") % 
dirent->d_name);
-            stats.totalFiles++;
             continue;
         }
 
@@ -86,6 +87,7 @@ Strings LocalStore::readDirectoryIgnoringInodes(const Path & 
path, const InodeHa
     return names;
 }
 
+
 void LocalStore::optimisePath_(OptimiseStats & stats, const Path & path, 
InodeHash & inodeHash)
 {
     checkInterrupt();
@@ -95,7 +97,7 @@ void LocalStore::optimisePath_(OptimiseStats & stats, const 
Path & path, InodeHa
         throw SysError(format("getting attributes of path `%1%'") % path);
 
     if (S_ISDIR(st.st_mode)) {
-        Strings names = readDirectoryIgnoringInodes(path, inodeHash, stats);
+        Strings names = readDirectoryIgnoringInodes(path, inodeHash);
         foreach (Strings::iterator, i, names)
             optimisePath_(stats, path + "/" + *i, inodeHash);
         return;
@@ -117,8 +119,6 @@ void LocalStore::optimisePath_(OptimiseStats & stats, const 
Path & path, InodeHa
         return;
     }
 
-    stats.totalFiles++;
-
     /* This can still happen on top-level files */
     if (st.st_nlink > 1 && inodeHash.count(st.st_ino)) {
         printMsg(lvlDebug, format("`%1%' is already linked, with %2% other 
file(s).") % path % (st.st_nlink - 2));
@@ -158,7 +158,6 @@ void LocalStore::optimisePath_(OptimiseStats & stats, const 
Path & path, InodeHa
     if (lstat(linkPath.c_str(), &stLink))
         throw SysError(format("getting attributes of path `%1%'") % linkPath);
 
-    stats.sameContents++;
     if (st.st_ino == stLink.st_ino) {
         printMsg(lvlDebug, format("`%1%' is already linked to `%2%'") % path % 
linkPath);
         return;
diff --git a/src/nix-store/nix-store.cc b/src/nix-store/nix-store.cc
index 341a4f6..5da401c 100644
--- a/src/nix-store/nix-store.cc
+++ b/src/nix-store/nix-store.cc
@@ -801,11 +801,9 @@ static void opRepairPath(Strings opFlags, Strings opArgs)
 static void showOptimiseStats(OptimiseStats & stats)
 {
     printMsg(lvlError,
-        format("%1% freed by hard-linking %2% files; there are %3% files with 
equal contents out of %4% files in total")
+        format("%1% freed by hard-linking %2% files")
         % showBytes(stats.bytesFreed)
-        % stats.filesLinked
-        % stats.sameContents
-        % stats.totalFiles);
+        % stats.filesLinked);
 }
 
 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]