Fix clang-tidy modernize-use-equals-delete warnings in headers

src/OpaqueVal.h

@@ -328,6 +328,10 @@ public:
     explicit BloomFilterVal(probabilistic::BloomFilter* bf);
     ~BloomFilterVal() override;
 
+    // Disable.
+    BloomFilterVal(const BloomFilterVal&) = delete;
+    BloomFilterVal& operator=(const BloomFilterVal&) = delete;
+
     ValPtr DoClone(CloneState* state) override;
 
     const TypePtr& Type() const { return type; }
@@ -350,10 +354,6 @@ protected:
 
     DECLARE_OPAQUE_VALUE_DATA(BloomFilterVal)
 private:
-    // Disable.
-    BloomFilterVal(const BloomFilterVal&);
-    BloomFilterVal& operator=(const BloomFilterVal&);
-
     TypePtr type;
     detail::CompositeHash* hash;
     probabilistic::BloomFilter* bloom_filter;

src/SmithWaterman.h

@@ -40,6 +40,8 @@ public:
 
     using BSSAlignVec = std::vector<BSSAlign>;
 
+    Substring() = delete;
+
     explicit Substring(const std::string& string) : String(string), _num(), _new(false) {}
 
     explicit Substring(const String& string) : String(string), _num(), _new(false) {}
@@ -75,8 +77,6 @@ public:
 private:
     using DataMap = std::map<std::string, void*>;
 
-    Substring();
-
     // The alignments registered for this substring.
     BSSAlignVec _aligns;
 

src/input/ReaderBackend.h

@@ -132,8 +132,7 @@ public:
             }
         }
 
-    private:
+        const ReaderInfo& operator=(const ReaderInfo& other) = delete;
-        const ReaderInfo& operator=(const ReaderInfo& other); // Disable.
     };
 
     /**

src/logging/WriterBackend.h

@@ -127,8 +127,7 @@ public:
         broker::data ToBroker() const;
         bool FromBroker(broker::data d);
 
-    private:
+        const WriterInfo& operator=(const WriterInfo& other) = delete;
-        const WriterInfo& operator=(const WriterInfo& other); // Disable.
     };
 
     /**

src/probabilistic/CounterVector.h

@@ -47,6 +47,8 @@ public:
      */
     virtual ~CounterVector();
 
+    CounterVector& operator=(const CounterVector&) = delete;
+
     /**
      * Increments a given cell.
      *
@@ -157,8 +159,6 @@ protected:
     CounterVector() = default;
 
 private:
-    CounterVector& operator=(const CounterVector&); // Disable.
-
     BitVector* bits = nullptr;
     size_t width = 0;
 };

src/spicy/cookie.h

@@ -185,6 +185,9 @@ struct Cookie {
             throw std::runtime_error("invalid cookie");
     }
 
+    Cookie(const Cookie& other) = delete;
+    Cookie& operator=(const Cookie& other) = delete;
+
 private:
     union Data {
         cookie::ProtocolAnalyzer protocol;
@@ -242,9 +245,6 @@ private:
             packet = &data.packet;
     }
 
-    Cookie(const Cookie& other) = delete;
-    Cookie& operator=(const Cookie& other) = delete;
-
     friend inline void swap(Cookie& lhs, Cookie& rhs) noexcept {
         Cookie tmp = std::move(lhs);
         lhs = std::move(rhs);