about summary refs log tree commit diff
diff options
context:
space:
mode:
authorEelco Dolstra <eelco.dolstra@logicblox.com>2016-05-31T11·31+0200
committerEelco Dolstra <eelco.dolstra@logicblox.com>2016-05-31T11·31+0200
commita9fa5e050aaebe852ea5dd5c2622062db15ba3ac (patch)
treeae32a1ee0e41a03c71141700e69c5ef8d485744d
parent10f3a2e5f213df4a1d50947066613bf87a5a1008 (diff)
Shut up some clang warnings
-rw-r--r--src/libstore/binary-cache-store.hh6
-rw-r--r--src/libstore/s3-binary-cache-store.cc12
-rw-r--r--src/nix/command.hh2
-rw-r--r--src/nix/progress-bar.cc2
4 files changed, 12 insertions, 10 deletions
diff --git a/src/libstore/binary-cache-store.hh b/src/libstore/binary-cache-store.hh
index bedb4c9f0c..3f269b8cf0 100644
--- a/src/libstore/binary-cache-store.hh
+++ b/src/libstore/binary-cache-store.hh
@@ -78,10 +78,10 @@ public:
     { return {}; }
 
     void querySubstitutablePathInfos(const PathSet & paths,
-        SubstitutablePathInfos & infos)
+        SubstitutablePathInfos & infos) override
     { }
 
-    bool wantMassQuery() { return wantMassQuery_; }
+    bool wantMassQuery() override { return wantMassQuery_; }
 
     void addToStore(const ValidPathInfo & info, const std::string & nar,
         bool repair = false, bool dontCheckSigs = false) override;
@@ -128,7 +128,7 @@ public:
 
     ref<FSAccessor> getFSAccessor() override;
 
-    void addSignatures(const Path & storePath, const StringSet & sigs)
+    void addSignatures(const Path & storePath, const StringSet & sigs) override
     { notImpl(); }
 
 };
diff --git a/src/libstore/s3-binary-cache-store.cc b/src/libstore/s3-binary-cache-store.cc
index 6ee27b48d6..371bf24fd4 100644
--- a/src/libstore/s3-binary-cache-store.cc
+++ b/src/libstore/s3-binary-cache-store.cc
@@ -56,7 +56,7 @@ struct S3BinaryCacheStoreImpl : public S3BinaryCacheStore
         diskCache = getNarInfoDiskCache();
     }
 
-    std::string getUri()
+    std::string getUri() override
     {
         return "s3://" + bucketName;
     }
@@ -69,7 +69,7 @@ struct S3BinaryCacheStoreImpl : public S3BinaryCacheStore
         return res;
     }
 
-    void init()
+    void init() override
     {
         if (!diskCache->cacheExists(getUri())) {
 
@@ -108,7 +108,7 @@ struct S3BinaryCacheStoreImpl : public S3BinaryCacheStore
        fetches the .narinfo file, rather than first checking for its
        existence via a HEAD request. Since .narinfos are small, doing
        a GET is unlikely to be slower than HEAD. */
-    bool isValidPathUncached(const Path & storePath)
+    bool isValidPathUncached(const Path & storePath) override
     {
         try {
             queryPathInfo(storePath);
@@ -118,7 +118,7 @@ struct S3BinaryCacheStoreImpl : public S3BinaryCacheStore
         }
     }
 
-    bool fileExists(const std::string & path)
+    bool fileExists(const std::string & path) override
     {
         stats.head++;
 
@@ -138,7 +138,7 @@ struct S3BinaryCacheStoreImpl : public S3BinaryCacheStore
         return true;
     }
 
-    void upsertFile(const std::string & path, const std::string & data)
+    void upsertFile(const std::string & path, const std::string & data) override
     {
         auto request =
             Aws::S3::Model::PutObjectRequest()
@@ -167,7 +167,7 @@ struct S3BinaryCacheStoreImpl : public S3BinaryCacheStore
         stats.putTimeMs += duration;
     }
 
-    std::shared_ptr<std::string> getFile(const std::string & path)
+    std::shared_ptr<std::string> getFile(const std::string & path) override
     {
         debug(format("fetching ‘s3://%1%/%2%’...") % bucketName % path);
 
diff --git a/src/nix/command.hh b/src/nix/command.hh
index 34affc43d9..a29cdcf7f5 100644
--- a/src/nix/command.hh
+++ b/src/nix/command.hh
@@ -49,6 +49,8 @@ public:
 
     StorePathsCommand();
 
+    using StoreCommand::run;
+
     virtual void run(ref<Store> store, Paths storePaths) = 0;
 
     void run(ref<Store> store) override;
diff --git a/src/nix/progress-bar.cc b/src/nix/progress-bar.cc
index 659d6572ad..69811b2828 100644
--- a/src/nix/progress-bar.cc
+++ b/src/nix/progress-bar.cc
@@ -89,7 +89,7 @@ public:
         state->progress[label].expected += value;
     }
 
-    void incProgress(const std::string & label, uint64_t value)
+    void incProgress(const std::string & label, uint64_t value) override
     {
         auto state(state_.lock());
         state->progress[label].progress += value;