Make classes final if they're not meant to be derived from

assert/AssertFailed.h

@@ -4,10 +4,11 @@
 
 #include <stdexcept>
 #include <string>
+#include "../macros.h"
 
 namespace cpputils {
 
-    class AssertFailed : public std::exception {
+    class AssertFailed final: public std::exception {
     public:
         AssertFailed(const std::string &message) : _message(message) { }
 

crypto/RandomPadding.h

@@ -7,7 +7,7 @@
 
 namespace cpputils {
     //TODO Test
-    class RandomPadding {
+    class RandomPadding final {
     public:
         static Data add(const Data &data, size_t targetSize);
         static boost::optional<Data> remove(const Data &data);

crypto/kdf/DerivedKey.h

@@ -8,7 +8,7 @@
 namespace cpputils {
 
     template<size_t KEY_LENGTH>
-    class DerivedKey {
+    class DerivedKey final {
     public:
         DerivedKey(DerivedKeyConfig config, const FixedSizeData<KEY_LENGTH> &key): _config(std::move(config)), _key(key) {}
         DerivedKey(DerivedKey &&rhs) = default;

crypto/kdf/DerivedKeyConfig.h

@@ -13,7 +13,7 @@ namespace cpputils {
     //TODO Test operator==/!=
     //TODO Use SCryptSettings as a member here instead of storing _N, _r, _p.
 
-    class DerivedKeyConfig {
+    class DerivedKeyConfig final {
     public:
         DerivedKeyConfig(Data salt, uint64_t N, uint32_t r, uint32_t p)
                 : _salt(std::move(salt)),

data/DataFixture.h

@@ -7,7 +7,7 @@
 
 namespace cpputils {
 
-class DataFixture {
+class DataFixture final {
 public:
   static Data generate(size_t size, long long int seed = 1);
 

io/Console.h

@@ -6,6 +6,7 @@
 #include <vector>
 #include <iostream>
 #include <boost/optional.hpp>
+#include "../macros.h"
 
 namespace cpputils {
 
@@ -17,7 +18,7 @@ public:
     virtual void print(const std::string &output) = 0;
 };
 
-class IOStreamConsole: public Console {
+class IOStreamConsole final: public Console {
 public:
     IOStreamConsole();
     IOStreamConsole(std::ostream &output, std::istream &input);
@@ -30,6 +31,8 @@ private:
 
     std::ostream &_output;
     std::istream &_input;
+
+    DISALLOW_COPY_AND_ASSIGN(IOStreamConsole);
 };
 
 }

io/pipestream.h

@@ -53,7 +53,7 @@
 
 namespace cpputils {
 
-    class pipestream : public std::streambuf {
+    class pipestream final : public std::streambuf {
     private:
         typedef std::streambuf::traits_type traits_type;
         typedef std::string::size_type string_size_t;

lock/ConditionBarrier.h

@@ -12,7 +12,7 @@ namespace cpputils {
     // Like a condition variable, but without spurious wakeups.
     // The waiting threads are only woken, when notify() is called.
     // After a call to release(), future calls to wait() will not block anymore.
-    class ConditionBarrier {
+    class ConditionBarrier final {
     public:
         ConditionBarrier() :_mutex(), _cv(), _triggered(false) {
         }
@@ -33,6 +33,8 @@ namespace cpputils {
         std::mutex _mutex;
         std::condition_variable _cv;
         bool _triggered;
+
+        DISALLOW_COPY_AND_ASSIGN(ConditionBarrier);
     };
 }
 

lock/MutexPoolLock.h

@@ -6,7 +6,7 @@
 
 namespace cpputils {
     template<class LockName>
-    class MutexPoolLock {
+    class MutexPoolLock final {
     public:
         MutexPoolLock(LockPool<LockName> *pool, const LockName &lockName): _pool(pool), _lockName(lockName) {
             _pool->lock(_lockName);

logging/Logger.h

@@ -7,7 +7,7 @@
 
 namespace cpputils {
 namespace logging {
-    class Logger {
+    class Logger final {
     public:
         void setLogger(std::shared_ptr<spdlog::logger> logger) {
             _logger = logger;

process/subprocess.h

@@ -3,15 +3,18 @@
 #define MESSMER_CPPUTILS_PROCESS_SUBPROCESS_H
 
 #include <string>
+#include "../macros.h"
 
 namespace cpputils {
     //TODO Test
-    class Subprocess {
+    class Subprocess final {
     public:
         static std::string call(const std::string &command);
         static int callAndGetReturnCode(const std::string &command);
     private:
         static FILE* _call(const std::string &command);
+
+        DISALLOW_COPY_AND_ASSIGN(Subprocess);
     };
 }
 

random/OSRandomGenerator.h

@@ -12,6 +12,9 @@ namespace cpputils {
 
     protected:
         void _get(void *target, size_t bytes) override;
+
+    private:
+        DISALLOW_COPY_AND_ASSIGN(OSRandomGenerator);
     };
 
     inline OSRandomGenerator::OSRandomGenerator() {}

random/Random.h

@@ -9,7 +9,7 @@
 #include <mutex>
 
 namespace cpputils {
-    class Random {
+    class Random final {
     public:
         static PseudoRandomPool &PseudoRandom() {
             std::unique_lock <std::mutex> lock(_mutex);
@@ -25,6 +25,8 @@ namespace cpputils {
 
     private:
         static std::mutex _mutex;
+
+        DISALLOW_COPY_AND_ASSIGN(Random);
     };
 }
 

random/RandomDataBuffer.h

@@ -7,7 +7,7 @@
 
 namespace cpputils {
     //TODO Test
-    class RandomDataBuffer {
+    class RandomDataBuffer final {
     public:
         RandomDataBuffer();
 

random/RandomGenerator.h

@@ -7,6 +7,8 @@
 namespace cpputils {
     class RandomGenerator {
     public:
+        RandomGenerator();
+
         template<size_t SIZE> FixedSizeData<SIZE> getFixedSize();
         Data get(size_t size);
 
@@ -14,8 +16,13 @@ namespace cpputils {
         virtual void _get(void *target, size_t bytes) = 0;
     private:
         static std::mutex _mutex;
+
+        DISALLOW_COPY_AND_ASSIGN(RandomGenerator);
     };
 
+    inline RandomGenerator::RandomGenerator() {
+    }
+
     template<size_t SIZE> inline FixedSizeData<SIZE> RandomGenerator::getFixedSize() {
         FixedSizeData<SIZE> result = FixedSizeData<SIZE>::Null();
         _get(result.data(), SIZE);

random/RandomGeneratorThread.h

@@ -8,7 +8,7 @@
 
 namespace cpputils {
     //TODO Test
-    class RandomGeneratorThread {
+    class RandomGeneratorThread final {
     public:
         RandomGeneratorThread(ThreadsafeRandomDataBuffer *buffer, size_t minSize, size_t maxSize);
 

random/ThreadsafeRandomDataBuffer.h

@@ -9,7 +9,7 @@
 
 namespace cpputils {
     //TODO Test
-    class ThreadsafeRandomDataBuffer {
+    class ThreadsafeRandomDataBuffer final {
     public:
         ThreadsafeRandomDataBuffer();